Make sure button-primary has button class

[iseulde]

This patch:

1. Adds the button class to all primary buttons. This is necessary to make them responsive.
2. Removes all secondary-buttonclasses and replaces it with button when needed. button-seconarydoes the same as button, but it's not responsive. It shouldn't be used.
3. Replaces classes inside submit_button() with a shorthand for some buttons, a shorthand is used most of the time.

[DrewAPicture]

Patch needs a refresh and feedback from the UI folks.

[chris_d2d]

I'm taking a look at this for #wcus-contributor-day, let me know in slack (chrissmith) if you want to collaborate or are already working on this.

[dimchik]

Updated the patch to incorporate the latest changes to the related files.

[chris_d2d]

I found a couple more locations where button-secondary could be removed:
src/wp-includes/customize/class-wp-customize-nav-menu-control.php
src/wp-includes/customize/class-wp-customize-new-menu-section.php
src/wp-includes/customize/class-wp-widget-area-customize-control.php

These are related to #19910's move away from switch statements for the customizer.

[helen]

Widgets in the customizer is using .button-secondary in the JS, that needs to be considered here. We could probably use some documentation somewhere about what the classes are and what remains as back-compat. That can just be in this ticket for now while we figure it out.

[DrewAPicture]

Assigning to mark the good-first-bug as "claimed".

[Presskopp]

related #37448

[afercia]

Related: #37138, #37448

[afercia]

It would be nice to fix this for 4.7 together with #37138 and #37448 🙂

[afercia]

In 38616:

Accessibility: Standardize the remove/delete/cancel links in the Menus screen and Publish meta boxes.

The submitdelete CSS class is used in various places across the admin for some
"red" action links. It is worth simplifying this rule for further improvements
related to color contrast.

• simplifies a non-standard styling for the "Remove/Cancel" links in the Menus screen
• underlines all the "Move to trash/Delete" red links in all the Publish meta boxes
• fixes CSS classes usage for all the Publish meta boxes primary buttons
• fixes broken layout for the old Link Manager publish meta box

Props karmatosed, hugobaeta, monikarao, afercia.
Fixes #37969, #37018. See #37448, #37138, #27314.

[afercia]

Refreshed patch. Tried to address all the points mentioned in the ticket description and comments. Additionally, there's a mixed situation now where submit_button() is sometimes used passing an empty string for the CSS class, and sometimes passing button (alone or together with other classes). Since passing button is unnecessary, I'd propose to standardise and remove it. This implies a small adjustment to get_submit_button() in order to remove empty items to avoid a single class button being printed out with a space at the end class="button ".

The patch is pretty big and would need to be merged as soon as possible :) Any feedback welcome, see also the QUnit tests part. /cc @helen @iseulde

[afercia]

In 38672:

Administration: Improve the usage of the button CSS classes.

Introduces some consistency in the usage of the button CSS classes, fixes the
focus style for accessibility and responsiveness of the buttons.

• Adds the button class to all primary buttons make them responsive
• Removes all secondary-button classes and replaces it with button when needed. button-secondary shouldn't be used and exists just for backward compatibility reasons
• Replaces classes inside submit_button() with a shorthand for some buttons, and use an empty string for the default button class. Passing button is unnecessary
• Adjusts get_submit_button() to remove empty items

Props iseulde, dimchik, chris_d2d, mhowell, afercia.
Fixes #27314, #37138, #37448.