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

Enhance admin note visibility and improve accessibility for quick actions #5968

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Abishekcs
Copy link
Contributor

@Abishekcs Abishekcs commented Sep 26, 2024

closes #5962

What this PR does

  • Added a visual indicator for admin notes to display when a note is present, without requiring the modal to be opened.
  • Standardized the size of all three admin quick action buttons for consistency.
  • Implemented screen reader messages to improve accessibility for the newly added features.

Screenshots ( visual indicator )

Before:

Screenshot from 2024-10-10 12-51-17

Before.1.mp4

After:

Screenshot from 2024-09-26 12-43-18

After.1.1.mp4

Screenshots ( quick action buttons)

Before:

Before

After:

After

Screenshots (For Screen Reader)

1 - When No Notes are available

1.-.For.Screen.Reader.When.No.Notes.are.available.mp4

2 - When there are no newly created Notes

2.-.For.Screen.Reader.When.there.are.no.newly.created.Notes.mp4

3 - When newly created notes are available

3-.Screen.Reader.When.newly.created.notes.are.available.mp4

Open questions and concerns

I noticed there was an empty status_panel.jsx file in the admin_notes folder. Since it was empty and there was no indication it was needed for future use, I deleted it. If it's required later, I can add it back.

…ader.

- Added a visual indicator for admin notes to display when a note is present, without requiring the modal to be opened.

- Standardized the size of all three admin quick action buttons for consistency.

- Implemented screen reader messages to improve accessibility for the newly added features.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Admin Notes should indicate when a note is present (without opening the modal first)
1 participant