Skip to content
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

examples: don't inject ol.css from loader.js #881

Merged
merged 1 commit into from Jul 25, 2013

Conversation

fredj
Copy link
Member

@fredj fredj commented Jul 25, 2013

Because the style sheet is added at the end of the document, overwriting style is difficult (!important flags must be added).

@ahocevar
Copy link
Member

I am in favor of making this change.

@@ -0,0 +1,41 @@
goog.require('ol.structs.RTree');
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you mean to commit this file?

Copy link
Member Author

Choose a reason for hiding this comment

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

thanks, file removed

@twpayne
Copy link
Contributor

twpayne commented Jul 25, 2013

+1 (although I think an extra file snuck into the commit).

fredj added a commit that referenced this pull request Jul 25, 2013
examples: don't inject ol.css from loader.js
@fredj fredj merged commit d30efda into openlayers:master Jul 25, 2013
@fredj fredj deleted the examples_css branch July 25, 2013 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants