sebnitu / vrembem / 23231982203 / 1

91%

main: 100%

push

github

Improve the drawer component and config module integration (#2705)

This PR primarily refactors the drawer component to the new CSS variable pattern and cleans up the drawer collection API. This pattern applies CSS variable references along with smart fallbacks. This replaces the initial Sass variable solution for component customization. The documentation has also been updated to reflect these changes.

**Additional changes**

- `core`: CollectionEntry now has the `deregister` method and this method has been removed from drawer, modal and popover entry classes.
- `core`: Added new `private` helper that exports the `_` function. This is used to set and get private properties for collection entries. It is used to store the private values of drawer `mode`, `state`, and `inlineState`. This fixes an issue where using the `#private` syntax was not compatible with proxied entries.
- `core`: Fixed issues with proxied entries not correctly calling actions on the receiver argument.
- `core`: Fixed core watch script.
- `drawer`: Added validation to drawer state properties `mode`, `state`, and `inlineState`.
- `drawer`: Removed the need for `setState` and `applyState` methods in favor of getter/setter properties.
- `drawer`: Removed custom events in favor of collection event system.
- `drawer`: Added documentation of drawer config and allow for type infer instead of explicit interface declaration.
- `drawer`: Simplified drawer preset for `propStore` plugin.
- `docs`: ReferenceTable now has the ability to be expandable.
- `docs`: ReferenceTable now prepends the ID for row permalinks.
- `docs`: ReferenceTable now has key and value label props.
- `docs`: ReferenceTable scripts have been moved into their own class files and are built using Collections.
- `docs`: UtilityTable has been deprecated in favor of ReferenceTable. Styles have been adjusted to better match the more minimal look of UtilityTable.
- `docs`: Refactored how Sass, CSS Vars and JS config are documen... (continued)

590 of 666 branches covered (88.59%)

Branch coverage included in aggregate %.

948 of 1014 relevant lines covered (93.49%)

31.73 hits per line