From f88e85e4f30302a37b399d39d260eeaf2c15ada3 Mon Sep 17 00:00:00 2001
From: Michael Mayer <michael@photoprism.app>
Date: Thu, 14 May 2026 17:34:38 +0000
Subject: [PATCH] Frontend: Mount face-marker overlay in read-only display mode
 for viewers

Drops shouldShowEditButton() from the <p-face-marker-overlay> v-if so
non-editable users (Plus / Pro viewer role, anyone reaching the eye
toggle in the sidebar) actually see marker rectangles AND the
in-overlay Back button when they activate display mode. The previous
gate required edit permission, so viewers entered face-marker mode
(chrome hidden), saw nothing, and had no way to exit. Edit gestures
stay gated separately by the overlay's isEditMode flag and the
lightbox-side onCreateFaceMarker / onRemoveFaceMarker /
toggleFaceMarkerEdit handlers that re-check shouldShowEditButton().
Pre-existing bug; surfaced while testing the recent ACL-driven
sidebar refactor (#4966).
---
 frontend/src/component/lightbox.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/component/lightbox.vue b/frontend/src/component/lightbox.vue
index 16096e252..e89908161 100644
--- a/frontend/src/component/lightbox.vue
+++ b/frontend/src/component/lightbox.vue
@@ -44,7 +44,7 @@
       >
         <div ref="lightbox" tabindex="-1" class="p-lightbox__pswp no-transition"></div>
         <p-face-marker-overlay
-          v-if="shouldShowEditButton() && featPeople && faceMarkers.active && pswp()"
+          v-if="featPeople && faceMarkers.active && pswp()"
           ref="faceMarkerOverlay"
           :mode="faceMarkers.mode"
           :markers="markers"