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.

Sign up for GitHub

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

Jump to bottom

UX: Cleanup html structure #26915

Closed

jordanvidrine wants to merge 1 commit into discourse:main from jordanvidrine:clean-topic-admin

Contributor

jordanvidrine commented May 7, 2024

This PR removes the two span wrappers from the topic admin button. This change fixes:

double border on the popup menu
the height of this button was 2px shorter than others, only noticed when another button is present near it
ran eslint & prettier on the file

After

Before


          UX: Cleanup html structure

ae7998c

CvX approved these changes

View reviewed changes

jordanvidrine closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment