Skip to content

Commit e88ebe8

Browse files
Merge pull request #1025 from TechnologyEnhancedLearning/Develop/Fixes/TD-5179-Focus-not-rendered-correctly-on-links-used-on-My-bookmarks-screen
TD-5179: Fixed focus issue on My bookmarks screen
2 parents 61464c8 + 6a857ba commit e88ebe8

File tree

2 files changed

+22
-5
lines changed

2 files changed

+22
-5
lines changed

LearningHub.Nhs.WebUI/Styles/nhsuk/pages/bookmark.scss

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,23 @@ td.col-type {
112112
tr.bookmark-item:hover td {
113113
background-color: $nhsuk-white;
114114
}
115+
.my-bookmark-btn:focus {
116+
outline: none;
117+
text-decoration: none !important;
118+
color: $nhsuk-black !important;
119+
background-color: $nhsuk-yellow !important;
120+
box-shadow: 0 -2px $govuk-focus-highlight-yellow, 0 4px $nhsuk-black;
121+
}
122+
123+
.my-bookmark-btn:hover {
124+
outline: none;
125+
text-decoration: none !important;
126+
}
127+
128+
.my-bookmark-btn {
129+
transition: none !important;
130+
border-radius: 0 !important;
131+
}
115132

116133
@media (min-width: 768px) {
117134

LearningHub.Nhs.WebUI/Views/Bookmark/_BookmarkAction.cshtml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
<span class="action-link-chevron @(Model.IsFirst? "empty" : "")">
77
@if (Model.IsFirst == false)
88
{
9-
<button name="action" value="moveup" class="btn btn-link action-link" title="Move up" aria-label="Move up">
9+
<button name="action" value="moveup" class="btn btn-link action-link my-bookmark-btn" title="Move up" aria-label="Move up">
1010
<i class="fa-solid fa-chevron-up fa-sm"></i>
1111
</button>
1212
}
@@ -15,25 +15,25 @@
1515
<span class="action-link-chevron @(Model.IsLast? "empty" : "")">
1616
@if (Model.IsLast == false)
1717
{
18-
<button name="action" value="movedown" class="btn btn-link action-link" title="Move down" aria-label="Move down">
18+
<button name="action" value="movedown" class="btn btn-link action-link my-bookmark-btn" title="Move down" aria-label="Move down">
1919
<i class="fa-solid fa-chevron-down fa-sm"></i>
2020
</button>
2121
}
2222
</span>
2323

2424
<span class="action-link-rename">
25-
<button name="action" value="rename" class="btn btn-link action-link" title="Rename">Rename</button>
25+
<button name="action" value="rename" class="btn btn-link action-link my-bookmark-btn" title="Rename">Rename</button>
2626
</span>
2727

2828
<span class="action-link-move">
2929
@if (Model.IsFolder == false)
3030
{
31-
<button name="action" value="move" class="btn btn-link action-link" title="Move">Move</button>
31+
<button name="action" value="move" class="btn btn-link action-link my-bookmark-btn" title="Move">Move</button>
3232
}
3333
</span>
3434

3535
<span class="action-link-delete">
36-
<button name="action" value="delete" class="btn btn-link action-link" title="Delete">Delete</button>
36+
<button name="action" value="delete" class="btn btn-link action-link my-bookmark-btn" title="Delete">Delete</button>
3737
</span>
3838
</form>
3939
</div>

0 commit comments

Comments
 (0)