![[MC-04 logo]](/mc-04/chrome/site/logo.png){kind=logo}
Opened 3 years ago
Closed 16 hours ago
#1191 closed улучшение (готово)
Скрывать панель поиска при выполнении поиска
| Reported by: | alx | Owned by: | Denis_N |
|---|---|---|---|
| Priority: | minor | Component: | БД изделий АДС |
| Keywords: | Cc: |
Description
В r275/base в левом верхнем углу интерфейса "Несоответствия" есть надпись "Поиск по S/N (CTRL + ?)". При ее клике появляется панель со строкой ввода серийного номера. После ввода серийного номера и нажатия ENTER открывается новая страница, однако на текущей странице панель ввода серийного номера по-прежнему продолжает отображаться.
Мне кажется очень маловероятным, что кто-то будет использовать эту панель чтобы открыть несколько главных страниц с поиском разных изделий по серийному номеру. Чаще всего пользователю требуется найти одно конкретное изделие, после чего работать дальше. Поэтому предлагаю при открытии новой страницы нажатием ENTER в панели поиска автоматически скрывать эту панель. Если пользователю все-таки требуется повторить поиск, он может снова открыть панель нажатием Ctrl-?.
Change History (7)
comment:1 by , 3 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
follow-up: 4 comment:2 by , 3 years ago
Не совсем по теме тикета, но увидел написанный код и решил дать один совет. Часто в случаях, когда надо проверить видимость элемента, лучше (и нагляднее) писать так:
if ($(".serialSearch.window").is(':visible')) { ..... }
или
if ($(".serialSearch.window").is(':hidden')) { ..... }
Лучше, как минимум, тем, что нет привязки к конкретному значению свойства display. Если через пару месяцев кто-нибудь решит отображать этот элемент не как grid, а, например, как block, функция поиска просто перестанет работать, и, возможно, придется долго искать, почему...
comment:3 by , 3 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Проверка в r295/base показала что при открытии результата поиска в новой вкладке в старой вкладке панель поиска по прежнему отображается
comment:4 by , 3 years ago
Replying to alx:
Не совсем по теме тикета, но увидел написанный код и решил дать один совет. Часто в случаях, когда надо проверить видимость элемента, лучше (и нагляднее) писать так:
if ($(".serialSearch.window").is(':visible')) { ..... }или
if ($(".serialSearch.window").is(':hidden')) { ..... }Лучше, как минимум, тем, что нет привязки к конкретному значению свойства
display. Если через пару месяцев кто-нибудь решит отображать этот элемент не какgrid, а, например, какblock, функция поиска просто перестанет работать, и, возможно, придется долго искать, почему...
Да, ты прав, Алексей. Создал для себя тикет #1201
comment:6 by , 3 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Проверка в r313/base показала что при открытии результата поиска в новой вкладке в старой вкладке панель поиска по прежнему отображается
comment:7 by , 16 hours ago
| Resolution: | → готово |
|---|---|
| Status: | reopened → closed |
Проблема относилась к старой всплывающей панели поиска S/N.
По тикету уже были сделаны исправления в r293/base и r302/base, но в r313/base проблема ещё воспроизводилась при открытии результата в новой вкладке.
Позже в r488/base старый всплывающий поиск был заменён новым постоянным виджетом в шапке: поле S/N, кнопка перехода и настройка открытия в новой/текущей вкладке. В текущей версии панели, которую требовалось скрывать после поиска, уже нет, поэтому описанное поведение больше не воспроизводится.
Закрываю как исправленное в текущем интерфейсе.
In 293/base: