The README.mdx file for the offcanvas component lists the backdrop property as a boolean, but if it is used this way it will not work correctly as the logic for setting a default value depends on a string.
If false is passed to this component instead of 'false' the resulting value from the following code will be 'true'.
{% set backdrop = backdrop|default('true') %}
The comments in offcanvas.twig accurately define this:
* - backdrop: (optional) Backdrop: 'true' | 'false' | 'static'.
The definition in README.mdx shows:
- `backdrop` (bool) (default: true): Backdrop.
Update the README, defining this as a string instead of a bool.
Active
6.0
Documentation
Thanks a lot Christopher, merged and updated the https://radix.trydrupal.com/radix/components/offcanvas as well
I noticed some duplicates there as well, that aside I wonder if we should make that into a boolean instead of string.
Would appreciate another MR