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

Доработки: map #17

Closed
6 of 12 tasks
slogger opened this issue Dec 3, 2014 · 5 comments
Closed
6 of 12 tasks

Доработки: map #17

slogger opened this issue Dec 3, 2014 · 5 comments
Assignees

Comments

@slogger
Copy link
Collaborator

slogger commented Dec 3, 2014

Я наконец снова добрался до кода

Что сделал:

  • Разобрался с инфраструктурой (не очень красиво, но работает)
  • Переименовал блок ymaps -> map-helper
  • Добавил стандартное значение zoom
  • (ymaps) Добавил возможность указывать тип карты
  • (ymaps) Теперь геообъекты поддерживают все свои опции из апи в bemjson'e
  • Разбил примеры, по вендорному признаку

Что нужно сделать

  • Покрыть код тестами
  • Написать документацию
  • Доделать подключение google maps
  • bh шаблон

С чем испытываю проблемы

  • Загрузка API Google Maps,
    Отправляю правильный урл в loader_type_js
    получаю в ответ ошибку
     Failed to execute 'write' on 'Document': It isn't possible to write into a document from an asynchronously-loaded external script unless it is explicitly opened. 
  • Еще мне не очень нравится как сделан map.bemhtml, есть идеи как сделать красивее
@slogger slogger self-assigned this Dec 3, 2014
@sipayRT
Copy link
Owner

sipayRT commented Dec 3, 2014

а сам код где?

@slogger
Copy link
Collaborator Author

slogger commented Dec 3, 2014

@sipayRT
Copy link
Owner

sipayRT commented Dec 3, 2014

@slogger сразу PR отправь и разрабатывайся дальше - там удобней смотреть на код, который ти пишешь. Сейчас сложно отследить твою работу

@sipayRT
Copy link
Owner

sipayRT commented Dec 3, 2014

только не забудь, что PR нужно добавить именно в моем форке ;)

@sipayRT
Copy link
Owner

sipayRT commented Mar 16, 2015

moved to bem-contrib/hackaton#2

@sipayRT sipayRT closed this as completed Mar 16, 2015
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

No branches or pull requests

2 participants