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

Set a max-width on the examples container as to more accurately reflect what it will look like #418

Closed
joelanman opened this issue Jun 29, 2018 · 6 comments

Comments

@joelanman
Copy link
Contributor

From @davehaigh on June 29, 2018 14:20

the examples wrapper is filling the entire width of the screen so components are rendered a lot wider than they are normally seen e.g open https://design-system.service.gov.uk/components/panel/ on a desktop pc. The 'confirmation panel' is very wide when normally its only 960px max.

I think the container should have a max width of 960px as to more accurately reflect the prototype kit.

image

Copied from original issue: alphagov/govuk-design-system-backlog#142

@dashouse
Copy link

dashouse commented Jul 2, 2018

If we want to achieve this we should set the width of the column not the example container. At the moment the paragraphs, images, tables are all constrained to a maximum width to allow for the code blocks to be 100% and break out of the visible layout.

If we no longer want the code blocks to be 100% we can remove all of the individual constraints and set a max width on the column itself.

@kr8n3r
Copy link
Contributor

kr8n3r commented Jul 2, 2018

they're only constrained in the page content, not the example (as that's using a different template

we should be able to add max width to the app-example-page in the example layout

@36degrees
Copy link
Contributor

@dashouse do you think there's a clear change that we need to make here, or does this need more thinking?

@dashouse
Copy link

@36degrees More thinking definitely, it's really about whether or not we're trying to represent the components as they actually behave (in this case 100% wide) or how we want them to be used (simulate 2/3 column width).

I prefer as is because it it's more of a representation of what you're going to get and makes the examples that DO have limited widths more obvious.

This caused a big debate so would prefer to look into it...

@dashouse
Copy link

dashouse commented Jan 8, 2019

Closing this issue as it has been recorded in Trello and will be considered when we look at the the layout improvements to the Design System - for example https://github.com/alphagov/govuk-design-system/tree/center-layout

@dashouse dashouse closed this as completed Jan 8, 2019
@dashouse
Copy link

Fixed by #687 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants