Skip to content

Conversation

@zubairshakoorarbisoft
Copy link
Contributor

@zubairshakoorarbisoft zubairshakoorarbisoft commented May 9, 2025

Description

Related to overhangio/tutor-indigo#146
On the Teams tab, press left arrow key to bring the focus on “My team” notice that the bottom indicator spreads through all the tabs, and doesn't stay specific to My team's tab, also “Teams” and “Instructor Dashboard” heading(First heading of page) should be H1

image

Useful information to include:

  • All user roles "Learner", "Course Author", "Developer", and "Operator".
  • When the focus at the tabs, focus should only move using left right error key, and on enter that tab should become active.
    image

After the fix, the focus should move with arrow key without spaning the bottom line on all the tab's bottom, and on pressing Enter key, it should be active.
Screenshot 2025-04-17 at 3 47 30 AM

Supporting information

You can have a look into the following issues to have more context on the issues(Access Required)

Testing instructions

One can simply pull the changes on local and build image, the changes should be reflected.

Deadline

N/A

Other information

Include anything else that will help reviewers and consumers understand the change.

  • Does this change depend on other changes elsewhere? No
  • Any special concerns or limitations? For example: deprecations, migrations, security, or accessibility. No Limitations
  • No Database Migrations

@zubairshakoorarbisoft zubairshakoorarbisoft changed the title fix: TIA-49 and TIA-60 fixes pushed in single comment Resolve accessibility issues for better WCAG compliance May 9, 2025
Copy link

@wittjeff wittjeff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, good fix, except can we omit the !important in each case here?

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.

2 participants