Skip to content

Commit 8056eae

Browse files
Maffoochclaude
andcommitted
Use <button> for banner toggle and clean focus styling
Anchor-based toggle picked up Bootstrap alert link styles and a lingering focus outline after click, which showed as a stray glyph next to the caret. A plain <button type="button"> avoids link decoration entirely; focus outline and transition are also dropped so the caret flips instantly. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent c6e23e3 commit 8056eae

2 files changed

Lines changed: 15 additions & 8 deletions

File tree

dojo/static/dojo/css/dojo.css

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1127,14 +1127,19 @@ div.custom-search-form {
11271127
}
11281128

11291129
.os-banner-toggle {
1130+
background: transparent;
1131+
border: 0;
1132+
padding: 0;
11301133
margin-left: 6px;
1131-
cursor: pointer;
11321134
color: inherit;
1133-
text-decoration: none;
1135+
cursor: pointer;
1136+
line-height: 1;
11341137
}
11351138

1136-
.os-banner-toggle .fa-caret-down {
1137-
transition: transform 0.15s ease-in-out;
1139+
.os-banner-toggle:focus,
1140+
.os-banner-toggle:active {
1141+
outline: none;
1142+
box-shadow: none;
11381143
}
11391144

11401145
.os-banner-toggle:not(.collapsed) .fa-caret-down {

dojo/templates/base.html

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -674,11 +674,13 @@
674674
<div role="alert" class="announcement-banner alert alert-{{ banner.style }} show">
675675
{{ banner.message|safe }}{% if banner.url %} <a href="{{ banner.url }}">{{ banner.link_text }}</a>{% endif %}
676676
{% if banner.expanded_html %}
677-
<a role="button" data-toggle="collapse" href="#os-banner-expanded-{{ forloop.counter }}"
678-
aria-expanded="false" aria-controls="os-banner-expanded-{{ forloop.counter }}"
679-
class="os-banner-toggle collapsed">
677+
<button type="button" class="os-banner-toggle collapsed"
678+
data-toggle="collapse"
679+
data-target="#os-banner-expanded-{{ forloop.counter }}"
680+
aria-expanded="false"
681+
aria-controls="os-banner-expanded-{{ forloop.counter }}">
680682
<i class="fa-solid fa-caret-down"></i>
681-
</a>
683+
</button>
682684
<div id="os-banner-expanded-{{ forloop.counter }}" class="collapse os-banner-expanded">
683685
{{ banner.expanded_html|safe }}
684686
</div>

0 commit comments

Comments
 (0)