SUI-Components / sui-components

SUI Components
https://sui-components.vercel.app/
MIT License
140 stars 59 forks source link

Feature/avoid page scrolling when drawer open prop #2722

Closed xmurcia closed 6 months ago

xmurcia commented 6 months ago

Category/Component

๐Ÿ” Show

Description, Motivation and Context

Allow a new prop to avoid the scroll on a page when the drawer is opened. The same feature is already implemented in the molecule/modal component.

Types of changes

github-actions[bot] commented 6 months ago
STATEMENTS BRANCHES FUNCTIONS LINES
โ‰ โ‰ 0๏ผ โ‰ 0.06โ†‘ โ‰ 0.07โ†‘ โ‰ 0.07โ†‘
% 75.92 63.83 65.69 77.62
ABS 3342 / 4402 1998 / 3130 653 / 994 3164 / 4076
andresin87 commented 6 months ago

tab navigation blocking is also implemented?

xmurcia commented 6 months ago

tab navigation blocking is also implemented?

This kind of behavior is not implemented, I guess you mean when the drawer is opened, to limit the scope of the focus just inside the drawer and not other DOM element, right? Maybe we can have a look in a different task in order to do it in modals and drawer because I've seen it is also missing in there.