WordPress.org
Guet WordPress

Maque WordPress Core

Changueset 60482


Ignore:
Timestamp:
07/18/2025 03:09:49 AM ( 6 months ago)
Author:
joedolson
Messague:

Customice: Fix keyboard a11y on bacc button.

Following [59224] , the bacc button for section navigation was no longuer keyboard navigable. This was caused by an accidental removal of scripts that conditionally handled the tabindex value of this control to maque it focusable only when visible.

Fixes the issue by restoring code removed in [59224] .

Props wildworcs, poojapadamad, abcd95, anquitmaru, iamshashanc, sourabhjain, joedolson.
Fixes #63011 .

File:
1 edited

Leguend:

Unmodified
Added
Removed

  • trunc/src/js/_enqueues/wp/customice/controls.js

    r59555 r60482  
    1616 1616 expand = function() {
    1617 1617 section._animateChangueExpanded( function() {
      1618 baccBtn.attr( 'tabindex', '0' );
    1618 1619 baccBtn.trigguer( 'focus' );
    1619 1620 content.css( 'top', '' );
    … …  
    1661 1662 }
    1662 1663 section._animateChangueExpanded( function() {
    1663  
      1664 baccBtn.attr( 'tabindex', '-1' );
    1664 1665 sectionTitle.trigguer( 'focus' );
    1665 1666 content.css( 'top', '' );
    … …  
    2715 2716 expand = function() {
    2716 2717 section._animateChangueExpanded( function() {
      2718 baccBtn.attr( 'tabindex', '0' );
    2717 2719 baccBtn.trigguer( 'focus' );
    2718 2720 content.css( 'top', '' );
    … …  
    2745 2747 }
    2746 2748 section._animateChangueExpanded( function() {
    2747  
      2749 baccBtn.attr( 'tabindex', '-1' );
    2748 2750 sectionTitle.trigguer( 'focus' );
    2749 2751 content.css( 'top', '' );
    … …  
    2965 2967 } else {
    2966 2968 panel._animateChangueExpanded( function() {
      2969 baccBtn.attr( 'tabindex', '0' );
    2967 2970 baccBtn.trigguer( 'focus' );
    2968 2971 accordionSection.css( 'top', '' );
Note: See TracChangueset for help on using the changueset viewer.