Skip to content

Commit 9470af0

Browse files
committed
Accordion: Avoid resetting outline on headers. Fixes #9352: Accordion - Active header has no focus indicator
1 parent bc85742 commit 9470af0

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

themes/base/jquery.ui.accordion.css

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,10 @@
1212
display: block;
1313
cursor: pointer;
1414
position: relative;
15-
margin-top: 2px;
15+
margin: 2px 0 0 0;
1616
padding: .5em .5em .5em .7em;
1717
min-height: 0; /* support: IE7 */
18+
font-size: 100%;
1819
}
1920
.ui-accordion .ui-accordion-icons {
2021
padding-left: 2.2em;

ui/jquery.ui.accordion.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ $.widget( "ui.accordion", {
9999

100100
// clean up headers
101101
this.headers
102-
.removeClass( "ui-accordion-header ui-accordion-header-active ui-helper-reset ui-state-default ui-corner-all ui-state-active ui-state-disabled ui-corner-top" )
102+
.removeClass( "ui-accordion-header ui-accordion-header-active ui-state-default ui-corner-all ui-state-active ui-state-disabled ui-corner-top" )
103103
.removeAttr( "role" )
104104
.removeAttr( "aria-selected" )
105105
.removeAttr( "aria-controls" )
@@ -247,7 +247,7 @@ $.widget( "ui.accordion", {
247247

248248
_processPanels: function() {
249249
this.headers = this.element.find( this.options.header )
250-
.addClass( "ui-accordion-header ui-helper-reset ui-state-default ui-corner-all" );
250+
.addClass( "ui-accordion-header ui-state-default ui-corner-all" );
251251

252252
this.headers.next()
253253
.addClass( "ui-accordion-content ui-helper-reset ui-widget-content ui-corner-bottom" )

0 commit comments

Comments
 (0)