Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(checkbox): ensure proper visual selection when navigating via VoiceOver in Safari #30300

Merged
merged 18 commits into from
Mar 28, 2025

Conversation

ShaneK
Copy link
Member

@ShaneK ShaneK commented Mar 24, 2025

Issue number: resolves internal


What is the current behavior?

Currently, MacOS voice over on Safari does not recognize ion-checkbox correctly and fails to highlight the element properly

What is the new behavior?

By adding the role property to the host element, we're correctly identifying ion-checkbox as a checkbox so Safari knows how to handle it.

Does this introduce a breaking change?

  • Yes
  • No

Other information

@ShaneK ShaneK requested a review from a team as a code owner March 24, 2025 21:19
@ShaneK ShaneK requested a review from brandyscarney March 24, 2025 21:19
Copy link

vercel bot commented Mar 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ionic-framework ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 28, 2025 1:12pm

@github-actions github-actions bot added the package: core @ionic/core package label Mar 24, 2025
Copy link
Member

@brandyscarney brandyscarney left a comment

Choose a reason for hiding this comment

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

A few nit comments but other than that it looks good.

ShaneK and others added 3 commits March 26, 2025 12:26
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
Co-authored-by: Brandy Smith <brandyscarney@users.noreply.github.com>
@brandyscarney brandyscarney requested a review from thetaPC March 26, 2025 19:31
@brandyscarney brandyscarney changed the title fix(checkbox): fix MacOS voice over with Safari not recognizing ion-checkbox as a checkbox fix(checkbox): ensure proper visual selection when navigating via VoiceOver in Safari Mar 26, 2025
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

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

Tested on Windows (Firefox, Chrome, Edge) and MacOS (Firefox, Chrome, Safari), sounds as expected.

Just some code requests.

ShaneK and others added 4 commits March 27, 2025 15:07
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Co-authored-by: Maria Hutt <thetaPC@users.noreply.github.com>
Copy link
Contributor

@thetaPC thetaPC left a comment

Choose a reason for hiding this comment

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

LGTM

@ShaneK ShaneK added this pull request to the merge queue Mar 28, 2025
Merged via the queue into main with commit bb40a1e Mar 28, 2025
50 checks passed
@ShaneK ShaneK deleted the FW-6256 branch March 28, 2025 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: core @ionic/core package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants