Скрывать панель поиска при выполнении поиска

[alx]

В r275/base в левом верхнем углу интерфейса "Несоответствия" есть надпись "Поиск по S/N (CTRL + ?)". При ее клике появляется панель со строкой ввода серийного номера. После ввода серийного номера и нажатия ENTER открывается новая страница, однако на текущей странице панель ввода серийного номера по-прежнему продолжает отображаться.

Мне кажется очень маловероятным, что кто-то будет использовать эту панель чтобы открыть несколько главных страниц с поиском разных изделий по серийному номеру. Чаще всего пользователю требуется найти одно конкретное изделие, после чего работать дальше. Поэтому предлагаю при открытии новой страницы нажатием ENTER в панели поиска автоматически скрывать эту панель. Если пользователю все-таки требуется повторить поиск, он может снова открыть панель нажатием Ctrl-?.

[Denis_N]

In 293/base:

Улучшено: Скрывать панель поиска при выполнении поиска

close #1191

[alx]

Не совсем по теме тикета, но увидел написанный код и решил дать один совет. Часто в случаях, когда надо проверить видимость элемента, лучше (и нагляднее) писать так:

if ($(".serialSearch.window").is(':visible')) {
     .....
}

или

if ($(".serialSearch.window").is(':hidden')) {
     .....
}

Лучше, как минимум, тем, что нет привязки к конкретному значению свойства display. Если через пару месяцев кто-нибудь решит отображать этот элемент не как grid, а, например, как block, функция поиска просто перестанет работать, и, возможно, придется долго искать, почему...

[san]

Проверка в r295/base показала что при открытии результата поиска в новой вкладке в старой вкладке панель поиска по прежнему отображается

[Denis_N]

Replying to alx:

Не совсем по теме тикета, но увидел написанный код и решил дать один совет. Часто в случаях, когда надо проверить видимость элемента, лучше (и нагляднее) писать так:

if ($(".serialSearch.window").is(':visible')) {
     .....
}

Да, ты прав, Алексей. Создал для себя тикет #1201
или

if ($(".serialSearch.window").is(':hidden')) {
     .....
}

Лучше, как минимум, тем, что нет привязки к конкретному значению свойства display. Если через пару месяцев кто-нибудь решит отображать этот элемент не как grid, а, например, как block, функция поиска просто перестанет работать, и, возможно, придется долго искать, почему...

[Denis_N]

In 302/base:

Исправлен баг: Скрывать панель поиска при выполнении поиска

Добавил скрытие панели с поиском серийного номера по нажатию на кнопку

close #1191

[alx]

Проверка в r313/base показала что при открытии результата поиска в новой вкладке в старой вкладке панель поиска по прежнему отображается