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

Instructor's Student Records Page: Accessibility issue for comment buttons #12664

Open
weiquu opened this issue Dec 16, 2023 · 13 comments · May be fixed by #13099
Open

Instructor's Student Records Page: Accessibility issue for comment buttons #12664

weiquu opened this issue Dec 16, 2023 · 13 comments · May be fixed by #13099
Labels
a-UIX User Interface, User eXperience, responsiveness good first issue Easy; restricted for first-time contributors

Comments

@weiquu
Copy link
Contributor

weiquu commented Dec 16, 2023

When using a screen reader, the comment buttons are simply read as "button", with no other information on what the button does. Here are the 2 comment buttons (edit and delete):
comment buttons

Let's add a description that the screen reader can pick up.

In addition, the buttons when adding/editing a comment are picked up in the wrong order by the screen reader. Here is the buttons (discard and save) for editing a comment; the same issue exists for adding a comment (cancel and save):
discard and save buttons

The screen reader first goes to Save and then Discard. Let's change the ordering so that the screen reader goes to Discard and then Save, to follow the visual order on the page

@weiquu weiquu added a-UIX User Interface, User eXperience, responsiveness good first issue Easy; restricted for first-time contributors labels Dec 16, 2023
Copy link

Good First Issue - Notes for Contributors
This issue is for first-time contributors only. If you are new to TEAMMATES, feel free to submit a PR for this issue.

Please note that we allow only one good first issue per contributor. If you have already made a prior contribution to TEAMMATES, you may wish to take a look at issues with the help wanted tag instead.

We do not assign issues to contributors. If you would like to pick up this issue, do post a comment below to express your interest and check if there is anyone else who is already working on the issue. We will do our best to reply and give you the go-ahead, but if we don't, feel free to submit a PR as long as there is no one else working on it.

To get started, do read through our contributing guidelines carefully, and set up a development environment on your local machine before making a PR.

If you need any clarifications on our developer guide, or are facing issues that are not found in our troubleshooting guide, please post a message in our discussion forum.

@rahulpro2001
Copy link

rahulpro2001 commented Dec 25, 2023

hii @weiquu I would like to work on this one and want to ask when we go to the Comment section the button for comment also has no description can I work on this one
ff

@weiquu
Copy link
Contributor Author

weiquu commented Dec 25, 2023

Hi @rahulpro2001, if I'm not wrong, the comment button is being picked up by the screen reader correctly. You can enable your own screen reader to check. I'm on mac, so here are the commands I use - if you're using a different OS, I'm sure you can find the instructions of how to enable their screen reader online.

Regardless, do ensure that all buttons are being correctly read by the screen reader and in the right order when you eventually submit a PR (i.e. please resolve this issue fully).

@rahulpro2001
Copy link

okk @weiquu

@rahulpro2001
Copy link

@weiquu can we change the order of buttons or not?

@weiquu
Copy link
Contributor Author

weiquu commented Dec 26, 2023

@weiquu can we change the order of buttons or not?

Hi @rahulpro2001, please change the order the screen reader picks up the buttons, but do not change the visual order of the buttons.

@rahulpro2001
Copy link

okk will update you shortly

@willzoo
Copy link

willzoo commented Jan 29, 2024

@weiquu Hi! I'm a first timer and would like to contribute to this issue if possible.

@weiquu
Copy link
Contributor Author

weiquu commented Jan 30, 2024

Hi @willzoo, sure please feel free to submit a PR for this issue

willzoo added a commit to willzoo/teammates-fork that referenced this issue Feb 11, 2024
willzoo added a commit to willzoo/teammates-fork that referenced this issue Feb 11, 2024
willzoo added a commit to willzoo/teammates-fork that referenced this issue Feb 11, 2024
@Manon-ds
Copy link

Manon-ds commented Apr 8, 2024

Hi, can I work on this issue?

@MinalDubey24
Copy link

Hi, is the issue still open? Can I work on it?

@weiquu
Copy link
Contributor Author

weiquu commented Apr 21, 2024

Sure, please feel free to submit a PR

@WesleyJammer
Copy link

Hey, i am a student and i am curious if this issue is still open? i like to work on this if it is possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-UIX User Interface, User eXperience, responsiveness good first issue Easy; restricted for first-time contributors
Projects
None yet
6 participants