Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#437 closed баг (fixed)

Не закрывается всплывающая ошибка в коррекции хода часов

Reported by: AlexLir Owned by: alx
Priority: средний Milestone: 2 очередь
Component: web-интерфейс (sw) Keywords:
Cc:

Description

При введении значения вне диапазона в поле коррекция хода часов, показывается всплывающее окно, которое не возможно закрыть. При нажатии "закрыть", показывается второе всплывающее окно поверх старого, при нажатии "закрыть" на втором окне, оно закрывается, но не закрывая старое.

Change History (4)

comment:1 by alx, 4 years ago

Ну, насчет "невозможно" - это, конечно, преувеличение. :) Нажатие ESC окно закрывает. Но кнопка "закрыть", действительно, не работает...

comment:2 by alx, 4 years ago

Resolution: fixed
Status: newclosed

In 1930/sw:

Исправлена ошибка: при попытке ввода недопустимого значения коррекции
хода часов реального времени сообщение об ошибке могло выдаваться дважды.
Теперь попытка ввода выполняется только если нет ни одного ваплывающего окна.
Closes #437.

comment:3 by alx, 4 years ago

Как оказалось, описанная проблема - следствие другой более общей (хоть и не такой серьезной) проблемы.

Запись нового значения коррекции хода в часы реального времени выполняется в двух случаях:

  • при нажатии Enter;
  • при переключении фокуса из поля ввода.

В описанном случае имело место следующее:

  • Пользователь вводит недопустимое значение в поле коррекции хода.
  • Пользователь нажимает Enter - скрипт проверяет валидность значения.
  • Скрипт обнаруживает, что введено недопустимое значение, и выдает диалог с сообщением. При этом фокус сотается в поле ввода значения**
  • Пользователь кликает кнопку "Закрыть" чтобы закрыть окно. При этом фокус из поля ввода переходит на кнопку - по этому событию скрипт снова пытается применить введенное значение, что приводит к появлению второго окна.

Сейчас устранено повторное появление окна, но то, что фокус сохраняется при выскакивании всплывающего окна, мне тоже кажется неправильным.

comment:4 by alx, 4 years ago

Создан тикет #438.

Note: See TracTickets for help on using tickets.