Skip to content

fix(material/tooltip): add opt-in for better touch device detection#32597

Merged
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:32503/tooltip-touch-devices
Jan 2, 2026
Merged

fix(material/tooltip): add opt-in for better touch device detection#32597
crisbeto merged 1 commit intoangular:mainfrom
crisbeto:32503/tooltip-touch-devices

Conversation

@crisbeto
Copy link
Copy Markdown
Member

@crisbeto crisbeto commented Dec 25, 2025

Currently the touch device detection in the tooltip is based purely on the Platform provider which isn't able to detect some newer iOS devices properly. These changes add an additional check through a media query.

The check is currently opt-in, because it can cause false positives in tests that run in headless browsers.

Fixes #32503.
Fixes #25287.

@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label Dec 25, 2025
@pullapprove pullapprove Bot requested review from adolgachev and ok7sai December 25, 2025 09:59
@crisbeto crisbeto force-pushed the 32503/tooltip-touch-devices branch 4 times, most recently from 368bde1 to 07507d0 Compare December 25, 2025 20:39
@crisbeto crisbeto changed the title fix(material/tooltip): better touch device detection fix(material/tooltip): add opt-in for better touch device detection Dec 25, 2025
@crisbeto crisbeto force-pushed the 32503/tooltip-touch-devices branch from 07507d0 to e0c91a4 Compare December 25, 2025 20:43
Currently the touch device detection in the tooltip is based purely on the `Platform` provider which isn't able to detect some newer iOS devices properly. These changes add an additional check through a media query.

The check is currently opt-in, because it can cause false positives in tests that run in headless browsers.

Fixes angular#32503.
Fixes angular#25287.
@crisbeto crisbeto force-pushed the 32503/tooltip-touch-devices branch from e0c91a4 to d030ee4 Compare December 25, 2025 20:45
@crisbeto crisbeto removed the request for review from adolgachev December 26, 2025 08:10
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Dec 26, 2025
@crisbeto crisbeto merged commit c8bd62b into angular:main Jan 2, 2026
26 checks passed
@crisbeto
Copy link
Copy Markdown
Member Author

crisbeto commented Jan 2, 2026

This PR was merged into the repository. The changes were merged into the following branches:

crisbeto added a commit that referenced this pull request Jan 2, 2026
…32597)

Currently the touch device detection in the tooltip is based purely on the `Platform` provider which isn't able to detect some newer iOS devices properly. These changes add an additional check through a media query.

The check is currently opt-in, because it can cause false positives in tests that run in headless browsers.

Fixes #32503.
Fixes #25287.

(cherry picked from commit c8bd62b)
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Feb 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: material/tooltip target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(Tooltip): Disable tooltip if media query is any-hover: none bug(COMPONENT): Tooltip display is wrong on ipad in PC mode

2 participants