Opened 17 months ago

Closed 17 months ago

Last modified 17 months ago

#1189 closed дефект (fixed)

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

Reported by: alx Owned by: Denis_N
Priority: minor Component: БД изделий АДС
Keywords: Cc:

Description

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

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

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

Change History (16)

comment:1 by Denis_N, 17 months ago

Resolution: fixed
Status: newclosed

In 289/base:

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

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

close #1189

in reply to:  1 comment:2 by alx, 17 months ago

Replying to Denis_N:

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

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

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

Last edited 17 months ago by alx (previous) (diff)

comment:3 by Denis_N, 17 months ago

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

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

comment:4 by Denis_N, 17 months ago

Resolution: fixed
Status: closedreopened
Type: дефектулучшение

comment:5 by Denis_N, 17 months ago

Resolution: не будем делать
Status: reopenedclosed

comment:6 by san, 17 months ago

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

comment:7 by san, 17 months ago

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

Last edited 17 months ago by san (previous) (diff)

comment:8 by san, 17 months ago

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

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

comment:9 by san, 17 months ago

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

comment:10 by san, 17 months ago

Resolution: не будем делать
Status: closedreopened
Type: улучшениедефект

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

comment:11 by Denis_N, 17 months ago

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

comment:12 by Denis_N, 17 months ago

Resolution: fixed
Status: reopenedclosed

In 298/base:

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

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

close #1189

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

in reply to:  6 comment:13 by alx, 17 months ago

Replying to san:

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

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

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

in reply to:  8 ; comment:14 by alx, 17 months ago

Replying to san:

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

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

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

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

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

Last edited 17 months ago by alx (previous) (diff)

in reply to:  14 ; comment:15 by Denis_N, 17 months ago

Replying to alx:

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

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

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

Last edited 17 months ago by Denis_N (previous) (diff)

in reply to:  15 comment:16 by alx, 17 months ago

Replying to Denis_N:

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

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

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

Да.

Note: See TracTickets for help on using tickets.