Merge pull request #2073 from fredj/named-colors
Change ol.ENABLE_NAMED_COLORS to be false by default
This commit is contained in:
+2
-2
@@ -86,9 +86,9 @@ ol.ENABLE_IMAGE = true;
|
|||||||
/**
|
/**
|
||||||
* @define {boolean} Enable named colors. Enabling named colors adds about 3KB
|
* @define {boolean} Enable named colors. Enabling named colors adds about 3KB
|
||||||
* uncompressed / 1.5KB compressed to the final build size. Default is
|
* uncompressed / 1.5KB compressed to the final build size. Default is
|
||||||
* `true`.
|
* `false`.
|
||||||
*/
|
*/
|
||||||
ol.ENABLE_NAMED_COLORS = true;
|
ol.ENABLE_NAMED_COLORS = false;
|
||||||
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
@@ -13,7 +13,7 @@ describe('ol.color', function() {
|
|||||||
ol.color.fromStringInternal_.restore();
|
ol.color.fromStringInternal_.restore();
|
||||||
});
|
});
|
||||||
|
|
||||||
if (ol.color.ENABLE_NAMED_COLORS) {
|
if (ol.ENABLE_NAMED_COLORS) {
|
||||||
it('can parse named colors', function() {
|
it('can parse named colors', function() {
|
||||||
expect(ol.color.fromString('red')).to.eql([255, 0, 0, 1]);
|
expect(ol.color.fromString('red')).to.eql([255, 0, 0, 1]);
|
||||||
});
|
});
|
||||||
@@ -36,7 +36,7 @@ describe('ol.color', function() {
|
|||||||
[255, 255, 0, 0.1]);
|
[255, 255, 0, 0.1]);
|
||||||
});
|
});
|
||||||
|
|
||||||
if (ol.color.ENABLE_NAMED_COLORS) {
|
if (ol.ENABLE_NAMED_COLORS) {
|
||||||
it('caches parsed values', function() {
|
it('caches parsed values', function() {
|
||||||
var count = ol.color.fromStringInternal_.callCount;
|
var count = ol.color.fromStringInternal_.callCount;
|
||||||
ol.color.fromString('aquamarine');
|
ol.color.fromString('aquamarine');
|
||||||
|
|||||||
Reference in New Issue
Block a user