Моя жена крутая. Решив вернуться s/в большой спорт/на работу/ после некоторого перерыва, она начала не с рассылок резюме, а с написания jQuery плагина: https://plugins.jquery.com/traceit...
Igor Sereda,
earlyadopter,
9000,
понура свинка,
je suis frf,
словарный запас кофемолки,
я та телка, все ок,
темноводное,
[email protected],
and
юдж
liked this
Фидик, покритикуй, пжлст.
- eugenio
Названия опций лучше бы единообразно камелкейсом.
- юдж
названия колбеков - onClick, onHide и onEndTrace
- юдж
файл с плагином переименовать в jquery.traceit.js
- юдж
В исходниках отбить отступы четырьмя пробелами на таб.
- юдж
Вообще прогнать бы через какой-нибудь форматтер, потому что первый же пулл-реквест будет состоять из одних изменений.
- юдж
спасибо, продолжай.
- eugenio
Ну да, прогнать через JSLint и, скажем, http://jsbeautifier.org/
- 9000
На картинке справа чекбоксы или радио? Вроде бы смысл имеет только одна опция за раз.
- 9000
Я пошел в issues, чего в фидике-то сидеть.
- юдж
@9000 ну, можно сравнить два хвоста.
- eugenio
Написал комментов, завотчил, будет возможность, после чистки кода еще загляну. Хороший и правильный подход, в целом.
- юдж