Skip to content

Commit

Permalink
FIX: Only show admin wrench when there are actions on mobile
Browse files Browse the repository at this point in the history
Corrects typo in logic from b725252
  • Loading branch information
davidtaylorhq committed Dec 31, 2019
1 parent a4a0d12 commit c7a74e5
Showing 1 changed file with 2 additions and 3 deletions.
Expand Up @@ -45,9 +45,8 @@ createWidget("topic-admin-menu-button", {

// We don't show the button when expanded on the right side on desktop
if (
(menu.attrs.actionButtons.length &&
!(attrs.rightSide && state.expanded)) ||
this.site.mobileView
menu.attrs.actionButtons.length &&
(!(attrs.rightSide && state.expanded) || this.site.mobileView)
) {
result.push(
this.attach("button", {
Expand Down

1 comment on commit c7a74e5

@discoursebot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/admin-wrench-appearing-for-regular-users-on-mobile/137404/8

Please sign in to comment.