odoo/odoo#163337

Created by Website, Quentin Smetz (qsm)

label

odoo-dev:master-navbar-expand-cls-fix-qsm

head

9b9070014923b5435efa84d25d90d86a4e7da96c

target

master

merged

3 weeks ago by Website, Quentin Smetz (qsm)

[FIX] website: allow proper customization of navbar expand classes

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.

[1]: https://github.com/odoo/odoo/commit/6854ff8026999a2de20d163ff7118a59c33064c8
[2]: https://github.com/odoo/odoo/commit/2dc3b28e570492b0484d185eb656e74df9c59d68
[3]: https://github.com/odoo/odoo/commit/bc13176de8d66bbdc1c536017b1f046c5fd31a86

Thanks to imanie383 <manuel@vauxoo.com>
Closes https://github.com/odoo/odoo/pull/126489