Open markusand opened 3 years ago
Ah, interesting suggestions. What does "load" mean for you in each of these contexts, and what are your use cases exactly?
I guess a loaded source means when all features have been set as data and are available, and a loaded layer would mean when has already been drawn on the map.
There are many events in mapbox such as sourcedata
or style.load
that may be used together to verify the resource status, but all implementations I've been using end up being a little bit buggy.
This is the most recent I'm using to detect source changes, still not giving 100% good results
const sourceId = 'source_id';
const sourceLoaded = source => {
console.log('Source loaded', source);
};
const onLoadStart = startEvent => {
if (startEvent.sourceId === sourceId) {
map.off('dataloading', onLoadStart);
const onLoadEnd = () => {
if (endEvent.sourceId === sourceId && endEvent.isSourceLoaded && map.isSourceLoaded(sourceId)) {
map.on('dataloading', onLoadStart);
map.off('sourcedata', onLoadEnd);
sourceLoaded(map.getSource(sourceId));
}
};
map.on('sourcedata', onLoadEnd);
}
};
map.on('dataloading', onLoadStart);
Use cases are multiple, in my case I'm using to detect when a vector tile source has finished to (re)load after a pan or zoom change.
I'm also using styledata
and style layers checks to trigger an event when some layers change in a legend plugin I'm working on, again with some headaches :P
Yeah, the inability to consistently detect these events is such a huge pain point :/
If you can come up with an implementation you're satisfied with, I'd be happy to include it.
I'm making some VERY OPINIONATED assumptions and I haven't tested widely so I'm not sure about possible side effects, but seems to work pretty well on my use case and it's clearly a lot more readable that my previous suggestion.
const onSourceLoad = (name, { onLoadStart, onLoadEnd }) => {
let isSourceLoading = false;
map.on('sourcedata', event => {
if (event.sourceId !== name) return;
// Source starts loading
// Consider an "initial" data event when sourceDataType is undefined or `content`
// Discard `visibility` as it triggers very irregularly
const { sourceDataType = 'content' } = event;
if (!isSourceLoading && sourceDataType === 'content') {
if (onStartLoading) onStartLoading(map.getSource(name));
isSourceLoading = true;
}
// Source finishes loading
if (isSourceLoading && map.isSourceLoaded(name)) {
if (onEndLoading) onEndLoading(map.getSource(name));
isSourceLoading = false;
}
});
};
I've changed a little bit the approach to trigger different actions when layer starts and finishes loading, so it could be used as:
map.U.onSourceLoad('sourcename', {
onLoadStart: source => { /* Do whatever */ },
onLoadEnd: source => { /* Do whatever */ },
});
I'm currently using this method to show and hide a loading spinner whenever a vt source is updating.
Thanks for this - looks useful. I will have to look more closely. I'm a little bit iffy about the function signature (having to spell out onLoadStart
and onLoadEnd
...). Maybe it would be cleaner/simpler to just have two separate functions: map.U.onSourceLoadStart(name, cb)
and map.U.onSourceLoadEnd(name, cb)
.
Does this work on both vector tiles and GeoJSON? Does it trigger per tile, or after all the tiles for the current view have loaded?
Object signature is convenient in my particular usecase, but certainly adds some bloating.
Works both on vt and geojson. Some considerations:
I wish Mapbox will release some day a good source and layers events API, because all this starts to feel very hacky to me 😓
Triggers on geojson when zooming and panning, like if load was batched internally. I may have misunderstood how geojson works, but I thought it was loaded at once.
GeoJSON is loaded in bulk but then converted into vector tiles by mapbox-gl-js.
I have now tested this and it does seem useful.
I found an edge case. Functions where triggered by sourcedata
on any source/layer change, including setting a simple filter in a layer. I solved it by splitting into two functions and binding the onStartLoading part to the sourcedataloading
event. This seems to work smoother only when actual source data is being loaded.
let isLoading = false;
const bindLoadStart = event => {
const { sourceDataType = 'content' } = event; // Little hack here
if (event.sourceId === sourceId && !isLoading && sourceDataType === 'content') {
if (onStartLoading) onStartLoading(event);
isLoading = true;
}
};
const bindLoadEnd = event => {
if (event.sourceId === sourceId && isLoading && map.isSourceLoaded(sourceId)) {
if (onEndLoading) onEndLoading(event);
isLoading = false;
}
};
const bindLoading = () => {
map.on('sourcedataloading', bindLoadStart);
map.on('sourcedata', bindLoadEnd);
};
const unbindLoading = () => {
map.off('sourcedataloading', bindLoadStart);
map.off('sourcedata', bindLoadEnd);
};
Sorry if I'm spamming too much on this topic. Just wanting to let you know.
Thanks, this is great.
This utility library is really handy. One valuable utility that would solve many people's headaches would be to have a reliable way to detect when a source or layer has been loaded properly.