-
Notifications
You must be signed in to change notification settings - Fork 53
Update SDL and ImGui to newer versions #281
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
base: main
Are you sure you want to change the base?
Conversation
|
I have not checked all draws for assert errors, but on a first pass it looks fine. Some of the coloring seems to have changed slightly when it comes to the background draw, but not sure if it might be related to the new styling options that is in the new ImGui version. Build will fail to complete until https://github.com/goatcorp/XIVLauncher.Core/blob/main/.github/workflows/build.yml#L28-L31 allows for .NET 9 builds which both .NET 8 and 9 comes pre-installed with the Linux runner |
86e574a to
fd60db3
Compare
|
Requesting goat's review for this one since to my knowledge prior work with HexaImGui was done in Dalamud (and I'm not all-to-knowledgable on the specifics). From a first glance though all this looks fine to me |
|
Builds but does not run on bazzite/fedora43. When launched from terminal: Tried with SDL_VIDEODRIVER=x11, and I get: |
|
As of d4cdb5d (fix id assert error), it launches and all the settings tabs work. Still some graphical issues, but the window pops up, and then it successfully launches the game. |
TODO: - Image loading isn't fully working (stride issue) - Font no longer is applied correctly - Remove veldrid and old imgui from code base once the aboves are fully working (currently only debug removes it)
be8e0f2 to
5130e6c
Compare
|
X11 seems to not report scaling correctly (could just be xwayland) so may need to add a scaling override setting if this becomes a problem. Default launches with video driver "wayland" unless explicitly set otherwise. Everything else works as the old XLCore did, just with newer ImGui version and better bindings |
|
Can confirm, aside from slightly wonky X11 scaling when in KDE wayland with "Apps scale themselves", it's working. There may be some edge cases we havent hit yet, but I was able to use all the settings tabs, change settings, save settings. One time password works as well. We need to have a steam deck user do some testing to see how it functions in that environment, though. Need to make sure they can still interact with it, enter info, etc. |
|
I’ll do device testing for steam deck and another set of tests for regular desktop setups as soon as I’m available to do so |
|
seems like it works for me (X11). however, the cursor (in at least the username/password boxes) is very thin and also red, so it's really hard to see. also, this manages to induce really bad coil whine somewhere in my computer, which nothing else has done with this computer ever?? the pitch modulates with window size and changes on different settings tabs, and doesn't reproduce with a screenshot. no idea at all |
Sounds like the frame rate isn’t being capped properly for the launcher UI and thus overworking the GPU? |
|
I just checked with mangohud ( |
I'm not seeing anything when I run it. No spike in cpu or gpu usage (btop, nvtop). With and without gamemode installed and active/inactive. Could be just because I have a very fast cpu and gpu, though. |
Blooym
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Marking as changes needed since this currently instantly crashes on the Steamdeck with my testing. I'll take a look as to what's causing it
|
User testing reports:
|
2e5da51 to
f5fe732
Compare
This sounds like you are compiling it wrong in as it specifically uses the RIDs that needs to be provided for a compile to work |

Things changing in this:
.NET 8 -> .NET 9
Serilog 2 -> Serlog 4
ImGui.NET -> Hexa.NET.ImGui
Veldrid -> Hexa.NET.SDL3
.NET update reason:
Pulling in newest XIVLauncher.Common for future additions.
Serilog update reason:
.NET 9 doesn't allow for Serilog 2.
ImGui update reason:
Need better bindings for SDL3.
SDL update reason:
Veldrid is abandoned.
Build pipeline will fail due to the .NET build only having SDK 8.0.x and not SDK 9.0.x