-
-
Notifications
You must be signed in to change notification settings - Fork 52
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
Bug: Sometimes the map shows incorrect area after typing a zipcode into search bar on landing page. #2186
Comments
@Shienny1 -- FYI, this is based on your bug report from this usability review spreadsheet: https://docs.google.com/spreadsheets/d/1dDZzrwRvulR-TyGP2Gj_NWhS5AzK_drQ5X3xpnuqOFY/edit?gid=0#gid=0 |
@Shienny1 Hi Shienny, I wanted to reach out to you since I am not able to reproduce this error. Maybe it got fixed as part of some other PR but just wanted to check how frequent did you notice the error, I tried around 10-15 times. Also, did the error appear on click or on pressing the enter key? I have noticed an error related to enter key but that may not be exactly related to this issue. Looking forward to hearing from you. Thanks. |
I was able to reproduce it right now on the public site and on devla on desktop by:
My guess is that whatever algorithm is powering the autocomplete, when the user hits the return key and doesn't select an auto-complete address, the autocomplete fails in unpredictable ways. Video with audio: Screen.Recording.2024-09-14.at.1.24.46.PM.mov |
@SAUMILDHANKAR Yes this only happens on desktop, because of the speed of hitting "ENTER", just like @fancyham mentioned. I tried it on my mobile devices - this problem does not occur. |
Thanks Bryan, Shienny for your help. I am able to reproduce the issue. I am looking into it and would share any update asap. |
Reopening this issue since the PR #2223 solution didn't seem to work (as discussed in the meeting today). I will work on reverting the changes of the PR next. |
Added this comment to the proposed solution above, PR #2235: Hi, this (the video above) is definitely an improvement!… but now feels broken in a different way :) The user quickly typing 90015 and hitting return… that should be a valid input, so us going to the map and showing no address in the field (as in the video) feels broken but in a different way. Like the user entered something and we ignored it. (I can see in the URL that the 90015 is in the request, too… that should be a valid input, no?) So, this is a tough piece as you can see from past issues around this: https://github.com/hackforla/food-oasis/issues?q=is%3Aissue+autocomplete+is%3Aclosed The desired flow is in this issue (it was iceboxed and then I closed it but now realize that I should have left it open) Describes how we should try to resolve what they typed in - perhaps autocomplete happens silently before displaying the map, if needed… and if a match isn’t found, then we display an error or ask user to disambiguate. So…
Might that be possible? Let me know if a sketch would be helpful. Btw, the video you made was very helpful! |
@hanapotski @Shienny1 Please provide update
|
Hi @MuthamilselvanG 👋 I haven't really taken a look at this much as I've been working on a different branch and helping John with conversion to vite but I might have some bandwidth to look into this this week 👌 |
Describe the bug
Sometimes the map shows incorrect area after typing a zipcode into search bar on landing page.
This happens only sometimes!
Steps to reproduce the issue
In this video (turn on audio), taken at the beginning of July 2024:
Devlasite_searchbarbug.mp4
The text was updated successfully, but these errors were encountered: