Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update leaflet version #262

Merged
merged 1 commit into from Aug 25, 2015
Merged

update leaflet version #262

merged 1 commit into from Aug 25, 2015

Conversation

ktoto
Copy link
Member

@ktoto ktoto commented Jul 7, 2015

Update leaflet to 1.0-beta1
Leaflet updated event handling Leaflet/Leaflet#3307
and tileSize now Point
Also the default z-index of panes to be bigger

@Trufi
Copy link
Contributor

Trufi commented Jul 7, 2015

👍

@ktoto
Copy link
Member Author

ktoto commented Jul 8, 2015

Есть проблема с проваливанием клика в карту, как раз, связанное с последними изменениями :(
Ждем:
Leaflet/Leaflet#3605
Leaflet/Leaflet#3606

@ktoto ktoto force-pushed the update-leaflet branch 2 times, most recently from c161435 to 5bda1f3 Compare July 13, 2015 06:16
@ktoto ktoto force-pushed the update-leaflet branch 2 times, most recently from f7081d2 to 92ed1de Compare July 15, 2015 06:32
@ktoto
Copy link
Member Author

ktoto commented Jul 15, 2015

В лифлете все поправили. Можно тестировать

@Trufi
Copy link
Contributor

Trufi commented Jul 15, 2015

@ktoto https://www.npmjs.com/package/html5shiv пропал из npm, пока не потестировать

@ktoto
Copy link
Member Author

ktoto commented Jul 16, 2015

@Trufi ребейзнул, все ок :)

@Trufi
Copy link
Contributor

Trufi commented Jul 16, 2015

Класс, мы хотим слить его после сегодняшнего релиза

@ktoto
Copy link
Member Author

ktoto commented Jul 16, 2015

Потестировать бы :)

@osvodef
Copy link
Contributor

osvodef commented Jul 16, 2015

👍

@andymost
Copy link
Contributor

👓

@andymost andymost removed the Testing label Jul 16, 2015
@andymost
Copy link
Contributor

Закрываются балуны при драге.
StR:

  1. Открыть любой балун (балун из демо подйдет)
  2. Драгнуть карту
    ER: Балун на месте
    Got: Закрываются

@ktoto
Copy link
Member Author

ktoto commented Jul 16, 2015

Опять ждем:
Leaflet/Leaflet#3632
Leaflet/Leaflet#3633

@andymost
Copy link
Contributor

@ktoto я посмотрел есть проблемы с событиями геометрий и лейблом линейки по ховеру. Линейку можно глянуть на http://mapsapi.test.webmaps.ostack.test/2.0/ а пример с геометрией можно из премеров взять.

@ktoto
Copy link
Member Author

ktoto commented Jul 30, 2015

@andymost zIndex поправил. В доке тоже поменял.
В новом лифлете события "всплывают", как в dom'е и поэтому click ловится сначала полигоном, а потом картой.
Надо ли поддерживать старое поведение? @ilfa

@ktoto ktoto force-pushed the update-leaflet branch 2 times, most recently from a4e58fb to 285a57b Compare July 30, 2015 09:50
@ktoto
Copy link
Member Author

ktoto commented Jul 30, 2015

Вернул старое поведение с событиями в DGCustomization

@osvodef
Copy link
Contributor

osvodef commented Aug 17, 2015

👍

@ilfa ilfa added Testing and removed CodeReview labels Aug 17, 2015
@andymost
Copy link
Contributor

👓

@andymost
Copy link
Contributor

При ховере маркера в ie10, он не подсвечивается.

@andymost
Copy link
Contributor

При ховере линейки в ie10 не появляеться лейбл с метражем.

@andymost
Copy link
Contributor

Поплыли контейнеры с контролами в IE10

@ilfa
Copy link
Contributor

ilfa commented Aug 19, 2015

Нашел корень зла. Leaflet определяет IE10 как мобильный браузер, завел тикет Leaflet/Leaflet#3761 Можем пока обновиться на лифлет до злополучного коммита.

@ilfa ilfa force-pushed the update-leaflet branch 2 times, most recently from 1acfa80 to 3871f0b Compare August 20, 2015 05:39
@ilfa
Copy link
Contributor

ilfa commented Aug 20, 2015

Запилил форк у нас и впихнул туда откат того неудачного фикса.

@andymost
Copy link
Contributor

При зуме inch`ом на мобилке карта не отвечает. Android 4.2, chrome 44. Воспроизведено на pages/base.html в демо страницах для тестов.

andymost added a commit that referenced this pull request Aug 25, 2015
@andymost andymost merged commit 0e0c93b into master Aug 25, 2015
@andymost andymost deleted the update-leaflet branch August 25, 2015 11:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants