Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Windowing Issues in Electron and Chromium #41682

Open
3 tasks done
LSeelig opened this issue Mar 24, 2024 · 2 comments
Open
3 tasks done

[Bug]: Windowing Issues in Electron and Chromium #41682

LSeelig opened this issue Mar 24, 2024 · 2 comments

Comments

@LSeelig
Copy link

LSeelig commented Mar 24, 2024

Preflight Checklist

Electron Version

29.1.5

What operating system are you using?

Other Linux

Operating System Version

Fedora Linux 39 (Workstation Edition)

What arch are you using?

x64

Last Known Working Electron version

No response

Expected Behavior

Windows display, maximize, split tile, and move without graphical glitches and inconsistences

Actual Behavior

I know this is not how actual behavior should be used, but I am submitting this on my own time. If the layout is too messy, disregard or remove this report.

Please see additional information for my notes, along with attached screenshots, on the windowing behavior. I include notes on Chromium too because some of this will need to be fixed upstream, and I hope to send this there, but I can't quite tell what is peculiar to electron vs chromium.

Testcase Gist URL

No response

Additional Information

Chromium and Electron
Both Have Chromium version 122.0.6261.0
Electron ver Electron v29.1.5

No scaling
Maximized/Splitscreen to Splitscreen: Electron has a single-pixel gap above and to left of title bar (screenshot 1) and some bugginess (screenshot 2)

Screenshot from 2024-03-24 15-38-40
Screenshot from 2024-03-24 15-44-25

Non-maximized to Splitscreen: Electron breaks, expanding right (both right and left tile expand right), graphical errors very slight on top bar (expands past windows, can see behind top of title bar, barely) (see screenshot)

Screenshot from 2024-03-24 15-38-40

Large text (125% text scaling)
Splitscreen chromium shows main window extending slightly right beyond the title bar (screenshot)
Screenshot from 2024-03-24 15-47-02

Electron unaffected (DOES NOT SCALE TEXT which violates accessibility)

Fractional Scaling (125%)

Electron - Scrollbar extends slightly out of window (screenshot) (only tested in windowed electron, might have seen in chromium)
Screenshot from 2024-03-24 15-50-43

Chromium - Title bar enlarges, shadowing sometimes expands on left and right further than main content, when tiled either left or right (Screenshots)
Screenshot from 2024-03-24 15-53-45
Screenshot from 2024-03-24 15-55-18

Electron - Intense graphical glitching around titlebar in full and splitscreen; titlebar width less than window content (couldn't easily capture in screenshot, as they resolved whenever I tried. Schrodinger's window, I guess)

Electron - Single-pixel gap above and to the left persists when maximized or splitscreened (screenshot); it is more appearant when scaled
Screenshot from 2024-03-24 15-55-18
Screenshot from 2024-03-24 15-56-51

Electron and Chromium - When windowed, shadows/boundaries drawn slightly outside of window content (screenshot)
Screenshot from 2024-03-24 15-58-35

Electron: Non-maximized to splitscreen behavior persists

Fedora KDE: Doesn't seem to have the same issues by default, but I will note this might be a problem with chromium's gtk implementaiton, instead of GNOME's mutter implementation, as it shows similar issues if switched from chrome://settings/appearance to "Use GTK" and "Use QT" doesn't work at all, but can "Use system title bar and borders" by right clicking the top of the window.
There is what might be a font rendering/spacing issue when fractional scaling.

I omitted testing an integer scale (i.e. 200%). I apologize; I am pretty busy.

@electron-issue-triage
Copy link

Hello @LSeelig. Thanks for reporting this and helping to make Electron better!

Would it be possible for you to make a standalone testcase with only the code necessary to reproduce the issue? For example, Electron Fiddle is a great tool for making small test cases and makes it easy to publish your test case to a gist that Electron maintainers can use.

Stand-alone test cases make fixing issues go more smoothly: it ensure everyone's looking at the same issue, it removes all unnecessary variables from the equation, and it can also provide the basis for automated regression tests.

Now adding the blocked/need-repro Needs a test case to reproduce the bug label for this reason. After you make a test case, please link to it in a followup comment. This issue will be closed in 10 days if the above is not addressed.

@LSeelig
Copy link
Author

LSeelig commented Mar 25, 2024

Hello @LSeelig. Thanks for reporting this and helping to make Electron better!

Would it be possible for you to make a standalone testcase with only the code necessary to reproduce the issue? For example, Electron Fiddle is a great tool for making small test cases and makes it easy to publish your test case to a gist that Electron maintainers can use.

Stand-alone test cases make fixing issues go more smoothly: it ensure everyone's looking at the same issue, it removes all unnecessary variables from the equation, and it can also provide the basis for automated regression tests.

Now adding the blocked/need-repro Needs a test case to reproduce the bug label for this reason. After you make a test case, please link to it in a followup comment. This issue will be closed in 10 days if the above is not addressed.

This isn't relevant to issue. The issue occurs with the base/demo package.

@electron-issue-triage electron-issue-triage bot removed the blocked/need-repro Needs a test case to reproduce the bug label Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: 👀 Unsorted Items
Development

No branches or pull requests

3 participants