Skip to content

Refactor reaction handler Typescript#6629

Draft
BurntimeX wants to merge 14 commits into6.3from
63-refactor-reaction-ts
Draft

Refactor reaction handler Typescript#6629
BurntimeX wants to merge 14 commits into6.3from
63-refactor-reaction-ts

Conversation

@BurntimeX
Copy link
Member

@BurntimeX BurntimeX commented Mar 13, 2026

Remaining tasks:

  • .reactionButton actually needs an aria-owns attribute
  • According to the ARIA Guide, buttons in the popover should have tabindex=-1 (but then focus-trap stops working — is it even necessary anymore? Is this configurable?)
  • Hide on window resize (is there another way to handle this?)
  • Support for scrollable popovers (too many reaction types) is currently missing (maybe there’s another way to solve this?)

Base automatically changed from 63-refactor-reaction-handler to 6.3 March 14, 2026 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants