Skip to content

Commit

Permalink
A11y: use <a> element for modx-button to be focusable in resource tre…
Browse files Browse the repository at this point in the history
…e for create/refresh modxcms#16612
  • Loading branch information
jenswittmann committed Sep 13, 2024
1 parent ec4e1d9 commit f903a73
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions manager/assets/modext/widgets/core/modx.button.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@ MODx.Button = function(config) {
config.iconCls = ''
}
Ext.applyIf(config,{
template: new Ext.XTemplate('<span id="{4}" class="x-btn icon {1} {3}" unselectable="on">'+
template: new Ext.XTemplate('<a href="javascrpt:;" id="{4}" class="x-btn icon {1} {3}">'+
' <i class="{2}">'+
' </i>'+
'</span>').compile()
'</a>').compile()
});

MODx.Button.superclass.constructor.call(this,config);
Expand Down

0 comments on commit f903a73

Please sign in to comment.