akhmadbabaev / o-slider Goto Github PK
View Code? Open in Web Editor NEWIt's another ordinary slider, no better than other.
License: MIT License
It's another ordinary slider, no better than other.
License: MIT License
у тебя щас в _basic.scss используются стили по тегам, по БЕМу можно пользоваться только классами, нельзя использовать id
и для чего ты используешь нижнее подчеркивание) имя файлам должно соответствовать имени папке в которой они лежат
If setState gets { value: 5 } and than { value: 5, min: 10 } he has to throw a mistake.
В стандартах есть принцип Single Responsibility, также методы не должны быть сильно большими.
В стандартах есть принцип Single Responsibility, также методы не должны быть сильно большими. пункт 4 https://github.com/fullstack-development/front-end-best-practices/blob/master/JS/goodPractice.md)
Разбей эти 2 метода handleThumbMouseDown и handleThumbTouchStart
Было бы неплохо добавить хотя-бы крайние значения слайдера в виде шкалы. Чтобы можно было ориентироваться непосредственно по слайдеру от какого и до какого значения идёт рэнж, т.к. панель опций, по факту, не является частью плагина. (скрин для примера)
Раньше в задании был критерий по полностью функциональной шкале, которая отрисовывается в зависимости от шага и по значениям можно кликать, чтобы тригерить слайдер. Если добавишь такой функционал - будет только плюсом, но это не обязательно :)
зачем ты подключаешь jquery в o-slaider.ts ты можешь его подключить с помощью вебпака. Еще ты подключаешь стили к странице через link и скрипты через script, webpack может сам их подключать)
The initialization of the components is called if even isEnabled was set as false.
страница должна лежать в отдельной папке
По стандарту все классы, которые используются для поиска по DOM, должны иметь префикс js-.
1 пункт. Хоть правило и написано в JQuery, но предполагается и для использования с чистым js.
$firstElement[0] для выбора индексов используй метод eq()
Values combination like { step: 0, min: 0, max: 0 } do the slider inapplicable.
Почему очень многие методы и поля обозначены как protected, а не как private, хотя используются только?
Использовать type assertion любого вида - крайне не тайп сейвово (в данном случае as).
Тут же вполне есть вероятность, что элемента с таким классом не существует и тогда всё посыпется.
Необходимо всегда использовать как минимум один запасной шрифт и одно запасное семейство, сейчас это не учтено. пункт 6
If set slider parameter from or other numeric parameters as a string that contains a number ( for example { from: '20' } I get error 'from is Infinity' but he isn't.
У тебя порядок в checkbox.scss нарушен селектор input идет перед label, нужно чтобы селекторы были в том порядке как указываются в pug
Нужно откорректировать разделение зависимостей на devDependencies и dependencies: в devDependencies ты должен оставить только те зависимости, которые тебе нужны исключительно в режиме разработки, и при отсутствии которых можно все равно без проблем собрать проект
Any click on 'to' button set focus to 'from' button.
$(selector).oSlider({ from: 50 });
throw error.
If setState gets { value: 0, min: 10 } he doesn't throw a mistake because he handles 0 as false.
По-хорошему, у меня не должно быть возможности выставлять слайдерам одинаковые значения и перетаскивать один за другой. Обычно в слайдерах(когда есть диапазон) мы выбираем значения между минимальным и максимальным и должна быть привязка одного бегунка как минимального значения, а второго как максимального.
It happens because throttling of mouse notification makes possible sending the repeated values and setState delete the value from changes properties as duplicated.
Не использовать id, кроме случаев, когда они семантически востребованы;
Когда гарантируется их уникальность например на уровне базы данных и когда вам нужно
требуется рабочая навигация в рамках одной страницы (например, как на Википедии или на этом сайте сборника best-practices;
требуется связать и , которые находятся в совершенно разных ветках дерева DOM (вообще, лучше всегда просто внутрь ставить и тогда никакие айдишники не нужны);
В плагине то че не используешь)
A declarative, efficient, and flexible JavaScript library for building user interfaces.
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
An Open Source Machine Learning Framework for Everyone
The Web framework for perfectionists with deadlines.
A PHP framework for web artisans
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
Some thing interesting about web. New door for the world.
A server is a program made to process requests and deliver data to clients.
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
Some thing interesting about visualization, use data art
Some thing interesting about game, make everyone happy.
We are working to build community through open source technology. NB: members must have two-factor auth.
Open source projects and samples from Microsoft.
Google ❤️ Open Source for everyone.
Alibaba Open Source for everyone
Data-Driven Documents codes.
China tencent open source team.