Skip to content

feat(header): allow the product name to be configured - #33

Open
printminion-co with Copilot wants to merge 1 commit into
ionos-dev-v33from
copilot/cherry-pick-pr-32
Open

feat(header): allow the product name to be configured#33
printminion-co with Copilot wants to merge 1 commit into
ionos-dev-v33from
copilot/cherry-pick-pr-32

Conversation

Copilot AI commented Aug 13, 2026

Copy link
Copy Markdown

Cherry-pick of #32 onto ionos-dev-v33. The global header's wordmark is a hardcoded SVG, making it impossible to rename the product for a deployment without forking or DOM injection.

Changes

  • lib/Listener/BeforeTemplateRenderedListener.php — reads ionos_product_name from system config and injects it into IInitialState
  • src/components/HeaderLogo.vue — adds optional productName prop; renders a text node when set, falls back to the existing SVG wordmark when empty/unset
  • src/main.ts — wires productName from initial state into HeaderLogo

Fully backwards compatible — unset or empty ionos_product_name leaves the header unchanged.

The product name in the global header is currently a hardcoded SVG with the
letters as vector paths, so a Nextcloud instance cannot name itself without
forking this app or reaching into its DOM from outside.

This adds an optional `productName` prop to HeaderLogo, provided through the
existing `IInitialState` pipeline from a new `ionos_product_name` system config
key — the same pattern already used for `ionos_homepage`,
`ionos_help_target_link` and `ionos_peer_products`.

Fully backwards compatible: when the key is unset or empty the wordmark SVG
renders exactly as before, so existing deployments see no change.

Verified: `npm run build` succeeds, `npm run lint` is clean, and
`npm run stylelint` reports the same 3 pre-existing warnings as on the base
branch (UserMenu.vue:190, styles.scss:30/32) with none added.

Co-authored-by: printminion-co <145785698+printminion-co@users.noreply.github.com>
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.

3 participants