Skip to content

feat(gh-999): default Header logo form values on add#1183

Merged
wesleyboar merged 1 commit into
feat/GH-999-let-cms-admin-edit-headerfrom
feat/GH-999-header-logo-defaults
Jun 16, 2026
Merged

feat(gh-999): default Header logo form values on add#1183
wesleyboar merged 1 commit into
feat/GH-999-let-cms-admin-edit-headerfrom
feat/GH-999-header-logo-defaults

Conversation

@wesleyboar

@wesleyboar wesleyboar commented Jun 16, 2026

Copy link
Copy Markdown
Member

Overview

Pre-fills the Header logo plugin admin form when adding a new instance so editors see home link, height, original-size cropping, and margin defaults (PR #1083 to-do).

Related

Changes

  • added HeaderLogoForm with add-only defaults
  • updated HeaderLogoPlugin to use the form
  • updated Let CMS Admin Edit Header #999 plan follow-up note (settings / Core-Styles parity deferred)

Testing

  1. Delete Header logo from header-content if present.
  2. Add TACC Header → Header logo; confirm form shows home page link, height 50, use original image, Attributes class mr-5.
  3. Save, publish; confirm logo renders with link and expected markup.
  4. Edit existing plugin; confirm defaults are not re-applied.

UI

Delete.Instance.Add.Instance.mov
Edit.Instance.Check.Instance.mov

New HeaderLogoForm pre-fills home link, height 50, original image, and mr-5
for editors; addresses #1083 follow-up. Plan doc notes deferred settings parity.
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@wesleyboar wesleyboar merged commit 15fc05c into feat/GH-999-let-cms-admin-edit-header Jun 16, 2026
@wesleyboar wesleyboar deleted the feat/GH-999-header-logo-defaults branch June 16, 2026 19:30
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