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

[#456] Feat: delete one or more snippets #507

Merged
merged 31 commits into from
Nov 12, 2024

Conversation

vyacheslav-pv
Copy link
Contributor

[#456] Реализован функционал удаления нескольких сниппетов путем отметки чекбоксов.

Демо https://vyacheslav-pv-runit.onrender.com

@bobrov-site
Copy link
Contributor

Работает кайфово. Класс!

@fey
Copy link
Contributor

fey commented Jun 17, 2024

image

  1. После снятия галочки с чекбокса, он не возвращается назад на иконку языка.
  2. Почему меняется именно иконка языка?)
  3. Давайте сделаем так - сверху добавляем бар для кнопок упражения (под именем юзера), там будет режим "выбрать" который вклчюает отображение чекбоксов. Логика похожа на то, как мы выбираем фото на телефоне. Ну и иконку языка не трогаем. Лучше заменить карандаш, ведь мы все равно находимся в "режиме выбора" и он нам ни к чему.

@vyacheslav-pv
Copy link
Contributor Author

  1. После снятия галочки с чекбокса, он не возвращается назад на иконку языка.
  2. Почему меняется именно иконка языка?)
  3. Давайте сделаем так - сверху добавляем бар для кнопок упражения (под именем юзера), там будет режим "выбрать" который вклчюает отображение чекбоксов. Логика похожа на то, как мы выбираем фото на телефоне. Ну и иконку языка не трогаем. Лучше заменить карандаш, ведь мы все равно находимся в "режиме выбора" и он нам ни к чему.

Приветствую!
1 и 2 пункты - изначально так и было задумано, сделал по аналогии сервиса replit
3. Давайте попробую.
Хотелось бы уточнение, в баре будет только пункт "выбрать", и после отметки чекбоксов появляется плашка "удалить" (как сейчас реализовано). Или делаем в баре еще пункт удалить, который изначально неактивный, активируется после отметки чекбоксов, пример выпадающее меню
Скриншот 18-06-2024 154326

@fey
Copy link
Contributor

fey commented Jun 18, 2024

Не совсем понял, можете пример показать, как это может быть выглядеть?

Идея в том, что вы можете выбрать несколько сниппетов и что-то с ними сделать, например удалить, переместить в "папку" и тд. Но пока только удаление.

@vyacheslav-pv
Copy link
Contributor Author

@fey думаю, как лучше.
Под именем юзера кнопка: выбрать сниппеты и кнопку удалить, пример
Скриншот 18-06-2024 230752
Но наверное будет лучше, если после нажатия кнопки выбрать и отметки одного или нескольких чекбоксов - появляется кнопки с действиями (удалить, и т.д.)

@fey
Copy link
Contributor

fey commented Jun 20, 2024

@vyacheslav-pv смотрите, кнопка выбора отвечает олько за переход в режим селекта. Т.е. на сниппетах появляется чекбокс
Дальше вы выбираете сниппеты и что-то с ними делаете, уже отдельные кнопки отвечают за отправку, удаление и тд. Лучше делать в виде кнопок, тк это же действия (просто вижу, что у вас это ссылки). Под именем юзера норм

@vyacheslav-pv
Copy link
Contributor Author

Доработал функционал, демо https://vyacheslav-pv-runit.onrender.com
Появилась возможность с помощью кнопки отображать/скрывать чекбоксы.
А также доработан тулбар для выбранных сниппетов, он "прилипает" снизу блока сниппетов, удобно при скроллинге.
В тулбаре для отмеченных сниппетов доступны кнопка "удалить" и добавлена кнопка "отмена".
Скриншот 05-07-2024 165735
Скриншот 05-07-2024 165847

@HelenOne
Copy link
Contributor

@vyacheslav-pv это итоговая версия? Можно проверять?

@vyacheslav-pv
Copy link
Contributor Author

@vyacheslav-pv это итоговая версия? Можно проверять?

@HelenOne Привет. да, посмотрите пожалуйста.

@fey fey requested a review from dzencot July 19, 2024 13:21
frontend/src/components/Modals/DeleteSnippetModal.jsx Outdated Show resolved Hide resolved
frontend/src/slices/snippetsSlice.js Outdated Show resolved Hide resolved
@dzencot
Copy link
Collaborator

dzencot commented Oct 18, 2024

Чет пр завис(

@vyacheslav-pv
Copy link
Contributor Author

Чет пр завис(

Привет. на днях отправлю pr

@dzencot dzencot marked this pull request as draft October 30, 2024 09:47
@vyacheslav-pv vyacheslav-pv marked this pull request as ready for review November 5, 2024 04:07
@fey fey requested a review from dzencot November 5, 2024 09:08
@dzencot dzencot merged commit 1761223 into hexlet-rus:main Nov 12, 2024
2 checks passed
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.

5 participants