WordPress.org
Get WordPress

Make WordPress Core

Ticket #21283: 21283.10.patch

File 21283.10.patch, 9.3 KB (added by JustinSainton, 12 years ago)

Patch with nacin's suggestions

  • wp-admin/css/customize-controls.css

     
    6767
    6868.control-section:hover .customize-section-title,
    6969.control-section .customize-section-title:hover,
    70 .control-section.open .customize-section-title {
     70.control-section.open .customize-section-title,
     71.customize-section-title:focus {
    7172        color: #fff;
    7273        text-shadow: 0 -1px 0 #333;
    7374        background: #808080;
    … …  
    126127        color: #999;
    127128}
    128129
     130#customize-info .customize-section-title:focus .preview-notice {
     131        color: #ccc;
     132        text-shadow: 0 -1px 0 #333;
     133}
     134
    129135#customize-info .theme-name {
    130136        font-size: 20px;
    131137        font-weight: 200;
    … …  
    135141        text-shadow: 0 1px 0 #fff;
    136142}
    137143
     144#customize-info .customize-section-title:focus .theme-name {
     145        color: #fff;
     146        text-shadow: 0 -1px 0 #333;
     147}
     148
    138149#customize-info .theme-screenshot {
    139150        width: 258px;
    140151        border: 1px solid #ccc;
    … …  
    155166        margin: 0;
    156167}
    157168
     169#customize-footer-actions .button-primary,
    158170#customize-header-actions .button-primary {
    159171        float: right;
    160172        margin-top: 10px;
    161173}
    162174
     175#customize-footer-actions .spinner,
    163176#customize-header-actions .spinner {
    164177        margin-top: 16px;
    165178        margin-right: 4px;
    166179}
    167180
     181.saving #customize-footer-actions .spinner,
    168182.saving #customize-header-actions .spinner {
    169183        display: block;
    170184}

  • wp-admin/css/wp-admin.css

     
    55715571        height: 100%;
    55725572}
    55735573
    5574 .wp-full-overlay-sidebar .wp-full-overlay-header,
    5575 .wp-full-overlay-sidebar .wp-full-overlay-footer {
     5574.wp-full-overlay-sidebar .wp-full-overlay-header {
    55765575        position: absolute;
    55775576        left: 0;
    55785577        right: 0;
    … …  
    55835582        margin: 0;
    55845583}
    55855584
    5586 .wp-full-overlay-sidebar .wp-full-overlay-header {
    5587         top: 0;
    5588         border-top: 0;
    5589         border-bottom: 1px solid #fff;
    5590         box-shadow: inset 0 -1px 0 0px #dfdfdf;
    5591 }
    5592 
    55935585.wp-full-overlay-sidebar .wp-full-overlay-footer {
    55945586        bottom: 0;
    55955587        border-bottom: 0;
    … …  
    56385630        display: block;
    56395631        width: 15px;
    56405632        height: 15px;
    5641         background: transparent url('../images/arrows.png') no-repeat 0 -72px;
     5633        background: transparent url('../images/arrows.png') no-repeat -1px -73px;
    56425634}
    56435635
    56445636.wp-full-overlay.collapsed .collapse-sidebar-arrow {
    5645         background-position: 0 -108px;
     5637        background-position: -1px -109px;
    56465638}
    56475639
    56485640.wp-full-overlay .collapse-sidebar-label {

  • wp-admin/customize.php

     
    6767<body class="<?php echo esc_attr( $body_class ); ?>">
    6868<div class="wp-full-overlay expanded">
    6969        <form id="customize-controls" class="wrap wp-full-overlay-sidebar">
     70               
    7071                <div id="customize-header-actions" class="wp-full-overlay-header">
    7172                        <?php
    7273                                $save_text = $wp_customize->is_theme_active() ? __( 'Save &amp; Publish' ) : __( 'Save &amp; Activate' );
    73                                 submit_button( $save_text, 'primary', 'save', false );
     74                                submit_button( $save_text, 'primary save', 'save', false );
    7475                        ?>
    7576                        <span class="spinner"></span>
    7677                        <a class="back button" href="<?php echo esc_url( $return ? $return : admin_url( 'themes.php' ) ); ?>">
    7778                                <?php _e( 'Cancel' ); ?>
    7879                        </a>
    7980                </div>
    80 
     81               
    8182                <?php
    8283                        $screenshot = $wp_customize->theme()->get_screenshot();
    8384                        $cannot_expand = ! ( $screenshot || $wp_customize->theme()->get('Description') );
    8485                ?>
    8586
    86                 <div class="wp-full-overlay-sidebar-content">
     87                <div class="wp-full-overlay-sidebar-content" tabindex="-1">
    8788                        <div id="customize-info" class="customize-section<?php if ( $cannot_expand ) echo ' cannot-expand'; ?>">
    88                                 <div class="customize-section-title">
     89                                <div class="customize-section-title" aria-label="<?php esc_attr_e( 'Theme Customizer Options' ); ?>" tabindex="0">
    8990                                        <span class="preview-notice"><?php
    9091                                                /* translators: %s is the theme name in the Customize/Live Preview pane */
    9192                                                echo sprintf( __( 'You are previewing %s' ), '<strong class="theme-name">' . $wp_customize->theme()->display('Name') . '</strong>' );

  • wp-admin/js/customize-controls.js

     
    9494                                        else
    9595                                                statuses.hide();
    9696                                };
    97 
     97                       
     98                        var toggleFreeze = false;
     99                       
    98100                        // Support the .dropdown class to open/close complex elements
    99                         this.container.on( 'click', '.dropdown', function( event ) {
     101                        this.container.on( 'click focus', '.dropdown', function( event ) {
    100102                                event.preventDefault();
    101                                 control.container.toggleClass('open');
     103                               
     104                                if (!toggleFreeze)
     105                                        control.container.toggleClass('open');
     106                               
     107                                // Don't want to fire focus and click at same time
     108                                toggleFreeze = true;
     109                                setTimeout(function () {
     110                                        toggleFreeze = false;
     111                                }, 400);
    102112                        });
    103113
    104114                        this.setting.bind( update );
    … …  
    210220                        });
    211221
    212222                        // Bind tab switch events
    213                         this.library.children('ul').on( 'click', 'li', function( event ) {
     223                        this.library.children('ul').on( 'click keydown', 'li', function( event ) {
     224                               
     225                                if ( event.type === 'keydown' &&  13 !== event.which )
     226                                        return;
     227                               
    214228                                var id  = $(this).data('customizeTab'),
    215229                                        tab = control.tabs[ id ];
    216230
    … …  
    225239                        });
    226240
    227241                        // Bind events to switch image urls.
    228                         this.library.on( 'click', 'a', function( event ) {
     242                        this.library.on( 'click keydown', 'a', function( event ) {
     243                        if ( event.type === 'keydown' &&  13 !== event.which ) // enter
     244                                        return;
     245
    229246                                var value = $(this).data('customizeImageValue');
    230247
    231248                                if ( value ) {
    … …  
    814831                }());
    815832
    816833                // Temporary accordion code.
    817                 $('.customize-section-title').click( function( event ) {
     834                var accordionFrozen = false;
     835                $('.customize-section-title').bind('click keydown', function( event ) {
     836                       
     837                        if ( event.type === 'keydown' &&  13 !== event.which ) // enter
     838                                        return;
     839                       
    818840                        var clicked = $( this ).parents( '.customize-section' );
    819841
    820                         if ( clicked.hasClass('cannot-expand') )
     842                        if ( clicked.hasClass('cannot-expand') || accordionFrozen )
    821843                                return;
     844                       
     845                        // Don't want to fire focus and click at same time
     846                        accordionFrozen = true;
     847                        setTimeout(function () {
     848                                accordionFrozen = false;
     849                        }, 400);
     850                       
     851                        // Scroll up if on #customize-section-title_tagline
     852                        if ('customize-section-title_tagline' === clicked.attr('id'))
     853                                $('.wp-full-overlay-sidebar-content').scrollTop(0);
    822854
    823855                        $( '.customize-section' ).not( clicked ).removeClass( 'open' );
    824856                        clicked.toggleClass( 'open' );
    … …  
    829861                $('#save').click( function( event ) {
    830862                        previewer.save();
    831863                        event.preventDefault();
     864                }).keydown( function( event ) {
     865                        if ( 9 === event.which ) // tab
     866                                return;
     867                        if ( 13 === event.which ) // enter
     868                                previewer.save();
     869                        event.preventDefault();
    832870                });
     871               
     872                $('.back').keydown( function( event ) {
     873                        if ( 9 === event.which ) // tab
     874                                return;
     875                        var thisHref = $(this).attr('href');
     876                        if ( 13 === event.which ) // enter
     877                                window.location = thisHref;
     878                        event.preventDefault();
     879                });
    833880
    834881                $('.collapse-sidebar').click( function( event ) {
    835882                        overlay.toggleClass( 'collapsed' ).toggleClass( 'expanded' );
    … …  
    948995                });
    949996
    950997                api.trigger( 'ready' );
     998
     999                // Make sure left column gets focus
     1000                var topFocus = $('.back');
     1001                topFocus.focus();
     1002                setTimeout(function () {
     1003                        topFocus.focus();
     1004                }, 200);
     1005
    9511006        });
    9521007
    953 })( wp, jQuery );
    954  No newline at end of file
     1008})( wp, jQuery );

  • wp-includes/class-wp-customize-control.php

     
    521521                        <span class="customize-control-title"><?php echo esc_html( $this->label ); ?></span>
    522522
    523523                        <div class="customize-control-content">
    524                                 <div class="dropdown preview-thumbnail">
     524                                <div class="dropdown preview-thumbnail" tabindex="0">
    525525                                        <div class="dropdown-content">
    526526                                                <?php if ( empty( $src ) ): ?>
    527527                                                        <img style="display:none;" />
    … …  
    537537                        <div class="library">
    538538                                <ul>
    539539                                        <?php foreach ( $this->tabs as $id => $tab ): ?>
    540                                                 <li data-customize-tab='<?php echo esc_attr( $id ); ?>'>
     540                                                <li data-customize-tab='<?php echo esc_attr( $id ); ?>' tabindex='0'>
    541541                                                        <?php echo esc_html( $tab['label'] ); ?>
    542542                                                </li>
    543543                                        <?php endforeach; ?>

  • wp-includes/class-wp-customize-section.php

     
    8080        protected function render() {
    8181                ?>
    8282                <li id="customize-section-<?php echo esc_attr( $this->id ); ?>" class="control-section customize-section">
    83                         <h3 class="customize-section-title" title="<?php echo esc_attr( $this->description ); ?>"><?php echo esc_html( $this->title ); ?></h3>
     83                        <h3 class="customize-section-title" tabindex="0" title="<?php echo esc_attr( $this->description ); ?>"><?php echo esc_html( $this->title ); ?></h3>
    8484                        <ul class="customize-section-content">
    8585                                <?php
    8686                                foreach ( $this->controls as $control )