Skip to content

Focusable img alt attribute in Widget DrawingArea#14310

Open
rparth07 wants to merge 3 commits intomainfrom
private/rparth07/focusable-img
Open

Focusable img alt attribute in Widget DrawingArea#14310
rparth07 wants to merge 3 commits intomainfrom
private/rparth07/focusable-img

Conversation

@rparth07
Copy link
Contributor

@rparth07 rparth07 commented Feb 3, 2026

From recent changes, we have aria-labelledby and alt attribute both in Widget DrawingArea which violates single accessible name rule. This changes will fix this issue. Here is how it works.
We have two types of images.

  1. focusable image: accessible name is set via alt attribute on image since it is focusable.
  2. decorative image: there is no need to provide accessible name as it will never get focus.

Checklist

  • I have run make prettier-write and formatted the code.
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

@github-project-automation github-project-automation bot moved this from To Review to To Test in Collabora Online Feb 3, 2026
@rparth07 rparth07 force-pushed the private/rparth07/focusable-img branch from 9bb21e3 to d59e066 Compare February 3, 2026 08:45
@caolanm caolanm force-pushed the private/rparth07/focusable-img branch from d59e066 to 5e946fb Compare February 3, 2026 15:37
…wing area

- if img is focusable then it should have appropriate alt attribute on it instead of parent container
- for decorative images, no need to provide accessible name via img alt attribute or via aria-labelledby or aria-label in parent container

Change-Id: I2bd3104621800587f167dd6750638a84d1ad10dd
Signed-off-by: Parth Raiyani <parth.raiyani@collabora.com>
Signed-off-by: Parth Raiyani <parth.raiyani@collabora.com>
Change-Id: I62b3545055ccd5d2a3b5916cc9c2b4e850659116
Signed-off-by: Parth Raiyani <parth.raiyani@collabora.com>
Change-Id: Ib735dbdc038baa9999826084dae5092a6291f78d
@rparth07 rparth07 force-pushed the private/rparth07/focusable-img branch from 5e946fb to 5f495f0 Compare February 6, 2026 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To Test

Development

Successfully merging this pull request may close these issues.

3 participants