-
Notifications
You must be signed in to change notification settings - Fork 23
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
Refine edge and content alignment #37
Comments
This is a little hard for me to visualize, but it sounds like what I see in the design mockups. |
Yes, indeed @iandunn @coreymckrill, I should have asked this before because it is not ideal as screen size increases. Is it possible to make the body text get bigger as the screen size gets bigger? For example, while at 1320px screen size we have a post in a Regular Paragraph |
That sounds like a good idea 👍🏻 I opened an issue to track it, since it can be done separately: #39 Some examples in the Figma would be great, thanks! |
Great, thanks. Added those examples to the Figma file. |
There are a couple related discussions in closed threads, so consolidating here.
goals
121
alignwide
48
alignfull
:116
@beafialho, @coreymckrill, does all that seem right to you?
@beafialho, what are your thoughts on the characters per line? they seem much higher than the usual guideline
implementation
body > * {...}
-- to apply the new one. the header can overwrite it, or we could do*:not(.site-header)
or whatever.alignfull
to only text children (not images/videos/etc)any thoughts @coreymckrill ?
The text was updated successfully, but these errors were encountered: