Skip to content

Commit 0d6d5fd

Browse files
authored
[chore] raster tile - hide server settings by default (#3163)
* [chore] raster tile - hide server settings by default Signed-off-by: Ihor Dykhta <dikhta.igor@gmail.com> * fix ts/lint Signed-off-by: Ihor Dykhta <dikhta.igor@gmail.com> --------- Signed-off-by: Ihor Dykhta <dikhta.igor@gmail.com>
1 parent 7551f5d commit 0d6d5fd

File tree

3 files changed

+21
-14
lines changed

3 files changed

+21
-14
lines changed

src/components/src/modals/tilesets-modals/tileset-raster-form.tsx

Lines changed: 16 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,8 @@ const RasterTileForm: React.FC<RasterTileFormProps> = ({setResponse}) => {
187187
setResponse
188188
]);
189189

190+
const showServerInput = getApplicationConfig().rasterServerShowServerInput;
191+
190192
return (
191193
<TilesetInputContainer>
192194
<div>
@@ -210,18 +212,20 @@ const RasterTileForm: React.FC<RasterTileFormProps> = ({setResponse}) => {
210212
Supports raster .pmtiles. Limited support for STAC Items and Collections.
211213
</TilesetInputDescription>
212214
</div>
213-
<div>
214-
<label htmlFor="tileset-raster-servers">Raster tile servers</label>
215-
<InputLight
216-
id="tileset-raster-servers"
217-
placeholder="Raster tile servers (separated by commas)"
218-
value={rasterTileServerUrls}
219-
onChange={onRasterTileServerUrlsChange}
220-
/>
221-
<TilesetInputDescription>
222-
Raster tile server URLs for Cloud Optimized GeoTIFF tilesets and elevation.
223-
</TilesetInputDescription>
224-
</div>
215+
{showServerInput && (
216+
<div>
217+
<label htmlFor="tileset-raster-servers">Raster tile servers</label>
218+
<InputLight
219+
id="tileset-raster-servers"
220+
placeholder="Raster tile servers (separated by commas)"
221+
value={rasterTileServerUrls}
222+
onChange={onRasterTileServerUrlsChange}
223+
/>
224+
<TilesetInputDescription>
225+
Raster tile server URLs for Cloud Optimized GeoTIFF tilesets and elevation.
226+
</TilesetInputDescription>
227+
</div>
228+
)}
225229
</TilesetInputContainer>
226230
);
227231
};

src/utils/src/application-config.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,8 @@ export type KeplerApplicationConfig = {
6868
rasterServerServerErrorsToRetry?: number[];
6969
/** Maximum number of simultaneous requests per raster server. 0 - no limit */
7070
rasterServerMaxPerServerRequests?: number;
71+
/** Whether to show the server input field in the raster tile layer setup form */
72+
rasterServerShowServerInput?: boolean;
7173

7274
// WMS layer config -- Experimental
7375
// WMS layer is under development and not ready for production use. Disabled by default.
@@ -123,6 +125,7 @@ const DEFAULT_APPLICATION_CONFIG: Required<KeplerApplicationConfig> = {
123125
rasterServerRetryDelay: 10000,
124126
rasterServerServerErrorsToRetry: [503],
125127
rasterServerMaxPerServerRequests: 0,
128+
rasterServerShowServerInput: false,
126129

127130
// WMS layer config
128131
enableWMSLayer: false

test/browser/layer-tests/wms-layer-specs.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -391,11 +391,11 @@ test('#WMSLayer -> edge cases and error handling', t => {
391391
});
392392

393393
// Test deckLayerRef storage
394-
const firstRender = layer.renderLayer(MOCK_RENDER_OPTS);
394+
layer.renderLayer(MOCK_RENDER_OPTS);
395395
t.ok(layer['deckLayerRef'], 'should store deckLayerRef after first render');
396396

397397
const firstRef = layer['deckLayerRef'];
398-
const secondRender = layer.renderLayer(MOCK_RENDER_OPTS);
398+
layer.renderLayer(MOCK_RENDER_OPTS);
399399
t.ok(layer['deckLayerRef'], 'should have deckLayerRef after second render');
400400
t.notEqual(layer['deckLayerRef'], firstRef, 'should update deckLayerRef on new render');
401401

0 commit comments

Comments
 (0)