Skip to content

feat: add Skip Steam Login button to login screen, if user just wants GOG, Epic, Custom Games#531

Merged
utkarshdalal merged 4 commits intoutkarshdalal:masterfrom
xXJSONDeruloXx:feat/skip-steam
Feb 16, 2026
Merged

feat: add Skip Steam Login button to login screen, if user just wants GOG, Epic, Custom Games#531
utkarshdalal merged 4 commits intoutkarshdalal:masterfrom
xXJSONDeruloXx:feat/skip-steam

Conversation

@xXJSONDeruloXx
Copy link
Contributor

@xXJSONDeruloXx xXJSONDeruloXx commented Feb 13, 2026

Now that GN has Epic and GOG support, some users may want to use this app exclusively for those stores, or for custom games, and not necessarily want to log into Steam.

This just adds a non intrusive "Skip Steam Login" button at the bottom of the UserLoginScreen so you can get into the app .

Once in there the "go online" button brings you back to login page, so if user wants to use Steam later on they are not stranded.

There is potentially more work to be done to make steam epic and got all act the same way, but for now this is a small scope change that I think solves a real pain point.


Summary by cubic

Adds a "Skip Steam login" button so users can enter the app for GOG, Epic, or custom games without Steam. Users can return to Steam login later via the "Go online" flow.

  • New Features

    • Show a TextButton when not logging in and login not successful that calls onContinueOffline.
    • Added login_skip_steam string and translations across supported locales.
  • Bug Fixes

    • Reduced padding in landscape QR code view for better readability.

Written for commit 2b03608. Summary will update on new commits.

Summary by CodeRabbit

  • New Features
    • Added a "Skip Steam login" option so users can continue using the app offline during the login process (localized in multiple languages).
  • Bug Fixes
    • The skip option currently appears twice on the login screen (duplicate button visible); UI will be consolidated in a follow-up.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 13, 2026

📝 Walkthrough

Walkthrough

Adds a conditional "Skip Steam login" TextButton to the UserLoginScreen that appears when a login attempt is not in progress and the login result is not success; it calls onContinueOffline. Adds the login_skip_steam string resource and localized variants.

Changes

Cohort / File(s) Summary
Login Screen UI
app/src/main/java/app/gamenative/ui/screen/login/UserLoginScreen.kt
Inserts a conditional skip-Steam UI block (TextButton showing stringResource(R.string.login_skip_steam)) that appears when isLoggingIn.not() && loginResult != Success, invoking onContinueOffline. The block is inserted in two locations.
String Resources (defaults & locales)
app/src/main/res/values/strings.xml, app/src/main/res/values-da/strings.xml, app/src/main/res/values-de/strings.xml, app/src/main/res/values-es/strings.xml, app/src/main/res/values-fr/strings.xml, app/src/main/res/values-it/strings.xml, app/src/main/res/values-pl/strings.xml, app/src/main/res/values-pt-rBR/strings.xml, app/src/main/res/values-ro/strings.xml, app/src/main/res/values-uk/strings.xml, app/src/main/res/values-zh-rCN/strings.xml, app/src/main/res/values-zh-rTW/strings.xml
Adds new string resource login_skip_steam in default and multiple localized resource files with locale-specific translations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 A tiny button gleams in sight,
"Skip Steam" I hum, with hops of light,
When logins stall and clouds roll by,
Tap me, friend — we'll play offline, sly. 🥕

🚥 Pre-merge checks | ✅ 2 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (19 files):

⚔️ app/src/main/java/app/gamenative/db/dao/EpicGameDao.kt (content)
⚔️ app/src/main/java/app/gamenative/db/dao/GOGGameDao.kt (content)
⚔️ app/src/main/java/app/gamenative/service/epic/EpicManager.kt (content)
⚔️ app/src/main/java/app/gamenative/service/epic/EpicService.kt (content)
⚔️ app/src/main/java/app/gamenative/service/gog/GOGManager.kt (content)
⚔️ app/src/main/java/app/gamenative/service/gog/GOGService.kt (content)
⚔️ app/src/main/java/app/gamenative/ui/screen/login/UserLoginScreen.kt (content)
⚔️ app/src/main/res/values-da/strings.xml (content)
⚔️ app/src/main/res/values-de/strings.xml (content)
⚔️ app/src/main/res/values-es/strings.xml (content)
⚔️ app/src/main/res/values-fr/strings.xml (content)
⚔️ app/src/main/res/values-it/strings.xml (content)
⚔️ app/src/main/res/values-pl/strings.xml (content)
⚔️ app/src/main/res/values-pt-rBR/strings.xml (content)
⚔️ app/src/main/res/values-ro/strings.xml (content)
⚔️ app/src/main/res/values-uk/strings.xml (content)
⚔️ app/src/main/res/values-zh-rCN/strings.xml (content)
⚔️ app/src/main/res/values-zh-rTW/strings.xml (content)
⚔️ app/src/main/res/values/strings.xml (content)

These conflicts must be resolved before merging into master.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title directly and accurately describes the main change: adding a 'Skip Steam Login' button to the login screen with context about why (for GOG, Epic, Custom Games users).
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch feat/skip-steam
  • Post resolved changes as copyable diffs in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@app/src/main/java/app/gamenative/ui/screen/login/UserLoginScreen.kt`:
- Around line 404-430: The new "Skip Steam login" button duplicates the existing
"Continue Offline" action when isSteamConnected is false; update the UI to avoid
showing both. Either hide this bottom Column (the block rendering the TextButton
that calls onContinueOffline) when !isSteamConnected, or remove the "Continue
Offline" action from ModernUsernamePassword so only the bottom TextButton
remains; locate the condition around userLoginState.isLoggingIn /
userLoginState.loginResult and the TextButton using onContinueOffline and adjust
the visibility logic accordingly (check isSteamConnected before rendering the
bottom Column or remove the duplicate callback from ModernUsernamePassword).

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@utkarshdalal utkarshdalal merged commit fb58ccf into utkarshdalal:master Feb 16, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants