Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#160 closed улучшение (fixed)

Чекбокс "Показать пароль"

Reported by: alx Owned by: dimag
Priority: trivial Milestone: 2 очередь
Component: ПО MC04-Dispatcher. Пульт диспетчера/техника Keywords:
Cc: san

Description (last modified by dimag)

Для чего нужен чекбокс "Показать пароль" в диалоге аутентификации?
По-моему он не нужен, и его надо убрать...
Мне Александр сделал замечание по Вариной версии программы, что неплохобы видеть введённый пароль, поэтому я добавил этот переключатель.

Change History (9)

comment:1 by dimag, 8 years ago

Description: modified (diff)
Resolution: fixed
Status: newclosed

comment:2 by alx, 8 years ago

Resolution: fixed
Status: closedreopened

Дима, отвечая на тикет, добавляйте, пожалуйста, комментарий, а не исправляйте текст. А то получается, что "Мне Александр сделал замечание..." написано от моего имени.

Закрывая тикет, устанавливайте, пожалуйста, подходящее решение. В данном случае Вы ничего не исправили, а решение по тикету - fixed. А ведь там есть варианты и wontfix, и invalid...

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

comment:3 by san, 8 years ago

Честно говоря не помню чтобы я хотел чекбокс "Показать пароль". Возможно какое-то недопонимание.
Я согласен с Алексеем и считаю что этот чекбокс не нужен.

Version 0, edited 8 years ago by san (next)

comment:4 by dimag, 8 years ago

Resolution: fixed
Status: reopenedclosed

rev 227

comment:5 by alx, 8 years ago

Resolution: fixed
Status: closedreopened

??? Нет ревизии r227 в репозитории.

comment:6 by alx, 8 years ago

Resolution: fixed
Status: reopenedclosed

r227 появился, чекбокс пропал.

comment:7 by alx, 8 years ago

Priority: minortrivial
Resolution: fixed
Status: closedreopened

Чекбокс-то с экрана пропал, но в программе он зачем-то по-прежнему есть, просто сделан невидимым с помощью showPasswordCheckbox.setVisible(false);. Зачем он нужен, если он все равно не показывается на экране? Предлагаю совсем убрать его из класса CStartupDlg.

comment:8 by dimag, 8 years ago

Resolution: fixed
Status: reopenedclosed

Я закомментировал всё что связанно с видимостью введённого пароля в r229, если надо раскомментирую.

comment:9 by san, 7 years ago

Milestone: Текущее2 очередь

Milestone renamed

Note: See TracTickets for help on using tickets.