hexlet-rus / runit

Online Editor Form Embedding
https://runit.hexlet.ru/
Other
34 stars 114 forks source link

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

Open vyacheslav-pv opened 1 month ago

vyacheslav-pv commented 1 month ago

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

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

bobrov-site commented 1 month ago

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

fey commented 1 month ago

image

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

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

Приветствую! 1 и 2 пункты - изначально так и было задумано, сделал по аналогии сервиса replit

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

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

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

vyacheslav-pv commented 1 month ago

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

fey commented 1 month ago

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

vyacheslav-pv commented 2 weeks ago

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

HelenOne commented 2 weeks ago

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

vyacheslav-pv commented 2 weeks ago

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

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