Opened 18 months ago
Last modified 18 months 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.