search

ange007 / JQueryFormStyler-Modern

JQuery HTML form styling plugin.
http://ange007.github.io/JQueryFormStyler-Modern/
Other
18 stars 4 forks source link

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

Open ange007 opened 8 years ago

ange007 commented 8 years ago

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

ange007 commented 8 years ago
ange007 commented 8 years ago
ange007 commented 8 years ago

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

ange007 commented 8 years ago

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

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

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

pafnuty commented 8 years ago

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

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

pafnuty commented 7 years ago

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

pafnuty commented 7 years ago

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

ange007 commented 7 years ago

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

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

pafnuty commented 7 years ago

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

14 апр 2017 г. 5:25 PM пользователь "Vladimir B." notifications@github.com написал:

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

Постараюсь сделать, я пока тут в пределах своего проекта делаю некоторые правки которые мне понадобились (в частности выношу в 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 .

yunusga commented 7 years ago

Dimox/jQueryFormStyler#131

yunusga commented 7 years ago
ange007 commented 7 years ago

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

pafnuty commented 7 years ago

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

ange007 commented 7 years ago

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

pafnuty commented 7 years ago

Это да.

3 мая 2017 г. 1:22 PM пользователь "Vladimir B." notifications@github.com написал:

@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 .