mirror/php-flarum
1
0
Fork 0
mirror of https://github.com/flarum/core.git synced 2025-07-18 15:21:16 +02:00
Code Issues Releases Wiki Activity

fix(a11y): add accessible labels to notification grid options (#3520)

Browse Source
This commit is contained in:
David Wheatley
2022-07-14 12:02:00 +01:00
committed by GitHub
parent 0daa24cf4b
commit 952a5891bb
2 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
framework/core/js/src/forum/components/NotificationGrid.js
View File

@@ -70,7 +70,14 @@ export default class NotificationGrid extends Component {
loading={this.loading[key]} loading={this.loading[key]}
disabled={!(key in preferences)} disabled={!(key in preferences)}
onchange={this.toggle.bind(this, [key])} onchange={this.toggle.bind(this, [key])}
/> >
<span class="sr-only">
{app.translator.trans('core.forum.settings.notification_checkbox_a11y_label_template', {
description: type.label,
method: method.label,
})}
</span>
</Checkbox>
</td> </td>
); );
})} })}

1
framework/core/locale/core.yml
View File

@@ -448,6 +448,7 @@ core:
account_heading: Account account_heading: Account
change_email_button: => core.ref.change_email change_email_button: => core.ref.change_email
change_password_button: => core.ref.change_password change_password_button: => core.ref.change_password
notification_checkbox_a11y_label_template: 'Receive "{description}" notifications via {method}'
notifications_heading: => core.ref.notifications notifications_heading: => core.ref.notifications
notify_by_email_heading: => core.ref.email notify_by_email_heading: => core.ref.email
notify_by_web_heading: Web notify_by_web_heading: Web