Inspect Popup exceeds window

[zaaakher]

Astro Info

Astro                    v4.8.3
Node                     v18.19.0
System                   Windows (x64)
Package Manager          npm
Output                   static
Adapter                  none
Integrations             @astrojs/react
                         @astrojs/tailwind
                         @astrojs/partytown

If this issue only occurs in one browser, which browser is a problem?

Chrome

Describe the Bug

When I try the inspect feature the popup exceeds the current window. I'm not sure if this happens because my site is RTL or not
(notice the scrollbar at the bottom is due to the popup)

What's the expected result?

It should stay within the window and pop to the left/right depending on conditional awareness of it's position

Link to Minimal Reproducible Example

n/a

Participation

• I am willing to submit a pull request for this issue.

[github-actions]

Hello @zaaakher. Please provide a minimal reproduction using a GitHub repository or StackBlitz. Issues marked with needs repro will be closed if they have no activity within 3 days.

[matthewp]

Please provide a reproduction. If RTL is the problem then you should be able to reproduce that way, thank you.

[zaaakher]

My apologies @matthewp, here you go https://stackblitz.com/edit/github-5k1mkz?file=src%2Fpages%2Findex.astro

Here's a quick gif showing it:

[bluwy]

Looks like rtl is messing with the positioning calculation

[V3RON]

@bluwy There is a check for right-side overflow, but not for left-side overflow. I'll open a PR adding one shortly.