knocklabs / javascript

Official JavaScript packages for interacting with Knock
https://knock.app/
MIT License
11 stars 2 forks source link

[Bug] wrong placement with React's NotificationFeedPopover in large screen (> 1400px) #44

Closed dinwwwh closed 5 months ago

dinwwwh commented 5 months ago

image

A bell icon represents the notification button. However, when the screen size is larger than 1400 pixels, the NotificationFeedPopover won't appear close to the button.

The bug occurs when I place Knock's popover inside container-type: inline-size; CSS. Although this style is not commonly used, I hope the team can soon support it.

Please take a look at my demo to view the problem: https://dinstack-web-preview.dinsterizer.com/.

dinwwwh commented 5 months ago

I used React Portal to fix the issue mentioned earlier, as well as the z-index problem.