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

Различные доработки запрошенные в оригинальном плагине #13

Open
ange007 opened this issue Dec 31, 2015 · 15 comments

Comments

@ange007
Copy link
Owner

ange007 commented Dec 31, 2015

Сюда буду скидывать найденные "предложения" на странице оригинального плагина, может что из этого будет полезным.

@ange007
Copy link
Owner Author

ange007 commented Dec 31, 2015

  • Уберите из кода инлайн стили и пропишите их в css.
  • Добавить колбеки на событие инициализации селекта, изменения размера выпадающей часи(на всякий случай), обновления и на destroy.
  • Допишите для ul и li классы по бэму «.jq-selectbox__list» и «.jq-selectbox__item» например.
  • В выпадающем блоке добавьте обертывающий див для списка и выставляйте для него min-height. Это нужно для подключения плагина стилизации скролбара.

@ange007
Copy link
Owner Author

ange007 commented Dec 31, 2015

  • Добавлял поддержку для телефонов. В текущей версии она есть, но не очень хорошая. Сделайте пожалуйста для Android и iOS чтобы открывался нативный селект(у вас почемуто только для iOS).
  • Сделать мультиселект так же выпадающим.(хотябы опцией).
  • Можно сделать опцию для того чтоб при клике на optgroup выбирались все опшены в мультиселекте.

@ange007
Copy link
Owner Author

ange007 commented Dec 31, 2015

У input number если задать атрибуту «step» десятичное число, то плагин отказывается работать.
По спецификации атрибут step может принимать любое положительное число с плавающей запятой.
Планируется ли поправить это в будущих версиях?

@ange007
Copy link
Owner Author

ange007 commented Dec 31, 2015

Прикольная фича конечно, красивая, удобная, но есть проблема, не понимает объект в value
например

<select>
<option value='{"with": 100,"without": 200, "d": 18, "dr": 16}'> 18
</select>

такую конструкцию просто ломает, потом допилю сам

@pafnuty
Copy link

pafnuty commented Jan 13, 2016

  1. В выпадающем блоке добавьте обертывающий див для списка и выставляйте для него min-height. Это нужно для подключения плагина стилизации скролбара.

Стилизация скроллбара навешивается совершенно нормально и без этого, проверено.

@pafnuty
Copy link

pafnuty commented Apr 2, 2017

@pafnuty
Copy link

pafnuty commented Apr 14, 2017

@ange007
Copy link
Owner Author

ange007 commented Apr 14, 2017

Приветствую.
Это намёк на перенос? ))

Постараюсь сделать, я пока тут в пределах своего проекта делаю некоторые правки которые мне понадобились (в частности выношу в CSS скрытие элемента и прочее + вызов триггера onclick для check/radio box). А потом уже посмотрю что там можно по переносить.

@pafnuty
Copy link

pafnuty commented Apr 14, 2017 via email

@yunusga
Copy link

yunusga commented Apr 17, 2017

@yunusga
Copy link

yunusga commented Apr 18, 2017

@ange007
Copy link
Owner Author

ange007 commented Apr 23, 2017

Залил обновление.
Надеюсь я ничего сильно не поломал, так как поменял я там много чего.
Боюсь сложновато теперь будет переносить какие-то глобальные правки из оригинального плагина.

@pafnuty
Copy link

pafnuty commented May 2, 2017

@ange007 круто!
Ну идеи записывать никто же не мешает :)

@ange007
Copy link
Owner Author

ange007 commented May 3, 2017

@pafnuty - речь больше о правках непосредственно которые реализует автор оригинального плагина. Но просто та каша которая существовала (да и сейчас существует) - совсем меня не вдохновляет на поддержку. Очень много мусорного кода как по мне.

@pafnuty
Copy link

pafnuty commented May 3, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants