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

Text-only buttons in toolbar have uneven padding #26748

Closed
karmatosed opened this issue Nov 5, 2020 · 3 comments · Fixed by #26769
Closed

Text-only buttons in toolbar have uneven padding #26748

karmatosed opened this issue Nov 5, 2020 · 3 comments · Fixed by #26769
Assignees
Labels
Needs Design Feedback Needs general design feedback. [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended

Comments

@karmatosed
Copy link
Member

When you turn on the option for text-only options the following issue arises with padding:

2020-11-05 19 39 02

Here is a suggestion of how a little tidy could be done, I will also work on a PR for this. The focus here is on tidying up some inconsistencies. This version takes it a little wider and far to 4px spacing, however I would like to explore how little space can be taken up, but still consistent.

2020-11-05 20 17 51

Feedback

As with anything, I would welcome input on this. I consider it a bug as this I think was previously not an issue.

@karmatosed karmatosed added [Type] Bug An existing feature does not function as intended Needs Design Feedback Needs general design feedback. labels Nov 5, 2020
@karmatosed karmatosed self-assigned this Nov 5, 2020
@mapk
Copy link
Contributor

mapk commented Nov 5, 2020

In the original PR, #24304, @tellthemachines went through several iterations to make sure the spacing and padding of each button was correct. Something must have caused a hiccup here.

If we can get back to that solution again, I think we'd be golden. :)

@tellthemachines
Copy link
Contributor

Looking back at the history of that PR, I think this comment might have been overlooked, as there are no commits after it. The issue has probably been there since that PR was merged.

Happy to help with this, and get it into 5.6. Something like

.edit-post-header-toolbar__left > *+* {
    margin-left: 4px;
}

should fix it; and maybe reducing the margins on some of the right hand side buttons, as per @mapk 's original comment?

Are you working on a PR for this @karmatosed ? Let me know if you need anything!

@karmatosed
Copy link
Member Author

@tellthemachines and @mapk thank you both so much for the context. I have worked on a speedy PR so hopefully that at least gets what was missing in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Feedback Needs general design feedback. [Status] In Progress Tracking issues with work in progress [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants