Skip to content

fix(mobile): only toggle backup from the switch, not the whole row#29236

Merged
shenlong-tanwen merged 1 commit into
mainfrom
fix/backup-toggle-tap-target
Jun 26, 2026
Merged

fix(mobile): only toggle backup from the switch, not the whole row#29236
shenlong-tanwen merged 1 commit into
mainfrom
fix/backup-toggle-tap-target

Conversation

@santoshakil

Copy link
Copy Markdown
Collaborator

Description

tapping anywhere on the "enable backup" row toggled backup on/off, not just the switch. easy to flip by accident — the reporter hit it reaching for "view details", and it's worse when the row shifts as the upload counters appear.

dropped the row-wide tap so only the switch toggles it now.

fixes #23051

tested on a pixel: tapping the row text used to flip the switch, now it doesn't, and the switch itself still works both ways.

tapping anywhere on the enable backup row flipped backup on or off, so it was easy to toggle by accident. now only the switch does it.
@immich-push-o-matic

Copy link
Copy Markdown

📱 Android release APK (universal)686da0656d57804afdbd905ca6cb9f6aeebb8daf

Download: https://github.com/immich-app/immich/actions/runs/27880076466/artifacts/7767850778

QR code QR code

Installs as a separate app (applicationId app.alextran.immich.pr29236), so it coexists with the Play Store version and any other PR builds.

@shenlong-tanwen shenlong-tanwen merged commit 29949be into main Jun 26, 2026
70 checks passed
@shenlong-tanwen shenlong-tanwen deleted the fix/backup-toggle-tap-target branch June 26, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable backup - tapping the text toggles the toggle

2 participants