Skip to content

Fix: Allow "Show alt text" to work in Notify and Drawer (fix #119)#122

Open
swashbuck wants to merge 5 commits intomasterfrom
issue/119
Open

Fix: Allow "Show alt text" to work in Notify and Drawer (fix #119)#122
swashbuck wants to merge 5 commits intomasterfrom
issue/119

Conversation

@swashbuck
Copy link
Copy Markdown
Contributor

@swashbuck swashbuck commented Feb 16, 2026

Fix #119

Fix

  • Fix bug when using "Show alt text" where annotation icons were not shown in Notify popups or Drawer. This changes where in the DOM the annotations appear. Previously, they were appearing underneath Notify / Drawer. I could not find a simple Less-based fix for this.
  • Fix incorrect class name syntax and remove comma: 'is-right, is-bottom'
  • Removed reference to invalid h7HTML tag. Created HEADING_SELECTOR constant to avoid repeating values.
  • Fix misspelling of computeAccessibleName().
Screenshot 2026-02-16 at 2 38 57 PM

Update

  • Improve annotation display. Icon is now a square instead of a strange, oblong shape. Added padding to the tooltip text part to increase readability.

Previous
current

Improved
Screenshot 2026-02-16 at 2 17 11 PM

@swashbuck swashbuck self-assigned this Feb 16, 2026
@swashbuck swashbuck added the bug label Feb 16, 2026
@swashbuck swashbuck marked this pull request as ready for review February 16, 2026 21:37
@swashbuck swashbuck requested a review from kirsty-hames April 13, 2026 22:37
@swashbuck
Copy link
Copy Markdown
Contributor Author

This is potentially an over-engineered solution, so there may be a better way to address it.

Copy link
Copy Markdown
Contributor

@kirsty-hames kirsty-hames left a comment

Choose a reason for hiding this comment

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

Works as expected thanks @swashbuck and the display is much better 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Show alt text" not working in Notify and Drawer

3 participants