tdurand / mapafortaleza

Mapa de ônibus de Fortaleza
https://tdurand.github.io/mapafortaleza
79 stars 34 forks source link

Adicionei a possibilidade de se "buscar rotas" #1

Closed regismesquita closed 12 years ago

regismesquita commented 12 years ago

Adicionei um botão que insere um marker no mapa e então ao se movimentar esse marker (via drag and drop) ele automaticamente filtra na lista de rotas, todas as rotas que passam a pelo menos 1km daquele ponto facilitando a busca de rotas que passem por onde a pessoa precisa.

basta inserir dois markers, arrastar para o destino e o outro para a origem e pronto , você tem uma lista de rotas para onde precisa ir por exemplo...

tdurand commented 12 years ago

Muito obrigado para sua contribuição, queria fazer isso também.

Quero só melhorar um pouco a interface com usuario antes de integrar a sua funcionalidade.

regismesquita commented 12 years ago

Fiz ontem de madrugada logo estava com bastante sono e por isso não pensei o suficiente na UX só na funcionalidade mesmo tambem estava pensando em como melhorar ela , antes de dormir pensei em ao invés de ter o botão simplesmente clicar na tela para adicionar , ao ser adicionado já começa a filtrar e um duplo clique removeria o botão.

Se preferir posso implementar já que seria bem simples , mas ainda não sei se seria intuitivo o suficiente para o programa, o que acha?

Ao fazer isso ainda poderia retirar o code smell do var markers=[] já que ele não tem necessidade de estar fora do model do mapa que é quem logicamente guardaria o esse atributo.

tdurand commented 12 years ago

Estou fazendo agora uma coisa, vou acabar daqui a pouco, depois podemos melhorar.

regismesquita commented 12 years ago

ok.

tdurand commented 12 years ago

Atualizei a aplicação com a sua primera contribuição. Muito obrigado, vou adicionar uma referência na lista na parte inferior da página.

Vou olhar para a sua última contribuição.

regismesquita commented 12 years ago

A minha ultima é conflitante com a sua atualização , embora a minha versão seja mais rápida do ponto de vista de utilização a sua é bem mais intuitiva, considerando a aplicação acredito que a sua seja mais interessante de ser mantida.

tdurand commented 12 years ago

Eu vi a sua contribuição. Eu acho que os dois podem coexistir.

Agora não tenho o tempo para integrar, más eu vou ;-).

PS: acho que é melhor fazer doble clic para adicionar, porque as vezes eu clico na mapa para ampliar.

regismesquita commented 12 years ago

O fato de os clicks se confundirem com o sistema de zoom atrapalha bastante a minha segunda implementação , acho que ficaria melhor um control no mapa podendo arrastar o marker para a posição desejada , enfim , sinta-se livre para fechar o pull request e adicionar ou não uma issue para efetuar essa mudança no futuro.

tdurand commented 12 years ago

Ta certo. Então, vou criar uma issue para fazer isso mais tarde.

Muito obrigado para a sua contribuição, adicionei uma referencia na pagina !