From ee7f970ffa98d82c1a04e8f8a39b2f67af4c4363 Mon Sep 17 00:00:00 2001
From: Andrea Fercia
Date: Wed, 28 Sep 2016 19:53:07 +0000
Subject: [PATCH] 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.
git-svn-id: https://develop.svn.wordpress.org/trunk@38672 602fd350-edb4-49c9-b593-d223f7449a82
---
src/wp-admin/css/customize-controls.css | 2 +-
src/wp-admin/css/ie.css | 9 ++++---
src/wp-admin/css/install.css | 2 +-
src/wp-admin/css/list-tables.css | 1 +
src/wp-admin/css/themes.css | 2 +-
src/wp-admin/custom-background.php | 6 ++---
src/wp-admin/custom-header.php | 8 +++----
src/wp-admin/customize.php | 2 +-
.../includes/class-wp-comments-list-table.php | 2 +-
.../includes/class-wp-links-list-table.php | 2 +-
src/wp-admin/includes/class-wp-list-table.php | 2 +-
.../includes/class-wp-media-list-table.php | 2 +-
.../class-wp-plugin-install-list-table.php | 2 +-
.../includes/class-wp-plugins-list-table.php | 4 ++--
.../includes/class-wp-posts-list-table.php | 8 +++----
.../includes/class-wp-terms-list-table.php | 4 ++--
.../class-wp-theme-install-list-table.php | 8 +++----
.../includes/class-wp-users-list-table.php | 2 +-
src/wp-admin/includes/dashboard.php | 4 ++--
src/wp-admin/includes/file.php | 2 +-
src/wp-admin/includes/media.php | 14 +++++------
src/wp-admin/includes/meta-boxes.php | 6 ++---
src/wp-admin/includes/nav-menu.php | 10 ++++----
src/wp-admin/includes/plugin-install.php | 4 ++--
src/wp-admin/includes/template.php | 13 +++++-----
src/wp-admin/includes/theme-install.php | 6 ++---
src/wp-admin/includes/widgets.php | 4 ++--
src/wp-admin/install.php | 2 +-
src/wp-admin/js/customize-widgets.js | 2 +-
src/wp-admin/js/updates.js | 2 +-
src/wp-admin/js/widgets.js | 4 ++--
src/wp-admin/nav-menus.php | 2 +-
src/wp-admin/network/themes.php | 6 ++---
src/wp-admin/plugin-editor.php | 2 +-
src/wp-admin/plugins.php | 4 ++--
src/wp-admin/theme-editor.php | 2 +-
src/wp-admin/theme-install.php | 10 ++++----
src/wp-admin/themes.php | 24 +++++++++----------
src/wp-admin/tools.php | 4 ++--
src/wp-admin/update-core.php | 8 +++----
src/wp-admin/user-edit.php | 12 +++++-----
src/wp-admin/user-new.php | 6 ++---
src/wp-admin/widgets.php | 8 +++----
.../class-wp-customize-nav-menus.php | 2 +-
.../class-wp-customize-widgets.php | 2 +-
.../class-wp-customize-nav-menu-control.php | 2 +-
.../class-wp-customize-new-menu-section.php | 2 +-
...class-wp-widget-area-customize-control.php | 2 +-
src/wp-includes/general-template.php | 2 +-
tests/qunit/fixtures/customize-menus.js | 2 +-
tests/qunit/fixtures/customize-widgets.js | 4 ++--
tests/qunit/index.html | 4 ++--
52 files changed, 126 insertions(+), 125 deletions(-)
diff --git a/src/wp-admin/css/customize-controls.css b/src/wp-admin/css/customize-controls.css
index 3f2193b286..04e0b54798 100644
--- a/src/wp-admin/css/customize-controls.css
+++ b/src/wp-admin/css/customize-controls.css
@@ -1204,7 +1204,7 @@ body.cheatin p {
* Widgets and Menus common styles
*/
-/* higher specificity than .wp-core-ui .button-secondary */
+/* higher specificity than .wp-core-ui .button */
#customize-theme-controls .add-new-widget,
#customize-theme-controls .add-new-menu-item {
cursor: pointer;
diff --git a/src/wp-admin/css/ie.css b/src/wp-admin/css/ie.css
index 90cf8f341c..d98f31bbcc 100644
--- a/src/wp-admin/css/ie.css
+++ b/src/wp-admin/css/ie.css
@@ -221,8 +221,7 @@ table.fixed td {
}
#wpbody-content input.button,
-#wpbody-content input.button-primary,
-#wpbody-content input.button-secondary {
+#wpbody-content input.button-primary {
overflow: visible;
}
@@ -447,8 +446,8 @@ div#dashboard-widgets {
display: block;
}
-.tablenav .button-secondary,
-.nav .button-secondary {
+.tablenav .button,
+.nav .button {
padding-top: 2px;
padding-bottom: 2px;
}
@@ -507,7 +506,7 @@ table.ie-fixed {
border-collapse: collapse;
}
-.tablenav a.button-secondary {
+.tablenav a.button {
display: inline-block;
padding: 2px 5px;
}
diff --git a/src/wp-admin/css/install.css b/src/wp-admin/css/install.css
index d1db4ec661..9de99ee366 100644
--- a/src/wp-admin/css/install.css
+++ b/src/wp-admin/css/install.css
@@ -443,7 +443,7 @@ body.language-chooser {
margin-right: 15px;
}
-.button-secondary.hide-if-no-js,
+.button.hide-if-no-js,
.hide-if-no-js {
display: none;
}
diff --git a/src/wp-admin/css/list-tables.css b/src/wp-admin/css/list-tables.css
index b66f0c0d84..67ef22e183 100644
--- a/src/wp-admin/css/list-tables.css
+++ b/src/wp-admin/css/list-tables.css
@@ -656,6 +656,7 @@ th.asc a:focus span.sorting-indicator {
margin-left: 2px;
}
+.tablenav a.button,
.tablenav a.button-secondary {
display: block;
margin: 3px 8px 0 0;
diff --git a/src/wp-admin/css/themes.css b/src/wp-admin/css/themes.css
index d4a7bcf624..857f14dc1a 100644
--- a/src/wp-admin/css/themes.css
+++ b/src/wp-admin/css/themes.css
@@ -114,7 +114,7 @@
margin-right: 3px;
}
-.theme-browser .theme .theme-actions .button-secondary {
+.theme-browser .theme .theme-actions .button {
float: none;
margin-left: 3px;
}
diff --git a/src/wp-admin/custom-background.php b/src/wp-admin/custom-background.php
index a03986f594..9626c291a0 100644
--- a/src/wp-admin/custom-background.php
+++ b/src/wp-admin/custom-background.php
@@ -242,7 +242,7 @@ class Custom_Background {
|
|
@@ -256,7 +256,7 @@ class Custom_Background {
|
@@ -272,7 +272,7 @@ class Custom_Background {
-
+
diff --git a/src/wp-admin/custom-header.php b/src/wp-admin/custom-header.php
index fc5adb670e..c339498c11 100644
--- a/src/wp-admin/custom-header.php
+++ b/src/wp-admin/custom-header.php
@@ -576,7 +576,7 @@ class Custom_Image_Header {
-
+
-
+
|
-
+
|
@@ -823,7 +823,7 @@ wp_nonce_field( 'custom-header-options', '_wpnonce-custom-header-options' ); ?>
diff --git a/src/wp-admin/customize.php b/src/wp-admin/customize.php
index 6c0b20d6c2..e45254c74a 100644
--- a/src/wp-admin/customize.php
+++ b/src/wp-admin/customize.php
@@ -168,7 +168,7 @@ do_action( 'customize_controls_print_scripts' );
-