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]: Window moving when custom scaling is set #35182

Open
3 tasks done
voxain opened this issue Aug 2, 2022 · 16 comments
Open
3 tasks done

[Bug]: Window moving when custom scaling is set #35182

voxain opened this issue Aug 2, 2022 · 16 comments
Labels

Comments

@voxain
Copy link

voxain commented Aug 2, 2022

Preflight Checklist

Electron Version

19.0.8

What operating system are you using?

Windows

Operating System Version

Windows 11 21H2

What arch are you using?

x64

Last Known Working Electron version

No response

Expected Behavior

Steady Application window.

Actual Behavior

When typing in the ueli window when custom scaling is set, every keystroke moves the window up left diagonally by one pixel. I have experienced this with a scaling of 110%, but other users have had it with other values set. When using fixed system values like 125%, the issue does not seem to appear.

Testcase Gist URL

No response

Additional Information

This bug was forwarded from an issue that appeared in ueli: oliverschwendener/ueli#515
It has appeared to me myself and also to other users of ueli, and as @oliverschwendener has pointed out, it appears to be an electron bug that has not been reported yet.

@voxain voxain added the bug 🪲 label Aug 2, 2022
@VerteDinde
Copy link
Member

Thanks for reporting this issue! 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.

I'm adding the blocked/need-repro label for this reason. After you make a test case, please link to it in a followup comment.

Thanks in advance!

@VerteDinde VerteDinde added the blocked/need-repro Needs a test case to reproduce the bug label Aug 2, 2022
@oliverschwendener
Copy link

@VerteDinde Here is a repo with instructions on how to reproduce the issue: https://github.com/oliverschwendener/electron-moving-window-bug

To me, it seems like the issue is caused by the BrowserWindow.setSize() method.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 8, 2022

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

@github-actions github-actions bot added the stale label Nov 8, 2022
@oliverschwendener
Copy link

bump

@github-actions github-actions bot removed the stale label Nov 9, 2022
@github-actions github-actions bot removed the blocked/need-repro Needs a test case to reproduce the bug label Jan 10, 2023
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

@github-actions github-actions bot added the stale label Apr 11, 2023
@voxain
Copy link
Author

voxain commented Apr 11, 2023

Bump - Haven't tested it myself, but since I've heard nothing from this yet, I'm assuming it's still open.

@oliverschwendener
Copy link

oliverschwendener commented Apr 11, 2023

I have updated the demo repository to the latest electron version, but I have to quickly check if this is still the problem with electron 24.

@oliverschwendener
Copy link

oliverschwendener commented Apr 11, 2023

I can confirm that this is still an issue with electron 24. Please check the repository that I mentioned above to reproduce it.

@github-actions github-actions bot removed the stale label Apr 12, 2023
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

@github-actions github-actions bot added the stale label Jul 11, 2023
@oliverschwendener
Copy link

bump

I just updated the repo to reproduce the bug with the latest electron version, and it's still there.

@github-actions github-actions bot removed the stale label Jul 13, 2023
@electron-issue-triage
Copy link

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

@oliverschwendener
Copy link

bump

@electron-issue-triage
Copy link

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

@oliverschwendener
Copy link

I give up

@electron-issue-triage
Copy link

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

@voxain
Copy link
Author

voxain commented Apr 11, 2024

bugs usually dont solve themselves this easily

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants