Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
On the extend/modules 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.
Comment | File | Size | Author |
---|---|---|---|
#5 | 2469889-button-type-modulepage-5.png | 101.27 KB | ckrina |
#3 | 2469889-button-type-modulepage-3.patch | 531 bytes | joshi.rohit100 |
Comments
Comment #1
LewisNyman CreditAttribution: LewisNyman commentedComment #2
LewisNyman CreditAttribution: LewisNyman commentedComment #3
joshi.rohit100Done.
Comment #4
vpeltot CreditAttribution: vpeltot commentedThis change is OK for me.
Comment #5
ckrinaIt applies fine for me and also looks ok.
Comment #6
alexpottCommitted 4d9c9ff and pushed to 8.0.x. Thanks!