-
Notifications
You must be signed in to change notification settings - Fork 39
Closed
Description
@kylebarron this is working well and looks good to merge. It would be good to add Playwright test coverage for the overlay mode, but I haven’t found the right way to handle pointer events yet. If we figure that out, we can implement it separately. The lockfile is outdated, we should probably run npm i and update it before merging. Also, we can remove the read code from the overlay component for now and reintroduce it later once we have a proper solution.
Originally posted by @vgeorge in #921 (review)
Metadata
Metadata
Assignees
Labels
No labels