Opened 2 years ago
Last modified 2 years ago
#1201 assigned улучшение
Использовать is(':visible'), вместо .css("display", "свойство")
| Reported by: | Denis_N | Owned by: | Denis_N |
|---|---|---|---|
| Priority: | minor | Component: | БД изделий АДС |
| Keywords: | Cc: |
Description
Не совсем по теме тикета, но увидел написанный код и решил дать один совет. Часто в случаях, когда надо проверить видимость элемента, лучше (и нагляднее) писать так:
if ($(".serialSearch.window").is(':visible')) { ..... }
или
if ($(".serialSearch.window").is(':hidden')) { ..... }
Лучше, как минимум, тем, что нет привязки к конкретному значению свойства display. Если через пару месяцев кто-нибудь решит отображать этот элемент не как grid, а, например, как block, функция поиска просто перестанет работать, и, возможно, придется долго искать, почему...
Note:
See TracTickets
for help on using tickets.
![[MC-04 logo]](/mc-04/chrome/site/logo.png)