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

Remove max-widths in layout #163

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

bjb568
Copy link

@bjb568 bjb568 commented Aug 19, 2015

Webpages should always be at 100% width.

It annoys me constantly seeing otherwise responsively-designed websites constrained within a tiny rectangle down the center of the screen, this site being a ridiculous 700px. It's way easier to read at full width.

@bjb568
Copy link
Author

bjb568 commented Aug 19, 2015

Now for images, I don't know which is better, either float them:

img p {
    float: right;
    clear: right;
    width: 25%; /* or whatever */
}

This would need to be in a media query, obviously it's only for larger viewports. The other option is centering them:

img p {
    display: block;
    margin: 0 auto;
}

Centering is safer, but it creates a large whitespace on both the left and right of the image when the viewport is large enough; this looks strange.

@JonHMChan
Copy link
Contributor

@Courtny @hellohynes any thoughts on this?

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

Successfully merging this pull request may close these issues.

2 participants