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

[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')) {
     .....
}

или

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

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

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

[Denis_N]

In 302/base:

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

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

close #1191

[alx]

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