-
Notifications
You must be signed in to change notification settings - Fork 88
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
Styling and icon for out of scale nodes #1161
Conversation
@@ -53,4 +53,8 @@ | |||
&.gmf-icon-text:after { | |||
content: '\e906'; | |||
} | |||
|
|||
&.gmf-icon-search-go:after { | |||
content: '\e908'; |
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.
should use the double dot :: for pseudo element
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.
Indeed.
Let's be consistent for now and change all the selectors once for all.
Can you please open an issue for addressing this right after that?
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
33919cb
to
0e28024
Compare
The icon don't mean anything at the standard zoom..., should the arrow be moved on the right? |
0e28024
to
362040f
Compare
@pgiraud Yes, wonderful :-) |
LGTM |
362040f
to
1539e81
Compare
Nice! |
1539e81
to
42b7bd7
Compare
4a5687a
to
8f0a4a0
Compare
Space added (with an arbitrary value chosen). Merging. |
I created an icon, and I hope it's clear enough.
The text is now italic when the layer is out of scale.
IMO the icon is intrusive and no really relevant when the node is not active so I hide it in this case.
Demos:
https://pgiraud.github.io/ngeo/layertree_out_of_scale/examples/contribs/gmf/apps/desktop
https://pgiraud.github.io/ngeo/layertree_out_of_scale/examples/contribs/gmf/apps/mobile
Remaining issues:
Positioning of elements is not perfect and the icon will be visible if the name of the layer is very long.
Please review.