-
Notifications
You must be signed in to change notification settings - Fork 13
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
Updates breakpoint preview feature #436
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's just one bit where I feel like some explanation is desirable ("limited support") but this is a good improvement
docs/guide/breakpoint-preview.md
Outdated
- Parameters like `orientation: landscape` or `orientation: portrait` do not apply to containers, as containers lack inherent orientation. | ||
- Example of unsupported query: | ||
- Parameters like `orientation: landscape` or `orientation: portrait` don't have equivalent behavior in container queries, as containers don't have the same orientation concept as viewports. | ||
- Example of query with limited support: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Limited how?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, reading with fresh eyes, that wording is a little off
docs/guide/breakpoint-preview.md
Outdated
|
||
/* Converted to */ | ||
.element { | ||
width: 50cqw; /* 50% of container width, not viewport */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this have any practical pitfalls? Reading about it, it seems like it should be fine?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I updated this to better describe the pitfalls.
Summary
Summarize the changes briefly, including which issue/ticket this resolves. If it closes an existing Github issue, include "Closes #[issue number]"
This PR updates the breakpoint preview documentation to reflect the limitations of the new PostCSS extension being used. Closes PRO-7372.
What are the specific steps to test this change?
For example:
What kind of change does this PR introduce?
(Check at least one)
Make sure the PR fulfills these requirements:
If adding a new feature without an already open issue, it's best to open a feature request issue first and wait for approval before working on it.
Other information: