[-]Зачем зависимости включены в git-репозиторий? Следовало добавитьnode_modulesиbower_componentsв.gitignore.[-]Не следовало мешать bower и npm.[-]Выполнено на канвасе. Он плохо стилизуется, невозможно выделить текст, довольно тяжеловесен в сравнении с простым div и более тормозной.[+]Удобное для юзера API (опции).[-]Вместо класса Gauge использован синглтон.[-]Небольшие неточности в английском: numbersInside → numbersAreInside, а лучше inset.[-]Небольшие расхождения в названиях: smallMarksStep, но smallMarkRadius[-]Жуткий paper.full в зависимостях (300kb) - для одного компонента это слишком.[-]В package.json мало информации. Следовало использоватьnpm init.
Last active
August 29, 2015 14:13
-
-
Save dy/4a69fd9ae9b1b493489a to your computer and use it in GitHub Desktop.
Feedback to https://github.com/andrejko/gauge.js
@andrejko конечно, вот component-gauge, с резиновой версткой, превосходным API, гибкой настройкой стилей и без всего лишнего. Был сделан в течение одного выходного дня, то бишь сегодня.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Можно посомтреть ваш пример реализации подобного функционала?