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

Minor cosmetic change #5147

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Minor cosmetic change #5147

wants to merge 3 commits into from

Conversation

Alex-e107nl
Copy link

I think the btn-sm was forgotten

Motivation and Context

2 buttons next to each other that are not the same height...

Description

Made one button the same as the other :-)

How Has This Been Tested?

e107.nl frontpage

Types of Changes

  • [x ] Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (a change to man pages or other documentation)

Checklist

@CaMer0n
Copy link
Member

CaMer0n commented May 10, 2024

Thank you @Alex-e107nl ! Did you happen to test with multiple themes? Also, I see there's another change regarding plupload language files. Intentional ?

@Alex-e107nl
Copy link
Author

@CaMer0n the buttons should be the same because they are next to each other, tested it with my own themes but when you use btn-sm on the one the other also should use that in stead of btn-default.

The plupload problem is solved for all languages when placed in the file it should be, so it is intentional!

Thx!

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.

None yet

2 participants