From 103dd796d7991310bc787bc91eb17b2e38894820 Mon Sep 17 00:00:00 2001 From: Wesley B <62723358+wesleyboar@users.noreply.github.com> Date: Wed, 2 Sep 2026 17:06:21 -0500 Subject: [PATCH 1/5] fix: hide generic Tapis logo next to TACC logo for the TACC User Portal client Reuses the footer's client_display_name check (matching "TUP" or "TACC User Portal", case-insensitive) so the header figure doesn't show the placeholder Tapis logo alongside the hardcoded TACC logo when the client is the TACC User Portal. --- service/templates/branding.html | 3 +++ 1 file changed, 3 insertions(+) diff --git a/service/templates/branding.html b/service/templates/branding.html index ef0eadc..13931db 100644 --- a/service/templates/branding.html +++ b/service/templates/branding.html @@ -40,7 +40,10 @@ {% elif tenant_id == "tacc" %} TACC {% else %} + {% set client_display_name_lower = (client_display_name or '') | lower %} + {% if 'tup' not in client_display_name_lower and 'tacc user portal' not in client_display_name_lower %} Tapis + {% endif %} {% endif %} {% endif %} From ad36c17b5645e9846587a2d2962ceaec9264bbe9 Mon Sep 17 00:00:00 2001 From: Wesley B <62723358+wesleyboar@users.noreply.github.com> Date: Wed, 2 Sep 2026 18:05:10 -0500 Subject: [PATCH 2/5] fix: match client_display_name exactly for Tapis-logo suppression Replace the case-insensitive "tup"/"tacc user portal" substring check with an exact, case-sensitive match against "TACC User Portal", matching the same fix applied to the footer's condition. --- service/templates/branding.html | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/service/templates/branding.html b/service/templates/branding.html index 13931db..95f5389 100644 --- a/service/templates/branding.html +++ b/service/templates/branding.html @@ -40,8 +40,7 @@ {% elif tenant_id == "tacc" %} TACC {% else %} - {% set client_display_name_lower = (client_display_name or '') | lower %} - {% if 'tup' not in client_display_name_lower and 'tacc user portal' not in client_display_name_lower %} + {% if client_display_name != "TACC User Portal" %} Tapis {% endif %} {% endif %} From 05e1d59c7b5e889480f25e40409c1a91d4d156d7 Mon Sep 17 00:00:00 2001 From: Wesley B <62723358+wesleyboar@users.noreply.github.com> Date: Wed, 2 Sep 2026 18:08:57 -0500 Subject: [PATCH 3/5] refactor: extract repeated header figure into content_figure.html Every form template duplicated the same tenant + hardcoded-tacc branding.html double-include inside its
. Extract it into a shared partial, same pattern as content_footer.html. --- service/templates/authorize.html | 9 +-------- service/templates/content_figure.html | 8 ++++++++ service/templates/device-code.html | 9 +-------- service/templates/login.html | 9 +-------- service/templates/logout.html | 9 +-------- service/templates/mfa.html | 9 +-------- service/templates/select_idp.html | 9 +-------- service/templates/success.html | 9 +-------- service/templates/tenant.html | 9 +-------- 9 files changed, 16 insertions(+), 64 deletions(-) create mode 100644 service/templates/content_figure.html diff --git a/service/templates/authorize.html b/service/templates/authorize.html index 315b0b8..66db61d 100644 --- a/service/templates/authorize.html +++ b/service/templates/authorize.html @@ -4,14 +4,7 @@