Skip to content

Commit

Permalink
topic_popover: Remove additional separator on spectator mode.
Browse files Browse the repository at this point in the history
In spectator mode, the separators encapsulating the topic visibility
switcher wasn't being hidden since the `hidden-for-spectators` class
was being applied only to the `popover-menu-inner-list-item` element.

As a follow-up commit to c3e4358, this removes the additional
separator by shifting the `hidden-for-spectators` class to the
`popover-menu-outer-list-item` element instead, which is responsible
for showing the separators.
  • Loading branch information
sayamsamal authored and timabbott committed Apr 27, 2024
1 parent 88e91fa commit 7cc7be5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions web/templates/topic_sidebar_actions.hbs
Expand Up @@ -8,9 +8,9 @@
</li>
</ul>
</li>
<li class="popover-menu-outer-list-item">
<li class="popover-menu-outer-list-item hidden-for-spectators">
<ul class="popover-menu-inner-list">
<li class="popover-menu-inner-list-item hidden-for-spectators">
<li class="popover-menu-inner-list-item">
<div class="tabs-container">
<div class="tab-option tippy-zulip-tooltip {{#if (eq visibility_policy all_visibility_policies.MUTED)}}selected-tab{{/if}}" data-visibility-policy="{{all_visibility_policies.MUTED}}" data-tippy-content="{{t 'Mute' }}" aria-label="{{t 'Mute' }}">
<i class="zulip-icon zulip-icon-mute-new"></i>
Expand Down

0 comments on commit 7cc7be5

Please sign in to comment.