From 3bb4f1a1a894a6bbc4b1610c3585b73e38f1753d Mon Sep 17 00:00:00 2001
From: Lutthy <22339795+lbossuyt@users.noreply.github.com>
Date: Sun, 9 Aug 2026 11:16:18 +0200
Subject: [PATCH] fix(branding): hide "Powered by PicPeak" on every page, not
only the gallery (#999)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
branding_hide_powered_by only hid the attribution on the main gallery footer. It
stayed visible on the gallery password screen, client access page, Premium
layout, admin and customer login, accept-invite and CMS pages — AdminLoginPage
rendered it unconditionally with no guard at all, so the setting genuinely did
not apply there.
Routes those surfaces through one component in components/common
that reads the public setting itself (the DynamicFavicon pattern) and renders
nothing when white-labeling is on, including while the settings are still
loading so a white-labelled instance never flashes the attribution.
Also collapses three duplicate translation keys (gallery.poweredBy,
adminLogin.poweredBy, customer.login.poweredBy) into a single common.poweredBy,
and translates pages that had 'Powered by' hardcoded in English across all 8
locales.
Fork-PR workflows were never approved so CI did not run. Verified locally
against cf243b44: tsc --noEmit clean, ESLint clean, vitest 124 passed across 24
files, and npm run build succeeds.
GalleryLayout.tsx keeps its own inline guard and is not routed through the new
component; tracked separately.
Co-authored-by: lbossuyt
---
.../src/components/common/CMSContentBlock.tsx | 7 +--
frontend/src/components/common/PoweredBy.tsx | 25 ++++++++
.../common/__tests__/PoweredBy.test.tsx | 63 +++++++++++++++++++
frontend/src/components/common/index.ts | 1 +
.../gallery/layouts/GalleryPremiumLayout.tsx | 4 +-
frontend/src/i18n/locales/de.json | 10 ++-
frontend/src/i18n/locales/en.json | 8 +--
frontend/src/i18n/locales/es.json | 4 +-
frontend/src/i18n/locales/fr.json | 8 +--
frontend/src/i18n/locales/nl.json | 10 ++-
frontend/src/i18n/locales/pt.json | 10 ++-
frontend/src/i18n/locales/ru.json | 10 ++-
frontend/src/i18n/locales/sl.json | 8 +--
frontend/src/pages/ClientAccessPage.tsx | 6 +-
frontend/src/pages/GalleryPage.tsx | 10 +--
frontend/src/pages/admin/AdminLoginPage.tsx | 6 +-
.../src/pages/customer/CustomerLoginPage.tsx | 6 +-
.../src/pages/public/AcceptInvitePage.tsx | 6 +-
18 files changed, 131 insertions(+), 71 deletions(-)
create mode 100644 frontend/src/components/common/PoweredBy.tsx
create mode 100644 frontend/src/components/common/__tests__/PoweredBy.test.tsx
diff --git a/frontend/src/components/common/CMSContentBlock.tsx b/frontend/src/components/common/CMSContentBlock.tsx
index d822e1de..1fae596b 100644
--- a/frontend/src/components/common/CMSContentBlock.tsx
+++ b/frontend/src/components/common/CMSContentBlock.tsx
@@ -5,6 +5,7 @@ import { useTranslation } from 'react-i18next';
import DOMPurify from 'dompurify';
import { Card } from './Card';
import { Loading } from './Loading';
+import { PoweredBy } from './PoweredBy';
import { cmsService } from '../../services/cms.service';
import { usePublicSettings } from '../../hooks/usePublicSettings';
import { buildResourceUrl } from '../../utils/url';
@@ -132,11 +133,7 @@ export const CMSContentBlock: React.FC = ({ slug, fallback
{lang === 'de' ? 'Datenschutz' : 'Privacy Policy'}
- {!settings?.branding_hide_powered_by && (
-
- Powered by PicPeak
-
- )}
+
);
diff --git a/frontend/src/components/common/PoweredBy.tsx b/frontend/src/components/common/PoweredBy.tsx
new file mode 100644
index 00000000..68b3fabf
--- /dev/null
+++ b/frontend/src/components/common/PoweredBy.tsx
@@ -0,0 +1,25 @@
+import React from 'react';
+import { useTranslation } from 'react-i18next';
+import { usePublicSettings } from '../../hooks/usePublicSettings';
+
+interface PoweredByProps {
+ className?: string;
+ style?: React.CSSProperties;
+}
+
+// "Powered by PicPeak" footer/login attribution. Reads the setting itself (like
+// DynamicFavicon) so callers don't repeat the guard; hidden for white-label.
+export const PoweredBy: React.FC = ({ className, style }) => {
+ const { t } = useTranslation();
+ const { data: settings } = usePublicSettings();
+
+ // Hide while settings are still loading too, otherwise a white-labelled
+ // instance flashes the attribution before branding_hide_powered_by resolves.
+ if (!settings || settings.branding_hide_powered_by) return null;
+
+ return (
+