Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lesson7 #7

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Lesson7 #7

wants to merge 3 commits into from

Conversation

alexcoach030
Copy link
Owner

Здравствуйте!
Добавил ряд фигур в тетрис (uFigure, yFigure), добавил счетчик задействованных фигур, а также выбор уровня сложности через соответствующие button (ускорение/замедление игры).
Над оформлением не особо думал, только над js.

В работе над корзиной фильтр реализовал через добавление к диву при выводе продуктов в каталог атрибута, соответствующего ключу type (как пример) из объекта, содержащего всю продукцию (line 23 в basket.js).
Далее реализовал баттоны для фильтрации (могут быть и чекбоксы), к баттонам делегируется обработчик через функцию filter (line 3 по line 16 в basket.js). Данная функция проверяет, какой от какого именно баттона отклик - если от "показать все" - возвращает функцию вывода каталога, если от других баттонов, содержащих дата-атрибут для фильтрации, то перебирает через forEach все элементы каталога,проверяет наличие такого же атрибута, как в нажатом баттоне, после чего удаляет дивы с продуктами, где нет такого атрибута. Так можно накладывать сразу множество фильтров одновременно с разными атрибутами.
Возникает вопрос, насколько это быстродейственный метод? И что можно улучшить/убрать лишнее? И еще один вопрос по такому способу - как в этом случае наиболее оптимально сделать откатить результаты фильтрации назад (допустим, пользователь снял галочку в чекбоксе)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant