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

Fix button margins after #1451 #1676

Closed
samuelsidler opened this issue Aug 18, 2023 · 1 comment · Fixed by #1677
Closed

Fix button margins after #1451 #1676

samuelsidler opened this issue Aug 18, 2023 · 1 comment · Fixed by #1677
Labels
[Type] Bug An existing feature is broken.
Milestone

Comments

@samuelsidler
Copy link
Contributor

Relevant Git Blame: https://github.com/GlotPress/GlotPress/blame/8f793a657e7bacda73f96cad267739cae5c1f334/assets/css/style.css#L336

In #1451, it appears the margins on buttons were changed, but it's unclear why they were changed. The additional margin makes all buttons appear misaligned within GlotPress, where previously they were vertically centred/aligned.

Examples:
Screenshot 2023-08-18 at 09 01 14
Screenshot 2023-08-18 at 09 01 39

@amieiro: Are there instances where this additional bottom margin is important? If so, can you target those instances specifically? I can't find any instances where this additional bottom margin is necessary.

@ocean90 ocean90 added the [Type] Bug An existing feature is broken. label Aug 18, 2023
@ocean90 ocean90 added this to the 4.0 milestone Aug 18, 2023
samuelsidler added a commit to samuelsidler/GlotPress that referenced this issue Aug 18, 2023
Buttons margins appear to have broken after GlotPress#1451 and are now misaligned. This adjusts them back to the pre-1451 state.

Fixes GlotPress#1676.
@amieiro
Copy link
Member

amieiro commented Aug 21, 2023

@samuelsidler if we need the previous change in another instance like translate.w.org, we will do this improvement there, and not here, so it is ok to revert this change. I have checked it and:

I am going to approve and merge this PR.

Thank you!

amieiro pushed a commit that referenced this issue Aug 21, 2023
Buttons margins appear to have broken after #1451 and are now misaligned. This adjusts them back to the pre-1451 state.

Fixes #1676.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Bug An existing feature is broken.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants