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

feat: add DXVK settings tab #17

Closed
wants to merge 9 commits into from

Conversation

rankynbass
Copy link
Contributor

@rankynbass rankynbass commented Dec 11, 2022

Here's the XL.Core side of the DXVK settings patch. This requires PR #1205 from FFXIVQuickLauncher to work, otherwise it will not compile. I also suggest adding PR #13, and PR #15, since those improve the UI to expose more information and make it more consistant. Edit: These PRs have been added already.

  • Added DXVK tab to settings.
  • Moved a few settings from wine tab to DXVK tab.
  • Added dxvk-related settings to launcher.ini.
  • Updated CreateCompatToolsInstance to work with new dxvkSettings object.

* removed a few settings from wine tab
* Added dxvk settings to launcher.ini
* Updated CreateCompatToolsInstance to work with new dxvkSettings
* Requires PR#1205 from FFXIVQuickLauncher
* Suggest doing PR#13, #15 on xlcore, for better look and feel.
@rankynbass
Copy link
Contributor Author

Updated to work with latest Dxvk rework changes in the submodule. See PR #1205. I don't know why the updates are showing as commited 5 months ago when I just posted them yesterday.

Copy link
Contributor

@marzent marzent left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just needs to happen after goatcorp/FFXIVQuickLauncher#1205

@Blooym Blooym added the enhancement New feature or request label Jun 16, 2023
@rankynbass rankynbass mentioned this pull request Jun 24, 2023
@rankynbass
Copy link
Contributor Author

Made obsolete by #64

@rankynbass rankynbass closed this Sep 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants