Improve tag visibility toggle logic (#4425) * Improve tag visibility toggle logic Cache selector once, and remove hardcoded animation duration. Takes 300 tags down from 3 seconds down to 2ms per toggle. * whitespace fix

6166348ec803e5a233f57bb9061955d380775ed9

DepFA <35278260+dfaker@users.noreply.github.com>

Signed

public/scripts/tags.js+9 -3 Viewed

 function onTagListHintClick() {
     $(this).toggleClass('selected');
 -    $(this).siblings('.tag:not(.actionable)').toggle(100);
 -    $(this).siblings('.innerActionable').toggleClass('hidden');
 +    const $tagSiblings = $(this).siblings('.tag:not(.actionable)');
++
 +    if ($(this).hasClass('selected')) {
 +        $tagSiblings.show();
 +    } else {
 +        $tagSiblings.hide();
 +    }
++
 +    $(this).siblings('.innerActionable').toggleClass('hidden');
     power_user.show_tag_filters = $(this).hasClass('selected');
     saveSettingsDebounced();
+-
     console.debug('show_tag_filters', power_user.show_tag_filters);
+}