The OL style called "Default style" is not the style applied by default.
In other words, the style defined at admin/structure/openlayers/styles/list/default/edit is not applied to new layers unless explicitly set to override the style applied by default.
For example, "Default style" has fill color #888888 and stroke color #222222.
However, when I create a new layer without setting any particular style, the style applied by default has fill color #ee9900 and stroke color #ee9900.
As a corollary, there's no way to globally adjust the style OpenLayers applies by default.
So, 2 parts to this feature request:
1. It's very confusing to have a style called "Default style" which is not actually the style applied by default.
It should be renamed to something like "Basic style" to address this confusion.
2. OL should provide some UI to adjust the style which is automatically applied.
Please let me know if I'm missing something.
Apologies if this issue has been filed already.
It's quite difficult to search for such generic terms in the queue.
If this seems like a reasonable request, I am willing to work on the patch to provide this feature.
Thanks in advance.
Comment | File | Size | Author |
---|---|---|---|
#3 | openlayers-apply_default_styles-2072741.patch | 1.68 KB | AaronBauman |
Comments
Comment #1
PolHi Aaron,
Looks like it's a bug indeed and it should be fixed.
Comment #2
AaronBaumanJust so we're on the same page, the bug is that the "Default style" is not applied by default?
I will take a crack at this.
Comment #3
AaronBaumanOK, here's the patch.
Applies the styles selected under "Default Styles" fieldset when "<use default style>" is selected in the Overlay Layers table under "Layers and Styles" fieldset.
Note: this means any existing map implementations which use OpenLayers' (the library) defaults will now be updated to use OpenLayers' (the module) defaults. (E.g. the markers which were #ee9900 will become #222222).
To me this doesn't seem like too big a deal, since technically these maps had style settings that were broken / ignored, and this patch fixes them.
Comment #4
AaronBaumanComment #5
PolMake sense ! Thank you very much !