Hexlet / hexlet-correction

Typo Reporter
https://fixit.hexlet.io/
GNU Affero General Public License v3.0
41 stars 72 forks source link

Bug: наличие кнопки удаления пользователя из воркспейса не у админа #259

Open bazilval opened 2 months ago

bazilval commented 2 months ago

Summary

На странице пользователей каждая строка в шаблоне выполнена "аккордеоном", который по идее должен раскрываться при нажатии и открывать нам кнопку удаления пользователя. Однако механизма открытия этого аккордеона не представлено.

Тем не менее, кнопка в разметке присутствует всегда (независимо от роли пользователя) и просто добавив в обрамляющий её элемент класс "show", мы получим к ней доступ.

image

Reproduction steps

1. Быть добавленным в воркспейс в качестве пользователя, но при этом не быть админом
2. Через инструменты разработчика добавить класс "show" в элемент обрамляющий кнопку удаления

Expected result

Отсутствие у рядового пользователя кнопки удаления пользователей в разметке в принципе

Actual result

Наличие в разметке у рядового пользователя кнопки удаления пользователей

Browsers

No response

OS

No response

bazilval commented 2 months ago

Шаблон выглядит так и мне тут в принципе не очень понятна задумка с двумя кнопками удаления. Одна появляется в последнем столбце, но только если пользователь Админ Вторая находится в спрятанном аккордеоне, который и развернуть даже нет возможности, судя по разметке. И тем более URL, который прописан там в форме не актуальный и кнопка не сработает image

bazilval commented 2 months ago

Удаление всей части после комментария про collapsed part решило бы проблему, потому что этот фрагмент абсолютно бесполезен.

Также нужно будет переработать фрагмент со строкой с данными пользователя и убрать из него все моменты про аккордеон и коллапсирование

fey commented 2 months ago

Да, выглядит как какая-то наркомания.

niyatanya commented 1 week ago

Привет! Я бы попробовала поразбираться в несрочном режиме. Я на четвертом проекте, это будет первый опыт с опенсорс.