Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Custom navbar config needs to reference the search bar #2763

Open
maelle opened this issue Aug 19, 2024 · 3 comments
Open

Custom navbar config needs to reference the search bar #2763

maelle opened this issue Aug 19, 2024 · 3 comments

Comments

@maelle
Copy link
Collaborator

maelle commented Aug 19, 2024

it used not to, the change is due to #2449

https://mastodon.social/@sebffischer/112954449837337953

@jayhesselberth
Copy link
Collaborator

Do you mean that previously if you had a custom navbar config but didn't include search, it would be included in the navbar anyway? If so, then the new behavior is actually more consistent with the documentation. Bummer it affects lots of sites though.

FWIW this was an unintentional "fix".

Can you link to a site that was affected by this change?

@maelle
Copy link
Collaborator Author

maelle commented Aug 19, 2024

Do you mean that previously if you had a custom navbar config but didn't include search, it would be included in the navbar anyway?

yes!

One example: https://github.com/ropensci/targets/pull/1303/files (more linked from ropensci-org/rotemplate#112)

@hadley
Copy link
Member

hadley commented Aug 19, 2024

I knew this was going to happen so I must have failed to advertise it enough 😞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants