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

[UX] Default to WineCrossover and DXVK on for Intel macs #4397

Merged
merged 1 commit into from
Mar 9, 2025

Conversation

arielj
Copy link
Collaborator

@arielj arielj commented Mar 8, 2025

Currently, Heroic defaults to using the Game Porting Toolkit for any mac machine, even though it doesn't support Intel mac. Also, because of that default, DXVK is turned off by default.

The default experience for Intel mac users is really poor because Heroic was setting an invalid configuration from the beginning, leading to a lot of support threads.

This PR changes that to default to Wine-Crossover and DXVK turned on when the computer is an Intel mac.


Use the following Checklist if you have changed something on the Backend or Frontend:

  • Tested the feature and it's working on a current and clean install.
  • Tested the main App features and they are still working on a current and clean install. (Login, Install, Play, Uninstall, Move games, etc.)
  • Created / Updated Tests (If necessary)
  • Created / Updated documentation (If necessary)

@arielj arielj added the pr:ready-for-review Feature-complete, ready for the grind! :P label Mar 8, 2025
@arielj arielj requested review from Etaash-mathamsetty, a team, flavioislima, CommandMC, Nocccer and imLinguin and removed request for a team March 8, 2025 14:04
@arielj arielj marked this pull request as ready for review March 8, 2025 14:05
@arielj arielj linked an issue Mar 8, 2025 that may be closed by this pull request

const [repository, setRepository] = useState<WineManagerUISettings>(
isLinux ? protonge : gamePortingToolkit
isLinux ? protonge : isIntelMac ? wineCrossover : gamePortingToolkit
Copy link
Member

Choose a reason for hiding this comment

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

I think those double ternary are not clean, maybe a method for that would be better.

Copy link
Member

@flavioislima flavioislima left a comment

Choose a reason for hiding this comment

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

Looks good, left a small comment

@flavioislima flavioislima merged commit 36ba06a into main Mar 9, 2025
8 of 9 checks passed
@flavioislima flavioislima deleted the intel-mac-wine-crossover-default branch March 9, 2025 18:16
@Heroic-Games-Launcher Heroic-Games-Launcher locked and limited conversation to collaborators Mar 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pr:ready-for-review Feature-complete, ready for the grind! :P
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use Wine-Crossover as default for Intel Mac instead of GPTK
2 participants