search

hexlet-rus / runit

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

Удалить кнопку(иконку) нереализованной фичи #482

Closed Yawerica closed 3 months ago

Yawerica commented 4 months ago

Убрать кнопку(иконку) нереализованной фичи.

Reproduction steps

precondition:
1. Быть авторизованным.

steps:
1. Зайти на главную страницу https://runit.hexlet.ru/u/zzz
2. Обратить внимание на кнопку(иконку) справа от Имени пользователя 
3. При нажатии на нее появляется окно "Эта функция пока недоступна"

Expected result

эта кнопка отсутствует.

Actual result

не реализовано

Browsers

Chrome

OS

Windows

fey commented 4 months ago

ага, это можно пока убрать

DimRux commented 4 months ago

Возьму в работу.

faciledictu commented 4 months ago

А может лучше реализовать фичу?)

fey commented 4 months ago

Тут не совсем понятно, что она означает. У нас сейчас нет публичных профилей. По идее нужно делать публичную страницу профиля, выводить туда инфу, давать ссылку на профиль, например в менюшке. Пока у нас есть другие траблы-баги, поэтому не сильно приоритетно :thinking:

faciledictu commented 4 months ago

@fey да, соглашусь с тобой. Сорри, мне кнопочка нравится, я ее рисовал ) Могу предложить @DimRux не удалять код, а вынести в компонент и не подключать этот компонент к странице профиля. Если, конечно, не хотите какую-то другую логику и она будет не нужна. Тогда удаляем смело. Если надо что-то нарисовать другое — обращайтесь

faciledictu commented 4 months ago

Вообще, сорри за занудство, но тыкать юзера, что он ввел лишние пробелы — это плохо. Типа "ну поправь пожалуйста". Ну пусть себе вводит хоть десять пробелов. Чем меньше мы мешаем юзеру сделать как он хочет, тем лучше. Тримить их и спокойно создавать. То же и с почтой. Если он потом будет входить с пробелами — нестрашно. Я придерживался такой логики, когда дорабатывал валидацию

fey commented 4 months ago

Нуу обиднее больше когда пароль вроде бы нормальный, но копируешь-вставляешь его с пробелами, а потом он не подходит. Обсудим с продукт овнером. Вообще это не самое критичное, ибо щас даже банально пароль восстановить низзя, пользователей у нас не выглядит критично. Можно создать отдельный ишшус, там дискуссию вести. Подобные штуки мониторятся через всякие трекеры, которые отслеживают поведение пользователя (например яндекс метрика).

faciledictu commented 4 months ago

Ой, я перепутал issue и не туда написал про пробелы. Но пароль с пробелами не должен принимать, и он не принимает. Я про имя пользователя — его можно тримить

bobrov-site commented 3 months ago

@Yawerica вроде бы уже нет этой кнопки