[-]Зачем зависимости включены в 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@andrejko конечно, вот component-gauge, с резиновой версткой, превосходным API, гибкой настройкой стилей и без всего лишнего. Был сделан в течение одного выходного дня, то бишь сегодня.