-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Primary Actions Not Emphasized #2254
Labels
Comments
Hello @boghyon , Thank you for sharing your enhancement proposal. I've created an internal incident 1880659270. The status of the issue will be updated here in GitHub. Regards, |
boghyon
changed the title
Main Actions Are Not Emphasized in Some Controls
Primary Actions Not Emphasized
Dec 1, 2018
boghyon
referenced
this issue
Dec 21, 2018
Change-Id: I443e247d7653e7a8cb126055d7e8a20479b06e2a JIRA: BGSOFUIRODOPI-1891
This was referenced Aug 23, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
According to the Fiori Design Guideline as well as many other UX guidelines, the primary action on the screen should be emphasized.
Also worth watching: How to Communicate Primary Actions (by Luke Wroblewski)
URL (minimal example if possible)
Controls with main / primary action but not yet emphasized:
→ Fixed by ba37f72sap.m.TablePersoDialog
→ Fixed by cb93939 and 1f421b0sap.m.SelectDialog
(multi select)→ Fixed by cb93939 and 1f421b0sap.m.TableSelectDialog
(multi select)→ Fixed by b3d36b0sap.m.DateTimePicker
→ Fixed by b3d36b0sap.m.TimePicker
→ Fixed as of SAPUI5 1.63.0sap.ui.comp.variants.VariantManagement
→ Fixed by dd07f84, closes [INTERNAL] sap.ui.unified.ColorPickerPopover: emphasize primary action #2642sap.ui.unified.ColorPickerPopover
Various→ Fixed by 8b2d9f7 and 3de1f89sap.m.MessageBox
dialogs if applicable(Deprecated) → Fixed by b2feb2dsap.m.UploadCollection
What is the expected result?
Main action buttons such as OK or Submit are emphasized like in the ValueHelpDialog or ViewSettingsDialog for example.
The text was updated successfully, but these errors were encountered: