-
Notifications
You must be signed in to change notification settings - Fork 273
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ui5-tabcontainer - tab is selected in a collapsed tab container, ui5-tab "title" attribute doesn't work #6406
Comments
As first step we should mark the Tab and TabSeparator as |
We'll keep the selection indicator, when the content is collapsed, as we want tab selection, when the control is always collapsed. In the OpenUI5 there is a different control for that. |
Hi Teo, Kind regards |
Hi @blaesius, OpenUI5 IconTabHeader control: This functionality can be achieved in ui5 web components by:
https://codesandbox.io/s/ui5-webcomponents-forked-psue9q Best, |
Ok, if this is needed, then it should only apply for this combination (collapsed + fixed tabcontainer) and not for the other combinations (e.g. if you collapse a 'collapsable' TabContainer the selection should not be shown) |
For me this is an unnecessary complication and it will leads to confusion when exactly the selection is shown or not. Anyway if you think this will help for the control usability, reopen the issue and we'll improve this. |
If you can collapse and expand the Tab Container, the selection must only be visible while the Tab is expanded. The standalone Tab Container Header is a special use case and only for this use case the other behavior (having always the selection shown) can apply. This can either happen automatically or via a property. As we are mixing multiple existing UI5 controls in this Web Component. To be consistent, we need to apply the same behavior for UI5 and WC. This means, if you can expand/collapse the Tab Container, the selection should only appear if the container is expanded (see https://ui5.sap.com/#/entity/sap.m.IconTabBar/sample/sap.m.sample.IconTabBarMulti) Another solution would also be to remove the expand/collapse feature. |
@blaesius, I would go for removing the expand/collapse via mouse/keyboard feature, but keep it as an API. So the ui5-tabcontainer will be always "fixed". |
Yes, that sounds good |
Added to #4460 2.0 breaking changes list. |
Expected Behavior
Expected Behavior
Steps to Reproduce
Priority
The text was updated successfully, but these errors were encountered: