-
Notifications
You must be signed in to change notification settings - Fork 272
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-wizard): improve progress navigator responsiveness #2590
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.
the number / icon in the circle in not correctly centered. (check compact as wel)
} | ||
} | ||
|
||
_adjustHeaderOverflow() { |
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.
add some comments and try to split this a little bit.
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.
add some comments and try to split this a little bit.
Done
packages/fiori/src/themes/Wizard.css
Outdated
user-select: none; | ||
background: #fff; | ||
font-size: .875rem; | ||
box-shadow: 0 0 0.25rem 0 rgba(0,0,0,0.15), inset 0 -0.0625rem 0 0 #d9d9d9; |
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.
use css vars
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.
use css vars
Done
-moz-user-select: none; | ||
-ms-user-select: none; | ||
user-select: none; | ||
background: #fff; |
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.
vars?
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.
vars?
Done
|
* [WIP] ui5-wizard: improve progress navigator responsiveness * feat(ui5-wizard): responsive behaviour * ui5-wizard: selection-change fired as expected * ui5-wizard: minor enhancements * ui5-wizard: CSS adjustments * ui5-wizard: CSS adjustments * [WIP] ui5-wizard: improve progress navigator responsiveness * feat(ui5-wizard): responsive behaviour * ui5-wizard: selection-change fired as expected * ui5-wizard: minor enhancements * ui5-wizard: CSS adjustments * css improvements * ui5-wizard: IE fix and other minor adjustments * ui5-wizard: attributes names changed Co-authored-by: Martin Hristov <[email protected]>
Overview
Wizard's navigation header is now responsive. On small sizes, when the space is not enough for the all ui5-wizard-tab instances to be fully visible, the are stacked in a group. Such a group could be positioned in the beginning or in the end of the wizard's navigation header, depending on the user interaction.
When a tab is stacked, it could be activated from a popover, which is presented on a click over the top most of the grouped tabs. The popover contains a button for each grouped tab. On mobile devices, a dialog is used.
For a disabled step, the button in the popover is also disabled, since it is not possible to activate such step.
Preview
WizardProgressNavigator.mov
FIXES: #2441