[FIX] website: allow proper customization of navbar expand classes #163337
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Commit 1 introduced a template for the website navbar with a configurable
_navbar_expand_class
variable... but the variable was acting as a boolean by mistake: if set, it would not set up any class and if not set, it would set the default navbar-expand-lg class.Meanwhile, a new
_navbar_classes
variable was introduced allowing customization anyway but it is still worth fixing for consistency. Also, commit 2 and 3 actually used the variable giving it the string "None"... which is just inconsistent and confusing, despite working as intended given the original implementation mistake.Thanks to imanie383 manuel@vauxoo.com
Closes #126489