Widgets: Improve the News dashboard widget no-js notice.

In the rare case this notice was displayed, it was unstyled. Uses the notice 
error style to make it prettier and consistent with other no-js notices.
Also, some CSS clean-up.

Fixes #40529.


git-svn-id: https://develop.svn.wordpress.org/trunk@40556 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Andrea Fercia 2017-04-25 16:41:26 +00:00
parent 626547e9cf
commit 5d027d64fd
2 changed files with 10 additions and 8 deletions

View File

@ -308,19 +308,21 @@
padding: 0;
}
#dashboard_primary .widget-loading,
#dashboard_primary .dashboard-widget-control-form {
#dashboard_primary .widget-loading {
padding: 12px 12px 0;
margin-bottom: 1em !important; /* Needs to override `.postbox .inside > p:last-child` in common.css */
}
/* Notice when JS is off. */
#dashboard_primary .inside .notice {
margin: 0;
}
body #dashboard-widgets .postbox form .submit {
margin: 0;
}
.dashboard-widget-control-form {
overflow: hidden;
}
/* Used only for configurable widgets. */
.dashboard-widget-control-form p {
margin-top: 0;
}

View File

@ -191,7 +191,7 @@ function wp_add_dashboard_widget( $widget_id, $widget_name, $callback, $control_
* @param array $meta_box
*/
function _wp_dashboard_control_callback( $dashboard, $meta_box ) {
echo '<form method="post" class="dashboard-widget-control-form">';
echo '<form method="post" class="dashboard-widget-control-form wp-clearfix">';
wp_dashboard_trigger_widget_control( $meta_box['id'] );
wp_nonce_field( 'edit-dashboard-widget_' . $meta_box['id'], 'dashboard-widget-nonce' );
echo '<input type="hidden" name="widget_id" value="' . esc_attr($meta_box['id']) . '" />';
@ -968,7 +968,7 @@ function wp_dashboard_rss_output( $widget_id ) {
* @return bool False on failure. True on success.
*/
function wp_dashboard_cached_rss_widget( $widget_id, $callback, $check_urls = array() ) {
$loading = '<p class="widget-loading hide-if-no-js">' . __( 'Loading&#8230;' ) . '</p><p class="hide-if-js">' . __( 'This widget requires JavaScript.' ) . '</p>';
$loading = '<p class="widget-loading hide-if-no-js">' . __( 'Loading&#8230;' ) . '</p><div class="hide-if-js notice notice-error inline"><p>' . __( 'This widget requires JavaScript.' ) . '</p></div>';
$doing_ajax = wp_doing_ajax();
if ( empty($check_urls) ) {