close
Skip to content

[2.x] feat: convert uploaded logos to WebP, support animated GIF#4458

Merged
imorland merged 2 commits into2.xfrom
im/webp-logo
Mar 15, 2026
Merged

[2.x] feat: convert uploaded logos to WebP, support animated GIF#4458
imorland merged 2 commits into2.xfrom
im/webp-logo

Conversation

@imorland
Copy link
Copy Markdown
Member

Changes proposed in this pull request:

Converts forum logo uploads to WebP (matching the approach used for avatars in #4431), and adds support for animated GIF logos.

  • Static logos → WebP
  • Animated logos → GIF (preserving animation)
  • UploadImageController default extension updated from png to webp
  • UploadLogoDarkModeController inherits this behaviour automatically

Favicon upload is unaffected — it explicitly sets its own extension and .ico passthrough is preserved.

Necessity

  • Has the problem that is being solved here been clearly explained?
  • If applicable, have various options for solving this problem been considered?
  • For core PRs, does this need to be in core, or could it be in an extension?
  • Are we willing to maintain this for years / potentially forever?

Confirmed

  • Frontend changes: tested on a local Flarum installation.
  • Core developer confirmed locally this works as intended.
  • Tests have been added, or are not appropriate here.

Co-Authored-By: ianmorland <ianmorland@users.noreply.github.com>
@imorland imorland marked this pull request as ready for review March 15, 2026 15:56
@imorland imorland requested a review from a team as a code owner March 15, 2026 15:56
@imorland imorland merged commit 1012e0e into 2.x Mar 15, 2026
27 checks passed
@imorland imorland deleted the im/webp-logo branch March 15, 2026 16:09
@imorland imorland added this to the 2.0.0-beta.8 milestone Mar 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants