-
Notifications
You must be signed in to change notification settings - Fork 290
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
consolidate map size css #4808
consolidate map size css #4808
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4808 +/- ##
=========================================
Coverage 30.95% 30.95%
Complexity 11372 11372
=========================================
Files 1167 1167
Lines 47861 47861
=========================================
Hits 14813 14813
Misses 33048 33048 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
|
||
<div class="col-sm-3"> | ||
<div class="col-sm-3 wt-sidebar wt-global"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use wt-global
on the body
element (because it is generallly better to target classes than elements).
I don't think we should use it elsewhere.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok but without it the sidebar background becomes black so I'll have to add the admin background image to another class
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We use wt-global on the body element
Fixed
a61108b
to
a55f0ec
Compare
a55f0ec
to
6c77a7e
Compare
Thanks! |
* consolidate map size css into one file (/resources/css/_maps.css) * Remove use of wt-global for sidebar background * Adjust class names and use ::backdrop pseudo class for all themes
* consolidate map size css into one file (/resources/css/_maps.css) * Remove use of wt-global for sidebar background * Adjust class names and use ::backdrop pseudo class for all themes
merge all into one file - renamed /resources/css/_leaflet.css to /resources/css/_maps.css