Skip to content
This repository was archived by the owner on Jan 6, 2025. It is now read-only.

fix(core): update breakpoint ranges to avoid overlapping #1075

Merged
merged 1 commit into from
May 30, 2019
Merged

fix(core): update breakpoint ranges to avoid overlapping #1075

merged 1 commit into from
May 30, 2019

Conversation

k0hamed
Copy link
Contributor

@k0hamed k0hamed commented May 30, 2019

In Chrome, .99 is treated as 1 which cause ranges to overlap at the exact breakpoint.

Note: the behavior might not be considered as a bug in chrome as in W3 specifications it's mentioned that for sub-pixels or fractions it's preferred to use the not widely supported range context.

Fixes #1052. Affects #1001.

In Chrome, .99 is treated as 1 which cause ranges to overlap at the exact breakpoint.

> Note: the behavior might not be considered as a bug in chrome as in [W3 specifications](https://www.w3.org/TR/mediaqueries-4/#mq-min-max) it's mentioned that for sub-pixels or fractions it's preferred to use [the not widely supported range context](https://www.w3.org/TR/mediaqueries-4/#range-context).

Fixes #1052. Affects #1001.
@k0hamed k0hamed changed the title fix(core): update breakpoints ranges to avoid overlapping fix(core): update breakpoint ranges to avoid overlapping May 30, 2019
@CaerusKaru CaerusKaru self-assigned this May 30, 2019
@CaerusKaru CaerusKaru added the pr: merge ready This PR is ready for the caretaker to presubmit and merge label May 30, 2019
@CaerusKaru CaerusKaru added this to the 8.0.0-beta.27 milestone May 30, 2019
@CaerusKaru CaerusKaru merged commit 31cb34e into angular:master May 30, 2019
@angular-automatic-lock-bot
Copy link

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 Sep 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes pr: merge ready This PR is ready for the caretaker to presubmit and merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Both sm and xs breakpoints are active when screen width is 600px
3 participants