Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
On the permissions page, the main button on the page is a standard button, when it should be 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 | Feature because it does not meet usability standards |
|---|---|
| Issue priority | Not critical because it's a minor usability problem |
| Unfrozen changes | Unfrozen because it only changes markup |
| Prioritized changes | The main goal of this issue is usability |
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | save_permissions.jpg | 200.7 KB | pakmanlh |
| #2 | 2469939-button-type-permission-page-2.patch | 758 bytes | joshi.rohit100 |
Comments
Comment #1
LewisNymanComment #2
joshi.rohit100Done.
Comment #3
pakmanlhLooks nice.
Comment #4
LewisNymanComment #5
alexpottCommitted 68ca854 and pushed to 8.0.x. Thanks!
Thanks for adding the beta evaluation to the issue summary.