VE-12. Не интуитивные кнопки в диалоге настроек Sip-пользователя

[san]

При создании или редактировании Sip-пользователя диалог имеет две кнопки Ок/Отмена.
При нажатии Ок я интуитивно ожидаю что окно диалога будет закрыто после передачи настроек, однако этого не происходит.
Предлагаю или переименовать кнопки в Применить/Закрыть, что будет точнее отражать их назначение, или закрывать диалог по нажатию Ок.

[alx]

Replying to san:

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

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

[san]

Хм, открываю окно настроек кнопкой шестерня или Добавить ввожу данные, нажимаю Ок
Пользователь появляется или изменяются его настройки но диалог не закрывается

[alx]

Replying to san:

Хм, открываю окно настроек кнопкой шестерня или Добавить ввожу данные, нажимаю Ок

Это не подробности. :) Какие именно данные вводишь?

Диалог закрывается только в том случае, если введены корректные данные (в противном случае должно появиться сообщение об ошибке, а поле, данные в котором содержат ошибку, должно обвестись красной рамкой).

Уточни, пожалуйста, что конкретно ты ввел в каждое из полей диалога.

[san]

Хм, кажется проблема серъёзнее.
Это не окно не закрывается, а настройки не изменяются... сейчас по шагам распишу...

[san]

1. Я обновляю страничку и захожу на вкладку Sip-пользователи
2. Там в таблице присутствует один пользователь alx
3. Нажимаю Добавить и в диалоге ввожу в поля значения 123, 123, 1, 1 (см скриншот ниже)
4. Нажимаю Ок - пользователь появляется в таблице в виде неоранжевой строки, диалог остаётся открытым.
5. Нажимаю Ок ещё раз - ошибка: Пользователь с таким именем уже существует
6. Закрываю диалог кнопкой Отмена затем Применяю и сохраняю конфиг
7. Обновляю страницу - в таблице по прежнему только alx

[alx]

Replying to san:

4. Нажимаю Ок - пользователь появляется в таблице в виде неоранжевой строки, диалог остаётся открытым.

Как странно... Если бы была обнаружена ошибка в данных, новая строчка не должна была бы появиться. Но если нет ошибок, диалог должен был закрыться!!! Парадокс... :( Да еще и докучи строчка почему-то не оранжевая...

5. Нажимаю Ок ещё раз - ошибка: Пользователь с таким именем уже существует
6. Закрываю диалог кнопкой Отмена затем Применяю и сохраняю конфиг
7. Обновляю страницу - в таблице по прежнему только alx

Как странно... Вот фрагмент кода сохранения пользователя:

            addUserLine(name, user, true);
            popup.close();

Первая строка фрагмента выполняется, так как в таблице появляется новая строчка. Вторая строка фрагмента закрывает диалог! Ничего не понимаю... Каким браузером ты пользуешься?

[alx]

О, вдруг воспроизвелось... А ведь вроде бы делал все то же самое, что и раньше...
Оказалось, что браузер ругается на tr.insertBefore($(this)); в строке 532...

[alx]

Исправлено в r2335.