-
Notifications
You must be signed in to change notification settings - Fork 254
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
feat(ui5-tabcontainer): content can be displayed above the tab strip #1516
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me. I just have one comment regarding the property name. I feel like the tabs position should be leading in the naming the property. Maybe sth similar to the Carousel arrowsPlacement="Content|Navigation" can be appropriate here as well, like: tabsPlacement="Top|Bottom, Top by default".
If it should be a boolean: bottom-navigation navigation-bottom, tabs-to-bottom. But, it is just an opinion, I will approve it as it is.
I'll change the name, what you said makes sense |
packages/main/src/TabContainer.js
Outdated
* bottom of the page and you want the tab strip to act as a menu. | ||
* | ||
* @type {boolean} | ||
* @defaultvalue false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type TabContainerTabsPlacement
defaultvalue Top
since 1.0.0-rc.7
The new
tabsPlacement
property (Top
by default) lets the user position the content area above the tab strip by settingtabsPlacement
toBottom
. By doing so, the overflow menu arrow also changes direction to point upwards, rather than downwards.