Skip to content

Add menu_id config option to MenuPresenter#1408

Merged
Bramjetten merged 2 commits intoSpinaCMS:mainfrom
aseroff:menu-presenter-id
Jan 11, 2026
Merged

Add menu_id config option to MenuPresenter#1408
Bramjetten merged 2 commits intoSpinaCMS:mainfrom
aseroff:menu-presenter-id

Conversation

@aseroff
Copy link
Contributor

@aseroff aseroff commented Aug 1, 2025

Context

I needed to be able to set the menu's id and had to monkeypatch it in. Figured that would happen frequently enough to support.

Changes proposed in this pull request

Introduces a configurable menu_id attribute to the MenuPresenter, allowing the rendered menu to have a custom HTML id. Updates the render_menu method to include the id attribute in the menu tag.

Guidance to review

No worries either way

Introduces a configurable menu_id attribute to the MenuPresenter, allowing the rendered menu to have a custom HTML id. Updates the render_menu method to include the id attribute in the menu tag.
@stale
Copy link

stale bot commented Sep 30, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Sep 30, 2025
@stale stale bot closed this Oct 7, 2025
@Bramjetten Bramjetten reopened this Jan 11, 2026
@stale stale bot removed the stale label Jan 11, 2026
@Bramjetten Bramjetten merged commit ba14ead into SpinaCMS:main Jan 11, 2026
1 check passed
@aseroff aseroff deleted the menu-presenter-id branch January 11, 2026 20:53
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