Skip to content

Menu: Allow to explicitly override if a menu url is remote or not#809

Open
anthraxx wants to merge 1 commit intoalex-shpak:mainfrom
anthraxx:feat/override-remote-url-item
Open

Menu: Allow to explicitly override if a menu url is remote or not#809
anthraxx wants to merge 1 commit intoalex-shpak:mainfrom
anthraxx:feat/override-remote-url-item

Conversation

@anthraxx
Copy link
Copy Markdown
Contributor

@anthraxx anthraxx commented Apr 1, 2026

This can be very useful if a hugo website f.e. on GitHub pages is constructed from different individual spec repos but still represent one coherent site.

This can be very useful if a hugo website f.e. on GitHub pages is
constructed from different individual spec repos but still represent
one coherent site.
@alex-shpak
Copy link
Copy Markdown
Owner

alex-shpak commented Apr 7, 2026

Hello! LGTM. I don't mind the param, but thinking should we find a better name? not sure if remote is clear enough for users. What do you think?

@anthraxx
Copy link
Copy Markdown
Contributor Author

anthraxx commented Apr 7, 2026

Sure, sounds reasonable. Do you have anything in mind? I just tried to related to the internal variable $isRemote, happy to change it to a better matching name.

@alex-shpak
Copy link
Copy Markdown
Owner

well, there is BookIcon param already, so feels good to keep patten, maybe BookNewTab, and then to make sure that this param takes over .IsAbs

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