Skip to content

Conversation

@Ginger-Leo
Copy link
Contributor

🎩 What? Why?

We were having an issue where when trying to share a something (a meeting for example), if using the keyboard only, user was unable to access the modal.

📌 Related Issues

Link your PR to an issue

Testing

Go to a meeting, click the 'share' button, navigate the modal using only a keyboard.

♥️ Thank you!

@github-actions github-actions bot added module: core module: comments type: fix PRs that implement a fix for a bug labels Dec 16, 2025
github-actions[bot]
github-actions bot previously approved these changes Dec 16, 2025
github-actions[bot]
github-actions bot previously approved these changes Dec 16, 2025
@Ginger-Leo Ginger-Leo marked this pull request as ready for review December 16, 2025 12:16
@alecslupu alecslupu self-assigned this Dec 16, 2025
Copy link
Member

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

@Ginger-Leo can you merge with the latest decidim develop branch, I think you have additional work from #15774 that should not be present in this PR.

@andreslucena andreslucena changed the title Fixing WCAG issue, focus does not move to sharing modal. Fix focus trap in modal dialog for sharing Dec 17, 2025
@alecslupu alecslupu added release: v0.30 Issues or PRs that need to be tackled for v0.30 release: v0.31 Issues or PRs that need to be tackled for v0.31 labels Dec 17, 2025
Copy link
Member

@alecslupu alecslupu left a comment

Choose a reason for hiding this comment

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

👍

@alecslupu alecslupu merged commit 243d573 into decidim:develop Dec 17, 2025
82 of 84 checks passed
@Ginger-Leo Ginger-Leo deleted the fix/focus-does-not-move-to-sharing-modal branch December 18, 2025 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: comments module: core release: v0.30 Issues or PRs that need to be tackled for v0.30 release: v0.31 Issues or PRs that need to be tackled for v0.31 type: fix PRs that implement a fix for a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants