IntenseWebs/discourse
3
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-20 11:48:26 -06:00
Code Issues Packages Projects Releases Wiki Activity

A11Y: remove title from notification menu icons (#23735)

Browse Source
This commit is contained in:
Kris 2023-10-02 13:54:59 -04:00 committed by GitHub
parent f5d4ddbff5
commit 24aea41d89
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/assets/javascripts/select-kit/addon/components/dropdown-select-box/dropdown-select-box-row.hbs
View File

@ -2,7 +2,7 @@
<div class="icons">
<span class="selection-indicator"></span>
{{#each this.icons as |icon|}}
{{d-icon icon translatedTitle=(dasherize this.title)}}
{{d-icon icon}}
{{/each}}
</div>
{{/if}}

4
app/assets/javascripts/select-kit/addon/components/notifications-button/notifications-button-row.js
View File

@ -13,9 +13,7 @@ export default DropdownSelectBoxRowComponent.extend({
return escapeExpression(I18n.t(`${this._start}.title`));
}),
title: readOnly("label"),
icons: computed("title", "item.icon", function () {
icons: computed("item.icon", function () {
return [escapeExpression(this.item.icon)];
}),