Skip to content

feat: Make the aria-label attribute for close button configurable. - #624

Merged
nilbuild merged 4 commits into
nilbuild:masterfrom
amcbain-globys:amcbain-globys/popover-close-btn-aria-label
Oct 3, 2026
Merged

nilbuild merged 4 commits into
nilbuild:masterfrom
amcbain-globys:amcbain-globys/popover-close-btn-aria-label

Conversation

@amcbain-globys

@amcbain-globys amcbain-globys commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Addresses issue #623

…rs configurable for full translation customization.
@amcbain-globys
amcbain-globys force-pushed the amcbain-globys/popover-close-btn-aria-label branch from 0859588 to 50d8aff Compare October 2, 2026 22:10
@amcbain-globys amcbain-globys changed the title #623 Make the aria-label attribute for close button in popovers confi… feat: issue #623 Make the aria-label attribute for close button configurable. Oct 2, 2026
@amcbain-globys amcbain-globys changed the title feat: issue #623 Make the aria-label attribute for close button configurable. feat: Make the aria-label attribute for close button configurable. Oct 2, 2026
@nilbuild
nilbuild merged commit 4dff1b9 into nilbuild:master Oct 3, 2026
@nilbuild

nilbuild commented Oct 3, 2026

Copy link
Copy Markdown
Owner

Thanks! I made a few small changes before merging: used setAttribute for the label, removed the hints option since hints don't show a close button, and added docs and tests.

@amcbain-globys
amcbain-globys deleted the amcbain-globys/popover-close-btn-aria-label branch October 5, 2026 04:31
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