Skip to content

Commit ad1d839

Browse files
authored
Merge pull request #490 from TechnologyEnhancedLearning/Develop/Features/TD-4175-Replace-notification-anchor-with-button
Replace Notification page Delete anchor with button
2 parents 5f7259f + 87b2111 commit ad1d839

File tree

2 files changed

+13
-2
lines changed

2 files changed

+13
-2
lines changed

LearningHub.Nhs.WebUI/Scripts/vuesrc/notification/notification.vue

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,7 @@
4545
</td>
4646
<td class="px-sm-3 py-sm-3">
4747
<div class="d-flex justify-content-between" v-if="notification.userDismissable">
48-
<a href="#deleteModal" data-toggle="modal" @click="selectNotification(notification)">Delete</a>
49-
&nbsp;<i class="fa-solid fa-trash-can-alt pt-1"></i>
48+
<button data-target="#deleteModal" data-toggle="modal" @click="selectNotification(notification)">Delete <i class="fa-solid fa-trash-can-alt pt-1"></i></button>
5049
</div>
5150
</td>
5251
</tr>

LearningHub.Nhs.WebUI/Styles/nhsuk/layout.scss

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -183,6 +183,18 @@
183183
}
184184
}
185185

186+
button[data-toggle="modal"] {
187+
color: #005eb8;
188+
padding: 0;
189+
border: none;
190+
background: none;
191+
192+
&:focus {
193+
background: yellow;
194+
border-bottom: 4px solid;
195+
}
196+
}
197+
186198
/* large desktop */
187199
@media (min-width: px2rem(990)) {
188200

0 commit comments

Comments
 (0)