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

[General] Offline mode only toggleable if allowed #970

Merged
merged 4 commits into from
Feb 21, 2022

Conversation

Nocccer
Copy link
Collaborator

@Nocccer Nocccer commented Feb 12, 2022

Currently the offline mode option is still present also if the game needs a internet connection.

  • option is now removed if game requires internet connection
  • launching of the game without internet adds the --offline flag if supported else showing a error message box

It seems that some games still need internet, also if they are marked as offline game.
But this is not a heroic issue.


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)

@Nocccer Nocccer added the pr:ready-for-review Feature-complete, ready for the grind! :P label Feb 12, 2022
@Nocccer Nocccer linked an issue Feb 12, 2022 that may be closed by this pull request
flavioislima
flavioislima previously approved these changes Feb 20, 2022
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.

Nice! looks good ⚔️

@Nocccer Nocccer merged commit 4854d09 into main Feb 21, 2022
@Nocccer Nocccer deleted the improve-offline-setting branch February 21, 2022 17:18
flavioislima added a commit that referenced this pull request Mar 2, 2022
…me colours (#964)

* increase touch targets for checkboxes

* increase touch targets for checkboxes

* make gamepad control a bit better, QoL and colours returned to previous on refresh and play buttons

* navigation optimisations

* make wine-mangager dwnld icons orang

* fix colour idiosyncrasies and make exit of gameinstall modal a bit bigger

* border align and varname notes moved

* revert gamepage title colour

* [General] Offline mode only toggleable if allowed (#970)

* Offline mode only setable if allowed.

* i18n and lint

Co-authored-by: Flávio F Lima <flavioislima@gmail.com>

* [Fix] External urls not opening (#994)

* Add installation on MacOS via Homebrew (#992)

* increase touch targets for checkboxes

* move infobox to var

* remove unecessary curlies

* improve cursor changes on hover a LOT

* prettier linting

* #964 (comment)

Co-authored-by: Niklas <61798668+Nocccer@users.noreply.github.com>
Co-authored-by: Flávio F Lima <flavioislima@gmail.com>
Co-authored-by: Paweł Lidwin <62100117+imLinguin@users.noreply.github.com>
Co-authored-by: Depal1 <47154119+Depal1@users.noreply.github.com>
imLinguin added a commit to imLinguin/HeroicGamesLauncher that referenced this pull request Mar 5, 2022
…me colours (Heroic-Games-Launcher#964)

* increase touch targets for checkboxes

* increase touch targets for checkboxes

* make gamepad control a bit better, QoL and colours returned to previous on refresh and play buttons

* navigation optimisations

* make wine-mangager dwnld icons orang

* fix colour idiosyncrasies and make exit of gameinstall modal a bit bigger

* border align and varname notes moved

* revert gamepage title colour

* [General] Offline mode only toggleable if allowed (Heroic-Games-Launcher#970)

* Offline mode only setable if allowed.

* i18n and lint

Co-authored-by: Flávio F Lima <flavioislima@gmail.com>

* [Fix] External urls not opening (Heroic-Games-Launcher#994)

* Add installation on MacOS via Homebrew (Heroic-Games-Launcher#992)

* increase touch targets for checkboxes

* move infobox to var

* remove unecessary curlies

* improve cursor changes on hover a LOT

* prettier linting

* Heroic-Games-Launcher#964 (comment)

Co-authored-by: Niklas <61798668+Nocccer@users.noreply.github.com>
Co-authored-by: Flávio F Lima <flavioislima@gmail.com>
Co-authored-by: Paweł Lidwin <62100117+imLinguin@users.noreply.github.com>
Co-authored-by: Depal1 <47154119+Depal1@users.noreply.github.com>
imLinguin added a commit to imLinguin/HeroicGamesLauncher that referenced this pull request Mar 5, 2022
…me colours (Heroic-Games-Launcher#964)

* increase touch targets for checkboxes

* increase touch targets for checkboxes

* make gamepad control a bit better, QoL and colours returned to previous on refresh and play buttons

* navigation optimisations

* make wine-mangager dwnld icons orang

* fix colour idiosyncrasies and make exit of gameinstall modal a bit bigger

* border align and varname notes moved

* revert gamepage title colour

* [General] Offline mode only toggleable if allowed (Heroic-Games-Launcher#970)

* Offline mode only setable if allowed.

* i18n and lint

Co-authored-by: Flávio F Lima <flavioislima@gmail.com>

* [Fix] External urls not opening (Heroic-Games-Launcher#994)

* Add installation on MacOS via Homebrew (Heroic-Games-Launcher#992)

* increase touch targets for checkboxes

* move infobox to var

* remove unecessary curlies

* improve cursor changes on hover a LOT

* prettier linting

* Heroic-Games-Launcher#964 (comment)

Co-authored-by: Niklas <61798668+Nocccer@users.noreply.github.com>
Co-authored-by: Flávio F Lima <flavioislima@gmail.com>
Co-authored-by: Paweł Lidwin <62100117+imLinguin@users.noreply.github.com>
Co-authored-by: Depal1 <47154119+Depal1@users.noreply.github.com>
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.

[BUG] Offline mode Unknown error.
2 participants