Opened 17 months ago

Last modified 17 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, функция поиска просто перестанет работать, и, возможно, придется долго искать, почему...

Change History (1)

comment:1 by Denis_N, 17 months ago

Component: Разное и всякоеБД изделий АДС
Note: See TracTickets for help on using tickets.