Skip to content

feat: Add optional APP_LOGO branding to the scaffold - #19

Closed
ek9r wants to merge 2 commits into
tempestphp:mainfrom
ek9r:logo
Closed

ek9r wants to merge 2 commits into
tempestphp:mainfrom
ek9r:logo

Conversation

@ek9r

@ek9r ek9r commented Sep 7, 2026 •

Copy link
Copy Markdown

Add support for an optional APP_LOGO environment variable and use the official Tempest logo as the example configuration.

When configured, the logo is rendered above the Tempest heading on the home page.

The official logo is provided as an example in .env.example:

APP_LOGO="/img/tempest-logo.png"

The included tempest-logo.png is sourced from the official Tempest branding repository.

If APP_LOGO is not set in the application's environment, no logo is rendered.

The custom logo assets in public/img/custom/ are community-created demonstrations and a small fan tribute from the German-speaking PHP user group community, including the PHP elephant as a nod to the wider PHP ecosystem. They are kept separately from the official branding and may also serve as examples for future custom/community branding.

And, well… a little fresh breeze never hurt the PHP elephant. 🐘💨


What changed?

  • Added optional APP_LOGO support to the homepage view
  • Added APP_LOGO with the official logo as an example to .env.example
  • Added the official Tempest logo to public/img/
  • Added community-created demonstration logos to public/img/custom/

@brendt

brendt commented Sep 8, 2026

Copy link
Copy Markdown
Member

I'm confused, these aren't Tempest's official logo? https://github.com/tempestphp/.github/tree/main/logo_current

@ek9r

ek9r commented Sep 8, 2026 •

Copy link
Copy Markdown
Author

You're absolutely right @brendt
that wasn't your official Tempest logo.

It was created as a demonstration/community asset and as a small fan tribute. We like it enough that we're about to have T-Shirts printed with it as our little Tempest fanboy outfit. 😄

I've updated the PR description.

@brendt

brendt commented Sep 9, 2026

Copy link
Copy Markdown
Member

You're absolutely right

Ok so I've got to ask how much AI is involved in this conversation 😅

To be clear: I think part of this PR makes sense, but I don't understand why another logo should be shipped with the default app?

@brendt brendt closed this Sep 9, 2026
@ek9r
ek9r deleted the logo branch September 9, 2026 11:37
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.

2 participants