From 19f2a73d5a6b6d0f45b803052815951d9c17c5af Mon Sep 17 00:00:00 2001
From: Jeff Daley <jeffdaley@users.noreply.github.com>
Date: Mon, 24 Jul 2023 12:41:01 -0400
Subject: [PATCH] Fix footer visibility conditions (#269)

---
 web/app/components/document/sidebar.hbs | 2 +-
 web/app/components/document/sidebar.ts  | 9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/web/app/components/document/sidebar.hbs b/web/app/components/document/sidebar.hbs
index 538c11ef7..168dda287 100644
--- a/web/app/components/document/sidebar.hbs
+++ b/web/app/components/document/sidebar.hbs
@@ -382,7 +382,7 @@
     </div>
   </div>
 
-  {{#if this.isOwner}}
+  {{#if this.footerIsShown}}
     <div class="sidebar-footer {{if this.editingIsDisabled 'locked'}}">
       {{#if this.editingIsDisabled}}
         <div class="px-3 -mb-1">
diff --git a/web/app/components/document/sidebar.ts b/web/app/components/document/sidebar.ts
index 16ebf4251..61e378034 100644
--- a/web/app/components/document/sidebar.ts
+++ b/web/app/components/document/sidebar.ts
@@ -381,6 +381,15 @@ export default class DocumentSidebarComponent extends Component<DocumentSidebarC
     }
   }
 
+  /**
+   * Whether the footer is shown.
+   * True for editors who may need to see the "doc is locked" message,
+   * as well as approvers and owners who need doc-management controls.
+   */
+  protected get footerIsShown() {
+    return this.isApprover || this.isOwner || this.isContributor;
+  }
+
   /**
    * Whether editing is enabled for basic metadata fields.
    * Used in the template to make some logic more readable.