Problem/Motivation
On the appearance page (/admin/appearance) the "Save configuration" button is styled as a standard button when it should be styled as a primary button.
There are three types of buttons supported in core and in the Seven style guide. Standard, primary, and danger.
It's important that the correct button types are used consistently throughout Drupal
Remaining tasks
Add the primary #button_type class to that form item, so it receives the correct class.
Beta phase evaluation
| Issue category | Bug because usability standards |
|---|---|
| Issue priority | Not critical because minor defect |
| Unfrozen changes | Unfrozen because it only changes markup |
| Prioritized changes | The main goal of this issue is usability |
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 2469921-theme-primary-button-3.png | 168.94 KB | ckrina |
| #2 | 2469921-theme-primary-button-2.patch | 551 bytes | joshi.rohit100 |
Comments
Comment #1
lewisnymanComment #2
joshi.rohit100Done.
Comment #3
ckrinaThe patch applies fine and looks ok. I'm attaching an screenshot with be before/after.
Comment #4
lewisnymanComment #5
alexpottCommitted ff04963 and pushed to 8.0.x. Thanks!