Skip to content

Constrain admin plugin logo size to recommended dimensions#349

Merged
simonhamp merged 1 commit intomainfrom
constrain-admin-logo-size
Apr 10, 2026
Merged

Constrain admin plugin logo size to recommended dimensions#349
simonhamp merged 1 commit intomainfrom
constrain-admin-logo-size

Conversation

@simonhamp
Copy link
Copy Markdown
Member

@simonhamp simonhamp commented Apr 10, 2026

Summary

  • Fixes oversized logo images on the admin plugin edit screen by using inline styles instead of Tailwind classes
  • Constrains the logo preview to 256x256px max, matching the dimensions recommended to developers in the plugin edit form ("Recommended: 256x256 pixels, square")
  • Tailwind classes (w-16 h-16) weren't being applied because Filament's CSS build doesn't scan inline HTML strings in PHP files

Test plan

  • Open a plugin with a custom logo in the admin edit screen
  • Verify the logo is constrained to a reasonable size (max 256x256) instead of filling the entire space
  • Verify plugins without logos still show "No logo" text

🤖 Generated with Claude Code

Use inline styles instead of Tailwind classes for the logo preview in
the admin plugin edit form, since Filament's CSS build doesn't include
classes from inline HTML strings.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@simonhamp simonhamp marked this pull request as ready for review April 10, 2026 11:52
@simonhamp simonhamp merged commit 85f461e into main Apr 10, 2026
2 checks passed
@simonhamp simonhamp deleted the constrain-admin-logo-size branch April 10, 2026 12:07
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