Skip to content

The mobile issue was in the /settings photo control inside src/compon…#90

Merged
sahilkashyap64 merged 1 commit into
mainfrom
updated/GroupDesign
Mar 27, 2026
Merged

The mobile issue was in the /settings photo control inside src/compon…#90
sahilkashyap64 merged 1 commit into
mainfrom
updated/GroupDesign

Conversation

@sahilkashyap64
Copy link
Copy Markdown
Collaborator

…ents/onboarding/OnboardingFlow.jsx. Desktop was reopening the file picker only

because the invisible full-area file input overlapped the Remove photo button; on mobile that overlap didn’t behave the same way.

I changed the button flow to be explicit: Remove photo now clears the current image and immediately calls the hidden file input via a ref, and I
raised the button above the transparent input so taps are handled consistently. That makes desktop and mobile match.

…ents/onboarding/OnboardingFlow.jsx. Desktop was reopening the file picker only

  because the invisible full-area file input overlapped the Remove photo button; on mobile that overlap didn’t behave the same way.

  I changed the button flow to be explicit: Remove photo now clears the current image and immediately calls the hidden file input via a ref, and I
  raised the button above the transparent input so taps are handled consistently. That makes desktop and mobile match.
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 27, 2026

Deploy Preview for ttp-coach-portal-prod ready!

Name Link
🔨 Latest commit f47f464
🔍 Latest deploy log https://app.netlify.com/projects/ttp-coach-portal-prod/deploys/69c6e17f6df6b3000818a992
😎 Deploy Preview https://deploy-preview-90--ttp-coach-portal-prod.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 27, 2026

Deploy Preview for thunderous-muffin-603a62 ready!

Name Link
🔨 Latest commit f47f464
🔍 Latest deploy log https://app.netlify.com/projects/thunderous-muffin-603a62/deploys/69c6e17f429d090008c53e93
😎 Deploy Preview https://deploy-preview-90--thunderous-muffin-603a62.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions
Copy link
Copy Markdown

PR Preview Action v1.6.2

🚀 View preview at
https://CodeMyMobile.github.io/TTP_Coach_Web_Public/pr-preview/pr-90/

Built to branch gh-pages at 2026-03-27 19:59 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@sahilkashyap64 sahilkashyap64 merged commit 66a9bf0 into main Mar 27, 2026
9 checks passed
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.

1 participant