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

Location search bug #57

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Location search bug #57

merged 1 commit into from
Nov 11, 2024

Conversation

dzole0311
Copy link
Collaborator

Addresses: #53

When searching a location in the app the view changes but after 2 searches the cursor/buffer does not show up as expected until the user does a second click.

Copy link

netlify bot commented Oct 6, 2024

Deploy Preview for eic-mobile-demo ready!

Name Link
🔨 Latest commit fac22f8
🔍 Latest deploy log https://app.netlify.com/sites/eic-mobile-demo/deploys/6702abf97b1e680008de804e
😎 Deploy Preview https://deploy-preview-57--eic-mobile-demo.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 site configuration.

@dzole0311
Copy link
Collaborator Author

The issue still occurs, more details added to the description of the issue.

i tested for a while, didn't see the behavior anymore, and then all of the sudden it started again :man-shrugging::skin-tone-4: . Others were trying it as well. I can't imagine it's dependent on the number of users? (edited)

it just took me 4 location searches to recreate it

this was after a hard refresh

Copy link

@j08lue j08lue left a comment

Choose a reason for hiding this comment

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

I hear from @amarouane-ABDELHAK that this fix may still be relevant. Approving, so we can try it out in staging.

@amarouane-ABDELHAK amarouane-ABDELHAK merged commit 02409d8 into develop Nov 11, 2024
5 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.

3 participants