sts-sponsors team mailing list archive

[Peter Makowski <mp+439727@xxxxxxxxxxxxxxxxxx>]

Peter Makowski has proposed merging ~petermakowski/maas-site-manager:use-default-x-large-breakpoint into maas-site-manager:main.

Requested reviews:
  MAAS Committers (maas-committers)

For more details, see:
https://code.launchpad.net/~petermakowski/maas-site-manager/+git/site-manager/+merge/439727

QA Steps
- Run the app locally
- Resize the window and verify the font size only increases size past the default breakpoint (1681px) 

https://github.com/canonical/vanilla-framework/blob/ea6f05b4a90169b36a5e9c5a86ceae40cbddd986/scss/_settings_breakpoints.scss#L8
-- 
Your team MAAS Committers is requested to review the proposed merge of ~petermakowski/maas-site-manager:use-default-x-large-breakpoint into maas-site-manager:main.

diff --git a/frontend/src/_settings.scss b/frontend/src/_settings.scss
index 1c7d703..55a751d 100644
--- a/frontend/src/_settings.scss
+++ b/frontend/src/_settings.scss
@@ -3,7 +3,6 @@
 // Vanilla settings:
 $grid-max-width: math.div(1920, 16) * 1rem;
 
-$breakpoint-x-large: 1440px;
 $breakpoint-xx-large: 1920px;
 $side-navigation-z-index: 103;
 $application-layout--breakpoint-side-nav-expanded: $breakpoint-xx-large;