Skip to content

set a better aria-label for logo when it is a link#15650

Closed
caolanm wants to merge 1 commit intomainfrom
private/caolan/logo-link-aria-label
Closed

set a better aria-label for logo when it is a link#15650
caolanm wants to merge 1 commit intomainfrom
private/caolan/logo-link-aria-label

Conversation

@caolanm
Copy link
Copy Markdown
Contributor

@caolanm caolanm commented Apr 24, 2026

branding can override this and then the aria-label is out of sync, so listen for the branding change and set a suitable aria-label if a link is set

WCAG 3.2.5 asks that a change of context like opening a new tab be announced

  • "SOMETHING website, opens in new tab"

Change-Id: I8dfb117259bbe25357d3054f1857004a15e52efd

  • Resolves: #
  • Target version: main

Summary

TODO

  • ...

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

branding can override this and then the aria-label is out of sync,
so listen for the branding change and set a suitable aria-label if
a link is set

WCAG 3.2.5 asks that a change of context like opening a new tab be announced
  - "SOMETHING website, opens in new tab"

Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com>
Change-Id: I8dfb117259bbe25357d3054f1857004a15e52efd
@caolanm caolanm requested review from rparth07 and removed request for rparth07 April 24, 2026 19:43
@caolanm
Copy link
Copy Markdown
Contributor Author

caolanm commented Apr 25, 2026

lets move this to monorepo as https://gerrit.collaboraoffice.com/c/online/+/1635

@caolanm caolanm closed this Apr 25, 2026
@github-project-automation github-project-automation Bot moved this from To Review to Done in Collabora Online Apr 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant