cod-offcanvas-body component was not allowing overflow for its content.
This PR:
Allow overflow-y: auto but only when the offcanvas body is not being used as a responsive navbar. If we allow overflow-y: auto on the responsive navbar, then things with dynamic height - like dropdowns - cause vertical scroll in the navbar.
Note: This conflict of styles between the responsive offcanvas nav bar and the offcanvas components is a good reason why we need to reduce dependencies across components.
Testing
Test out responsive navbar and offcanvas with long list in Storybook:
Fixes #229
cod-offcanvas-body
component was not allowing overflow for its content.This PR:
overflow-y: auto
but only when the offcanvas body is not being used as a responsive navbar. If we allowoverflow-y: auto
on the responsive navbar, then things with dynamic height - like dropdowns - cause vertical scroll in the navbar.Note: This conflict of styles between the responsive offcanvas nav bar and the offcanvas components is a good reason why we need to reduce dependencies across components.
Testing
Test out responsive navbar and offcanvas with long list in Storybook:
https://github.com/CityOfDetroit/COD-Design-System/assets/143553259/3a7fa484-f36b-4aee-915d-67a313f6586e