JQueryFormStyler-Modern icon indicating copy to clipboard operation
JQueryFormStyler-Modern copied to clipboard

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

Open ange007 opened this issue 10 years ago • 15 comments

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

ange007 avatar Dec 31 '15 17:12 ange007

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

ange007 avatar Dec 31 '15 17:12 ange007

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

ange007 avatar Dec 31 '15 17:12 ange007

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

ange007 avatar Dec 31 '15 17:12 ange007

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

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

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

ange007 avatar Dec 31 '15 17:12 ange007

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

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

pafnuty avatar Jan 13 '16 19:01 pafnuty

https://github.com/Dimox/jQueryFormStyler/pull/127

pafnuty avatar Apr 02 '17 12:04 pafnuty

https://github.com/Dimox/jQueryFormStyler/pull/128

pafnuty avatar Apr 14 '17 10:04 pafnuty

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

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

ange007 avatar Apr 14 '17 13:04 ange007

Нет, не намёк. Переношу сюда интересные и нужные моменты, что бы при возможности можно было заняться этим вопросом. Ну а кто заниматься будет - не столь важно :smile:

14 апр 2017 г. 5:25 PM пользователь "Vladimir B." [email protected] написал:

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

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

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/ange007/JQueryFormStyler-Modern/issues/13#issuecomment-294153055, or mute the thread https://github.com/notifications/unsubscribe-auth/ABj1XznWPvULWK474l6lLRwAcwm27Qj7ks5rv3O8gaJpZM4G9RM6 .

pafnuty avatar Apr 14 '17 14:04 pafnuty

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

ange007 avatar Apr 23 '17 21:04 ange007

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

pafnuty avatar May 02 '17 16:05 pafnuty

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

ange007 avatar May 03 '17 09:05 ange007

Это да.

3 мая 2017 г. 1:22 PM пользователь "Vladimir B." [email protected] написал:

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

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ange007/JQueryFormStyler-Modern/issues/13#issuecomment-298861787, or mute the thread https://github.com/notifications/unsubscribe-auth/ABj1XyOHz6v_3-01bg-aJT56w1ruN_N3ks5r2Ec7gaJpZM4G9RM6 .

pafnuty avatar May 03 '17 12:05 pafnuty