Skip to content

Commit 6c9f86b

Browse files
committed
bug symfony#14916 [WebProfilerBundle] Added tabindex="-1" to not interfer with normal UX (drAlberT)
This PR was submitted for the 2.7 branch but it was merged into the 2.3 branch instead (closes symfony#14916). Discussion ---------- [WebProfilerBundle] Added tabindex="-1" to not interfer with normal UX | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Commits ------- 53cdb1a [WebProfilerBundle] Added tabindex="-1" to not interfer with normal UX
2 parents 1c5a213 + 53cdb1a commit 6c9f86b

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/Symfony/Bundle/WebProfilerBundle/Resources/views/Profiler/toolbar.html.twig

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<!-- START of Symfony Web Debug Toolbar -->
22
{% if 'normal' != position %}
33
<div id="sfMiniToolbar-{{ token }}" class="sf-minitoolbar" data-no-turbolink>
4-
<a href="javascript:void(0);" title="Show Symfony toolbar" onclick="
4+
<a href="javascript:void(0);" title="Show Symfony toolbar" tabindex="-1" accesskey="D" onclick="
55
var elem = this.parentNode;
66
if (elem.style.display == 'none') {
77
document.getElementById('sfToolbarMainContent-{{ token }}').style.display = 'none';
@@ -36,7 +36,7 @@
3636
{% endfor %}
3737

3838
{% if 'normal' != position %}
39-
<a class="hide-button" title="Close Toolbar" onclick="
39+
<a class="hide-button" title="Close Toolbar" tabindex="-1" accesskey="D" onclick="
4040
var p = this.parentNode;
4141
p.style.display = 'none';
4242
(p.previousElementSibling || p.previousSibling).style.display = 'none';

0 commit comments

Comments
 (0)