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

[Fix] Not able to enable Steam Runtime #1108

Merged

Conversation

imLinguin
Copy link
Member

This is a easy fix.
Closes #1107

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)

@imLinguin imLinguin added the pr:ready-for-review Feature-complete, ready for the grind! :P label Mar 13, 2022
@@ -211,14 +211,14 @@ export default function OtherSettings({
</span>
{isLinuxNative && (
<span className="setting">
<span className={classNames('toggleWrapper', { isRTL: isRTL })}>
<label className={classNames('toggleWrapper', { isRTL: isRTL })}>
<ToggleSwitch
Copy link
Member

Choose a reason for hiding this comment

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

Why changing from a span to a label fixes it?
Also, would be good to check the logic on the backend as well. To be sure it's using the right path.

Copy link
Member Author

Choose a reason for hiding this comment

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

It was about not being able to handle the press event, label is being used in other check boxes as well.

Copy link
Member

Choose a reason for hiding this comment

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

I see, ok then. 👍🏽

@imLinguin imLinguin merged commit 215d892 into Heroic-Games-Launcher:main Mar 14, 2022
@imLinguin imLinguin deleted the fix_steamruntime_checkbox branch March 14, 2022 18:17
dawidgarus pushed a commit to dawidgarus/HeroicGamesLauncher that referenced this pull request Mar 15, 2022
dawidgarus pushed a commit to dawidgarus/HeroicGamesLauncher that referenced this pull request Mar 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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.

Unable to check the steam runtime box (even after setting the path)
2 participants