mapbox/mapbox-gl-js

off('load') then on('load') does not work using setStyle

Open

#10.946 aberto em 19 de ago. de 2021

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (2.203 forks)batch import
bug :lady_beetle:good first issue

Métricas do repositório

Stars
 (10.532 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

mapbox-gl-js version: v2.4.0

browser: Chrome Version 92.0.4515.159

Steps to Trigger Behavior

Reproduce steps

  1. add Map.on('load', func)
  2. add Map.off('load', func)
  3. add Map.on('load', func2)
  4. call Map.setStyle()

Link to Demonstration

https://jsfiddle.net/tsuzk/6o8m9nad

This should work because of this test

Expected Behavior

The Map.on('load', func2) should get called after Map.setStyle()

Actual Behavior

The Map.on('load', func2) does not get called after Map.setStyle()

Workaround

re-instantiate the map with a new stlye https://jsfiddle.net/tsuzk/6o8m9nad/1/

Other info

  • This occurs on v1.9.0 so it's not a new behavior in v2.

Priority

Low, since there is a proper workaround

Guia do colaborador

Direção de pesquisa
Investigue o tratamento de eventos no Mapbox GL JS, especificamente como o ouvinte do evento 'load' é gerenciado quando setStyle é chamado. Comece examinando o código fonte da classe Map em /src/ui/map.js e o sistema de eventos. O relatório de bug faz referência a um teste em test/unit/ui/map.test.js por volta das linhas 136-151 que espera que o comportamento funcione, mas parece que o teste pode estar incorreto ou a implementação falha em certos casos. Concentre se nos métodos on/off para o evento 'load' e em como setStyle dispara o reacionamento do evento. Revise quaisquer PRs ou comentários vinculados para contexto adicional.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptEvent handling
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70