palcarazm / bs-darkmode-toggle / 24042907350 / 2

98%

v1: 98%

push

github

feat: add button layout as alternative to bootstrap-toggle slider (#74)

* feat: add button layout as alternative to bootstrap-toggle slider

Add a new layout option that creates a native Bootstrap 5 button instead
of the bootstrap-toggle slider component. This reduces external
dependencies and bundle size while maintaining full functionality.

Changes:
- Add Layout enum (BUTTON, TOGGLE) to OptionResolver.types
- Add layout property to DarkModeOptions and ResolvedOptions
- Refactor DOMBuilder into AbstractLayout with two implementations:
  - ToggleLayout: preserves original bootstrap-toggle behavior
  - ButtonLayout: creates a native button with Bootstrap 5 classes
- Add DomManager facade to select appropriate layout
- Update DarkModeToggle to use DomManager with onChange handler
- Add layout option to documentation (features and API table)
- Add ButtonLoader to test application for manual testing
- Add comprehensive unit tests for all layout components

Default layout remains "toggle" for backward compatibility.
Invalid layout values fall back to "toggle" gracefully.

Implements #61

* chore: make bootstrap5-toggle an optional peer dependency

Since the new button layout (introduced in #61) does not require the
bootstrap5-toggle library, this dependency is now optional. Users who
only use the button layout no longer need to install it, reducing
unnecessary dependencies and bundle size.

This change maintains backward compatibility for existing users who
rely on the default toggle layout.

Implements #61

* refactor: remove unnecessary non-null assertions from layouts

Remove the non-null assertion operators (!) from ButtonLayout and
ToggleLayout methods. The button and input getters already guarantee
that the elements exist (lazy initialization via nullish coalescing
assignment), making the assertions redundant.

This change resolves SonarQube warning typescript:S4325 (redundant
casts and non-null assertions) and improves code safety without
affectin... (continued)

205 of 210 branches covered (97.62%)

Branch coverage included in aggregate %.

478 of 484 relevant lines covered (98.76%)

29.47 hits per line