Skip to content

V15: Allows certain adjustments of the logo#18499

Merged
leekelleher merged 16 commits intov15/devfrom
v15/feature/logo-css-sizes
Mar 3, 2025
Merged

V15: Allows certain adjustments of the logo#18499
leekelleher merged 16 commits intov15/devfrom
v15/feature/logo-css-sizes

Conversation

@iOvergaard
Copy link
Contributor

@iOvergaard iOvergaard commented Feb 27, 2025

This pull request includes several changes to the src/Umbraco.Web.UI.Client and src/Umbraco.Web.UI.Login directories, focusing on the app-logo.element.ts, backoffice-header-logo.element.ts, and auth-layout.element.ts files. The changes enhance the flexibility and customization of logo elements and improve code consistency.

This should resolve #18424

Changes

  • Made the app-logo component listen for the active theme and select logo based on that
  • Added more options to the app-logo element to override the theme and lazyload or not
  • Added new alternative logo (the blue logomark) as an option in the backoffice, although it is not being used anywhere in the core UI, but this ensures we are consistent with the login logo, which has two versions
  • Optimised the logo SVG files to remove unneeded metadata
  • Added more CSS variables to control the dimensions of the logo and documented them
  • Added appropriate mock handlers for the new logo

Enhancements to Logo Elements:

This pull request introduces support for an alternative backoffice logo, along with various updates to the logo handling in the application. The changes span across multiple files, including controllers, configurations, UI components, and mock handlers.

Addition of Alternative Backoffice Logo:

Updates to UI Components:

@iOvergaard iOvergaard requested a review from leekelleher March 3, 2025 13:05
@leekelleher leekelleher merged commit 51cce23 into v15/dev Mar 3, 2025
23 checks passed
@leekelleher leekelleher deleted the v15/feature/logo-css-sizes branch March 3, 2025 15:54
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