UI: fix window position is not updated at startup on linux #169

Closed
minhhut wants to merge 3 commits from fix-remember-window-position-on-linux into master
minhhut commented 2024-11-02 22:20:28 +00:00 (Migrated from github.com)

On Linux, the startup window position is not working. I find this Arrange() function could make it works. It has been tested on my Linux desktop. I don't know if this problem happens on other platform.

On Linux, the startup window position is not working. I find this Arrange() function could make it works. It has been tested on my Linux desktop. I don't know if this problem happens on other platform.
github-actions[bot] commented 2024-11-02 22:31:39 +00:00 (Migrated from github.com)
Download the artifacts for this pull request: * [ryujinx-Release-1.2.0+44fb06c-linux_arm64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155302771.zip) * [ryujinx-Release-1.2.0+44fb06c-linux_x64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155307062.zip) * [ryujinx-Release-1.2.0+44fb06c-macos_universal](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155312385.zip) * [ryujinx-Release-1.2.0+44fb06c-win_x64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155316277.zip) <details><summary>GUI-less (SDL2)</summary> * [sdl2-ryujinx-headless-Release-1.2.0+44fb06c-linux_arm64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155302873.zip) * [sdl2-ryujinx-headless-Release-1.2.0+44fb06c-linux_x64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155307158.zip) * [sdl2-ryujinx-headless-Release-1.2.0+44fb06c-macos_universal](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155312468.zip) * [sdl2-ryujinx-headless-Release-1.2.0+44fb06c-win_x64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155316416.zip) </details> <details><summary>Only for Developers</summary> * [ryujinx-Debug-1.2.0+44fb06c-linux_arm64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155302454.zip) * [sdl2-ryujinx-headless-Debug-1.2.0+44fb06c-linux_arm64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155302777.zip) * [ryujinx-Debug-1.2.0+44fb06c-linux_x64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155308898.zip) * [sdl2-ryujinx-headless-Debug-1.2.0+44fb06c-linux_x64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155309062.zip) * [ryujinx-Debug-1.2.0+44fb06c-macos_universal](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155312407.zip) * [sdl2-ryujinx-headless-Debug-1.2.0+44fb06c-macos_universal](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155312493.zip) * [ryujinx-Debug-1.2.0+44fb06c-win_x64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155320488.zip) * [sdl2-ryujinx-headless-Debug-1.2.0+44fb06c-win_x64](https://nightly.link/GreemDev/Ryujinx/actions/artifacts/2155320646.zip) </details>
minhhut commented 2024-11-09 22:28:44 +00:00 (Migrated from github.com)

Tested with version 1.2.72, the issue is fixed

Tested with version 1.2.72, the issue is fixed
GreemDev commented 2024-11-09 22:30:18 +00:00 (Migrated from github.com)

That's weird. I didn't fix this. Lol

That's weird. I didn't fix this. Lol
minhhut commented 2024-11-09 22:37:48 +00:00 (Migrated from github.com)

Could it be this one? here

Could it be this one? [here](https://github.com/GreemDev/Ryujinx/pull/169/commits/456e34f20bab1f40038f974f13e13d5a5aabaf1c)
GreemDev commented 2024-11-09 22:43:16 +00:00 (Migrated from github.com)

I doubt it, because that's effectively just an inlining of what the method it replaced did.

I doubt it, because that's effectively just an inlining of what the [method it replaced did](https://github.com/GreemDev/Ryujinx/blob/a2c003501371463fd1f98d2e5a7602ae19c21d7c/src/Ryujinx/UI/Windows/MainWindow.axaml.cs#L407-L429).
minhhut commented 2024-11-10 21:58:24 +00:00 (Migrated from github.com)

Me too, but it could be something different on my machine. If anyone else has same issue, I could reopen the PR. Otherwise, it's fixed 👍

Me too, but it could be something different on my machine. If anyone else has same issue, I could reopen the PR. Otherwise, it's fixed :+1:

Pull request closed

Sign in to join this conversation.
No description provided.