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

Serious Bug: Map - Mobile map pans incorrectly when viewing an organization #2219

Open
1 of 4 tasks
fancyham opened this issue Sep 17, 2024 · 2 comments
Open
1 of 4 tasks
Assignees
Labels
dev Code Development Missing: Milestone This issue is not part of any milestone P-Feature: Map Food seeker map view Priority - High Production requires hotfix Ready for dev lead Issue ready for dev lead to review Release Note: Bug Something isn't working (Shows on release notes under "Bug Fixes") Role: Front-end Front End Developer size: 3pt The lift to complete this user story 4-8hrs

Comments

@fancyham
Copy link
Collaborator

fancyham commented Sep 17, 2024

VERY HIGH PRIORITY -- HOT FIX REQUESTED.

Progress

(~Sept 15, 2024)
Re-creating this as a new issue per @hanapotski because the solution for

A hint for this: Part of the issue seems to be the map's zoom level. At default zoom, it's not a problem. Changing the zoom causes the problem.

Describe the bug

Serious bug:

When viewing a list of organizations, and tapping one to get details about it, the map does not update show or highlight the organization’s pin. Often the pin is way off-screen.

The map should re-center (ideally with a smooth scroll/pan) on the highlighted pin automatically, ideally showing both the user’s current location and the pin so that they can determine the relative location.

Steps to reproduce the issue

(if applicable, please specify platform (iOS, Android, Windows, Mac version) and brower)

  1. Go to foodoasis.net
  2. Enter a location to get a list of organizations
  3. Scroll around the map, zoom in or zoom out
  4. Tap an organization to view the organization detail page
  5. Map does not update to show the organization’s pin.

See more videos at
-#2028

What's the expected result?

Map should re-center on the organization’s pin automatically, each time a new organization is viewed.

Ideally, the map smoothly pans to the new location, and ideally, the user’s current location / entered address is also visible on the map (though this may be difficult if the user has panned the map manually)

See Apple or Google maps or Yelp on mobile phones for example.

What's the actual result?

The organization’s pin may be highlighted, but is often off-screen with no indication for how to find the pin itself.

This is a regression — the previous version centered on highlighted pins properly.

Additional details / screenshot

A similar problem appears when searching from the map screen.

  1. From landing page, enter an address. On desktop, map Pin appears in the wrong place but map pans correct place at first.
    IMG_1253

  2. Do a search again from the map page and the map now re-centers in the wrong place, and the pin doesn’t move to the correct place.
    IMG_1252

ScreenRecording_07-19-2024.17-15-01_1.mp4

To it might have something to do with loading the page for the first time vs going to a new location

Device configuration

  • Device: iPhone 13 mini
  • OS version: 17.4
  • Browser: Safari

Related issues

When making your PR

Please assign @fancyham and @Shienny1 to PRs so we can review

@fancyham fancyham added Release Note: Bug Something isn't working (Shows on release notes under "Bug Fixes") dev Code Development Role: Front-end Front End Developer P-Feature: Map Food seeker map view Ready for dev lead Issue ready for dev lead to review Priority - High Production requires hotfix labels Sep 17, 2024
@ExperimentsInHonesty ExperimentsInHonesty added the Missing: Milestone This issue is not part of any milestone label Sep 20, 2024
@entrotech entrotech added size: 3pt The lift to complete this user story 4-8hrs and removed Ready for dev lead Issue ready for dev lead to review labels Sep 24, 2024
@hanapotski hanapotski self-assigned this Sep 27, 2024
@fancyham
Copy link
Collaborator Author

fancyham commented Sep 29, 2024

Hi - this is a clever fix for now and if we can push this as a hot-fix, that would be terrific!
@John Darragh That would help all of our users immediately.

Long-term, we should still figure it out a solution such that the zoom stays the same so that weird/frustrating things don't happen (for example, if you look at 0:16 on the video, clicking on a cluster of locations starts an unintended zoom, which prevents the user from clicking on other items in the cluster, because they've disappeared at the default zoom level). We should keep the map's zoom state what the user last chose when zipping between locations for this reason.

Also, side note: I noticed in the video that the first autocomplete entry is 90045 (when you typed 9... the first item was 90045)... I think that's a hint at why we keep jumping to 90045 if the user types in an address and hits return quickly (before autocomplete finishes) 🙂

What should I do since this is a partial solution? Re-open&put-back-in-queue? or recreate? (We shouldn't reset the zoom level when switching between locations - leads to issues with accessing close pins).

If I don't hear back in a few days, I'll re-open, re-tag with 'ready for dev lead' and put it in the prioritized backlog (as there may be other higher priority items)

@fancyham fancyham added the Ready for dev lead Issue ready for dev lead to review label Oct 2, 2024
@fancyham
Copy link
Collaborator Author

fancyham commented Oct 2, 2024

hanapotski: ” I guess in this case, it’s fine to re-open 👌👀”

Reopening! Please do push the interim fix to public as hot-fix ASAP, and next step get it to work while keeping the zoom level.

@fancyham fancyham reopened this Oct 2, 2024
@hanapotski hanapotski assigned whitneywind and unassigned hanapotski Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev Code Development Missing: Milestone This issue is not part of any milestone P-Feature: Map Food seeker map view Priority - High Production requires hotfix Ready for dev lead Issue ready for dev lead to review Release Note: Bug Something isn't working (Shows on release notes under "Bug Fixes") Role: Front-end Front End Developer size: 3pt The lift to complete this user story 4-8hrs
Projects
Status: Prioritized Backlog
Development

No branches or pull requests

6 participants