-
Notifications
You must be signed in to change notification settings - Fork 64
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
Increase @max-content-width
variable
#427
Comments
Now, that is an interesting question.... Factors to include:
Everything in green in the charts below, signifies that the media will use less than or equal 80% of the available screen height. The colours spreading into the top left are simply media ratios vs device ratios, showing the % device height occupied. What you should be able to see, is that:
More specifically:
I think 1024 is the maxium you could go, supporting 16:9 media only at short laptop/desktop widths. Obviously this changes slightly if you choose 90% as a good available height, the above is only a rough model. It may be worth having a max-width + horizontal center on media which must retain their aspect ratios, where we know they will overhang the viewport? That would a mean variable max content width on media components. References: |
The max content width went up from 900 to 960 with the breakpoints merge. |
I mentioned this in this thread (adaptlearning/adapt-contrib-core#364), going forward we need to think about this in the context of a more modern layout grid mentioned in that breakpoints thread. Personally, I would leave this at 960 for now and increase the width with some supporting less for other layout furniture. If we wanted to do this now, ideally this would be the same as a future content width, I'm suggesting 1200 but this needs some more thought. @oliverfoster is absolutely right about media components and viewport height due to aspect, but I don't think this should limit the content width because of this. Rather we should limit the width of a full span media component and / or redesign it to work nicely / look good. |
Subject of the enhancement
Proposal to increase the size of the content container to be wider than the current 960px equivalent.
Initial thoughts on what the value could be range from 1024px to 1280px with pros and cons for each end.
Past knowledge suggests 1120px would be a happy medium between the two ends of the range whilst also allowing the media component to fit entirely on screen for the wide, but small in height, laptop screens.
Your environment
The text was updated successfully, but these errors were encountered: