Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix position of ScaleLine Control (OL3) #93

Merged
merged 3 commits into from
Oct 12, 2016
Merged

Fix position of ScaleLine Control (OL3) #93

merged 3 commits into from
Oct 12, 2016

Conversation

jbelien
Copy link
Contributor

@jbelien jbelien commented Oct 12, 2016

Fix #92

Copy link
Owner

@Turbo87 Turbo87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the file that needs to be changed is https://github.com/Turbo87/sidebar-v2/blob/master/scss/ol3-sidebar.scss, the CSS files are generated from that file.

@@ -26,6 +26,26 @@ $sidebar-inner-border-radius: 2px !default;
margin-left: $width-lg + $sidebar-margins + $sidebar-border-width * 2;
}
}

.ol-scale-line {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we use .ol-zoom, .ol-scale-line { above to reduce code duplication?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of course, stupid me !
Fix in a few seconds !

@Turbo87
Copy link
Owner

Turbo87 commented Oct 12, 2016

cool, thanks!

@Turbo87 Turbo87 merged commit 9fe8ae3 into Turbo87:master Oct 12, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants