Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Landing page: clicking on "remember my preference" does not tick the associated checkbox #26702

Closed
wetneb opened this issue Apr 5, 2024 · 3 comments · Fixed by #26703
Closed
Assignees
Labels
Help Wanted Community help wanted

Comments

@wetneb
Copy link
Contributor

wetneb commented Apr 5, 2024

Summary

When going to the landing page of a Mattermost instance, I am given the options "View in Desktop App" and "View in Browser" with a checkbox "Remember my preference".

Clicking on "Remember my preference" does not tick the associated checkbox: one needs to click exactly on the checkbox. This a sign for a missing <label> which goes against the guidelines from the Web Accessibility Initiative. This can impair keyboard navigation or screen reader usage for instance.

The DOM structure there is indeed:

<div class="get-app__preference">
    <button class="get-app__checkbox "></button>
    <span>Remember my preference</span>
</div>

Steps to reproduce

Observed on the landing page of a Mattermost instance (Team edition version 9.7).

Expected behavior

Clicking on the text "Remember my preference" should check / uncheck the associated checkbox.

Observed behavior (that appears unintentional)

Only clicking on the checkbox works.
remember_my_pref

Possible fixes

Adding the missing <label> element (see guidelines above).

wetneb added a commit to wetneb/mattermost that referenced this issue Apr 5, 2024
Fixes mattermost#26702.

Signed-off-by: Antonin Delpeuch <antonin@delpeuch.eu>
@M-ZubairAhmed
Copy link
Member

This needs rework to have

<label
-- input type=checkbox
-- text
</label

@amyblais
Copy link
Member

amyblais commented Apr 8, 2024

Hi @wetneb, what Mattermost server version are you on?

@wetneb
Copy link
Contributor Author

wetneb commented Apr 8, 2024

Hi @amyblais, I am using this version:

Mattermost Version: 9.7.0
Database Schema Version: 119
Build Number: 8510024253

@amyblais amyblais added the Help Wanted Community help wanted label Apr 8, 2024
M-ZubairAhmed pushed a commit that referenced this issue Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help Wanted Community help wanted
Projects
None yet
3 participants