Skip to content

Conversation

@TheSecurityDev
Copy link
Contributor

After connecting to a WiFi network during onboarding, this PR makes it automatically go back to the network setup page so the user can continue to downloading, instead of remaining in the WiFi page.

This was based on user feedback in Discord about how the current flow was confusing. Also partly addresses #36916

When the user connects...
image

... it will now go back to here:
image

Copilot AI review requested due to automatic review settings January 23, 2026 01:47
@github-actions github-actions bot added the ui label Jan 23, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Jan 23, 2026

mici raylib UI Preview

✅ Videos are identical! View Diff Report

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the onboarding flow by automatically navigating back to the network setup page after successfully connecting to a WiFi network, addressing user feedback about the confusing WiFi connection flow during device setup.

Changes:

  • Added a back_on_activate parameter to WifiUIMici to enable automatic navigation after network connection
  • Modified the onboarding flow to use this new feature while keeping the settings flow unchanged

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
system/ui/mici_setup.py Enables automatic back navigation during onboarding by passing back_on_activate=True to WifiUIMici
selfdrive/ui/mici/layouts/settings/network/wifi_ui.py Implements the back_on_activate feature by closing modal overlays and triggering the back callback when a network connection is activated

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 23, 2026

raylib UI Preview

All Screenshots

@sshane
Copy link
Contributor

sshane commented Jan 23, 2026

I thought I already added this, did it get reverted?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants