Skip to content

Use auth logo image in admin header instead of shield icon - #33

Closed
timujinne wants to merge 2 commits into
mainfrom
admin-header-logo-from-settings
Closed

Use auth logo image in admin header instead of shield icon#33
timujinne wants to merge 2 commits into
mainfrom
admin-header-logo-from-settings

Conversation

@timujinne

Copy link
Copy Markdown
Owner

Summary

  • Replace hardcoded shield icon in admin top bar with the configurable auth logo image from Settings (auth_logo_file_uuid)
  • When no logo is configured, the logo area is hidden entirely (no placeholder)
  • Suppress pre-existing dialyzer opaque type warnings in ConnectionsLive MapSet functions (dialyxir/OTP compatibility bug)

Test plan

  • Upload a logo image in Settings → Authorization → "Logo Image"
  • Verify the logo appears in the admin header (top bar, left side next to project title)
  • Remove the logo image from settings
  • Verify the logo area is hidden (no shield, no empty space)
  • Test on multiple projects to confirm universal behavior

🤖 Generated with Claude Code

timujinne and others added 2 commits March 24, 2026 16:34
…I toggle

- Replace hardcoded cookie/privacy URLs with Routes.path() to fix double-slash bug
- Add dynamic legal_links from published pages + single /legal index link
- Use bg-base-100/70 backdrop instead of bg-black for light/dark theme compatibility
- Improve glass opacity (0.95→0.98), card bg (50→80%), text contrast
- Replace custom toggle with standard daisyUI toggle toggle-primary
Read auth_logo_file_uuid setting and display the uploaded logo
in the admin top bar. When no logo is configured, the logo area
is hidden entirely instead of showing the default shield icon.

Also suppress pre-existing dialyzer opaque type warnings in
ConnectionsLive MapSet-related functions (dialyxir bug with
newer Elixir/OTP versions).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@timujinne timujinne closed this May 11, 2026
@timujinne
timujinne deleted the admin-header-logo-from-settings branch May 11, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant