Don't hide sticky elements

[emilkowalski]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
vaul	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 15, 2024 10:38am

[Dearest]

when can I use this pr code in new vaul version (maybe 0.8.1)?

[emilkowalski]

Very soon!

[emilkowalski]

@Dearest This is live now!

[jsefiani]

Hey @emilkowalski, I just tested out the latest version, and I noticed that scrolling up inside a drawer doesn't work anymore, most likely because of this change.

[emilkowalski]

Where doesn't it work? It works fine for me in the codesandbox demo.

RPReplay_Final1705763620.mov

[Lcarv20]

Hi there, sorry for bringing this back up, but I am using the latest version and the sticky components still disappear. Maybe I am doing something wrong, would you care to take a look at this sandbox?

• sandbox

also a vid:

output.mp4

(this maybe should be a new issue) One suggestion I also would like to make is to keep track the body background state before and after toggling, to not keep it black after closing, my current solution is something like:

...
       onClose={() => {
         document.body.style.background = "";
       }}
...

I am saying this because I noticed on iPhones with notch the background on the top corners stays black after interacting with the drawer component

#191 #143