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

Clean up media query breakpoints in components #4107

Merged
merged 2 commits into from
Nov 1, 2021

Conversation

bartaz
Copy link
Contributor

@bartaz bartaz commented Oct 29, 2021

Done

  • Remove or rename any remaining misaligned media query breakpoints in compoenents.
  • make sure max-width media queries don't overlap min-width media queries by 1px

Fixes #4082

QA

@webteam-app
Copy link

Demo starting at https://vanilla-framework-4107.demos.haus

@@ -24,7 +24,7 @@
use 13rem. This value should be adjusted on the application level based on
the content that needs to stay visible on the screen.
*/
@media (min-width: 772px) {
@media (min-width: 620px) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are in local CSS in the examples, so we need to update px values in example code.

@@ -5,7 +5,7 @@
<style>
{% include "docs/examples/layouts/application/_styles.css" %}

@media (min-width: 772px) {
@media (min-width: 620px) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

These are in local CSS in the examples, so we need to update px values in example code.

Copy link
Contributor

@sowasred2012 sowasred2012 left a comment

Choose a reason for hiding this comment

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

LGTM

@sowasred2012 sowasred2012 merged commit 3498e4b into canonical:vanilla-3.0 Nov 1, 2021
@bartaz bartaz deleted the breakpoints-cleanup-2 branch May 27, 2022 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants