Вместо обновления текущей страницы открывается новая

[alx]

В r275/base в интерфейсе "Несоответствия" выполняю следующие действия:

• Кликаю "Поиск по S/N".
• В появившейся панели ввожу серийный номер, например "F01187".
• Нажимаю ENTER - открывается новая страница с единственным изделием в таблице.
• Закрываю открывшееся окно и возвращаюсь на страницу "Несоответствия".
• В строке поиска "Комментарий" ввожу C30 и нажимаю ENTER.
• Открывается новая страница, как и во втором пункте.

Ожидал, что новая страница не откроется, а будет обновлена текущая в соответствии с введенной поисковой строкой.

[Denis_N]

In 289/base:

Исправлен баг: Вместо обновления текущей страницы открывается новая

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

close #1189

[alx]

Replying to Denis_N:

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

Хм... Или я не понял написанное здесь, или, возможно, я недостаточно явно изложил суть проблемы в описании тикета. На всякий случай уточню.

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

[Denis_N]

Теперь понял, что я не так понял =)

С любого интерфейса поиск по S/N переводит на главную. Думаю, если такой элемент будет отличаться, то это будет не интуитивным для пользователя. На правах разработчика откажусь от реализации этой идеи

[san]

Кажется вы всё-равно друг друга не поняли)
Алексей имел ввиду(насколько я понял) что после использования функции "Поиск по S/n"(с переходом на главную), на странице Несоответствий, если ввести что угодно в один из фильтров и нажать Enter, то вместо ожидаемого применения фильтра, снова сработает "Поиск по S/n".

[san]

Подозреваю что после выполнения #1191 негативный эффект описанный в тикете должен исчезнуть, т.к. панель "Поиск по S/n" будет закрыта и второй раз Enter туда не уйдёт, но это только подозрения))

[san]

Ожидал, что новая страница не откроется, а будет обновлена текущая в соответствии с введенной поисковой строкой.

Кстати, Алексей, а зачем ты нажимаешь Enter? В интерфейсе Несоответствие обновление данных Денис сделал "автоматическим", после ввода символа он выжидает некоторый таймаут и начинает обновлять данные.

[san]

А я понял, снимаю последний вопрос.
Оказывается можно нажатием Enter "ускорить" обновление данных.

[san]

Тогда дефект имеет место: после нажатия Enter в панели "Поиск по S/n", нажатия Enter в полях фильтров приводят не к обновлению данных, а к очередному "Поиску по S/n"
r295/base

[Denis_N]

И правда. Я невнимательно прочитал тикет. Прошу прощения. Сейчас поправлю

[Denis_N]

In 298/base:

Исправлен баг: вместо обновления текущей страницы открывается новая

Привязал событие, которое отвечает за переход на главную к плашке поиска S/N. Раньше был привязан к документу

close #1189

--стыд, что не знал, что так можно--

[alx]

Replying to san:

Алексей имел ввиду(насколько я понял) что после использования функции "Поиск по S/n"(с переходом на главную), на странице Несоответствий, если ввести что угодно в один из фильтров и нажать Enter, то вместо ожидаемого применения фильтра, снова сработает "Поиск по S/n".

Да, именно так, спасибо за помощь. Мне казалось, я именно это и написал в comment:2...

По-моему мы вместе с Denis_N уже разобрались, в чем была причина (обработчик нажатия ENTER, который активировал поиск по s/n, подключался не к панели поиска, а ко всему документу, поэтому он срабатывал не только на ENTER, нажатый в панели поиска, но и на любой другой ENTER в любом месте документа).

[alx]

Replying to san:

Кстати, Алексей, а зачем ты нажимаешь Enter?

Чтобы получить результат поиска по новым заданным мной условиям.

В интерфейсе Несоответствие обновление данных Денис сделал "автоматическим", после ввода символа он выжидает некоторый таймаут и начинает обновлять данные.

В таком случае, он его сделал неправильно (или, как минимум, не совсем правильно). Это автоматическое обновление работает только при вводе значения набором его на клавиатуре. А когда, например, вводишь значение путем вставки из буфера обмена - не работает, пока ENTER не нажмешь. Подозреваю (код не смотрел), это потому что Denis_N использует для этой функции события клавиатуры (keydown, keypress, keyup), а не событие change, эмитируемое при любом изменении значения строки ввода...

А я, кстати, наверное, чаще вставляю значения именно из буфера обмена, чем набираю руками...

[Denis_N]

Replying to alx:

В таком случае, он его сделал неправильно (или, как минимум, не совсем правильно). Это автоматическое обновление работает только при вводе значения набором его на клавиатуре. А когда, например, вводишь значение путем вставки из буфера обмена - не работает, пока ENTER не нажмешь. Подозреваю (код не смотрел), это потому что Denis_N использует для этой функции события клавиатуры (keydown, keypress, keyup), а не событие change, эмитируемое при любом изменении значения строки ввода...

Событие change не поможет. Используя change, пришлось бы нажимать, либо кликать вне элемента, к которому привязано событие change, чтобы событие сработало

Да, на текущий момент используется keyup, поскольку это срабатывает со вставкой через ctrl + v. И я правильно понимаю, что идет речь про вставку с помощью мыши?

[alx]

Replying to Denis_N:

Событие change не поможет. Используя change, пришлось бы нажимать, либо кликать вне элемента, к которому привязано событие change, чтобы событие сработало

Ты прав. Здесь больше подошло бы событие input, а не change.

И я правильно понимаю, что идет речь про вставку с помощью мыши?

Да.