-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Change default indent to 40px #4025
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@egonbolton Thanks for the contribution! I'm on board with this change, even if we might initially encounter some issues with nested lists on smaller view ports. I think aiming for browser consistency though is a decent aim though. Mind updating the failing E2Es and I can merge? |
Cool! This is done :) |
@egonbolton Sorry, I know this is annoying, but do you mind fixing the merge conflicts and then I can merge? |
873c654
to
0db2144
Compare
40px would be a better default because that's what browsers use for lists.
Indentations in paragraphs, headings, etc, would match that of the lists, and converting the nodes would not generate a strange displacement.
https://www.loom.com/share/6443d73269d04769bd88c00c8f1b973b
You may not see a perfect match with the list indents in the playground, because it's using some styles like
list-style-position: inside
.Issue covered here: #3999