-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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 CSS cleanup for the navbar #6553
Conversation
Codecov Report
@@ Coverage Diff @@
## master #6553 +/- ##
==========================================
- Coverage 40.37% 40.36% -0.02%
==========================================
Files 405 405
Lines 54260 54260
==========================================
- Hits 21907 21901 -6
- Misses 29337 29344 +7
+ Partials 3016 3015 -1
Continue to review full report at Codecov.
|
Small fix pushed to navbar height. Now the min-height is set on the navbar itself rather than the logo which should allow users to use logos less than 30px high. |
@silverwind any screenshot? |
* Minor CSS cleanup for the navbar * move height fix to navbar Fixes: go-gitea#6776
Few minor tweaks related to the navbar:
!important
to navbar wrapper'smargin
to eliminate a small gap visible in the navbar border on top-right caused by semantic-ui margins.min-height
to the logo to prevent navbar changing height while the image is still loading.