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

Text panel obscures map and pin on mobile, off-center on desktop #2168

Closed
Shienny1 opened this issue Jul 2, 2024 · 5 comments · Fixed by #2174 or #2194
Closed

Text panel obscures map and pin on mobile, off-center on desktop #2168

Shienny1 opened this issue Jul 2, 2024 · 5 comments · Fixed by #2174 or #2194
Labels
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
Milestone

Comments

@Shienny1
Copy link
Member

Shienny1 commented Jul 2, 2024

severity 5/5
Near impossible to see location on map on mobile devices
Maps are centered as if text panel doesn’t exist leading to obscured or weird alignments. Map should center using its visible area, not the entire screen.

Problem:

On mobile, tapping a result to get details loads org detail view, but layout of split screen obscures map and pin, and pin is off-screen.

Desktop has a similar problem.

To reproduce the desktop version of this bug:

  1. From landing page, enter an address. On desktop, map Pin appears in the 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.
    IMG_1252

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

User goal:

to get see the cross streets of the location indicated on the map so i tap on the icon

my expectation:

To zoom in to that specific location and see more information of the name and hours of operation (maybe covering maximum bottom half, and leave the upper half of the screen on the map showing the selected location zoomed in.

actual result:

80% screen covered by TEXT on location's information etc. Covering the map.
So I swipe down to open the window to map again.
BUT the map window has migrated to a different location, no where close to the selected location.

screenshots and video

Devlasite_Textpanel.on.Map.mov
ScreenRecording_07-19-2024.17-15-01_1.mp4

image

possible solutions

make map larger - see figma designs - I believe it’s supposed to be 50% text at first? — Bryan

Ensure that with new split-screen view that pin is centered in map viewport at first load. This might mean calculating visible map and centering it as text panel is resized.

for example if text is 80% of screen, map is remaining 20% and pin should be at approx 10%.

Ideally the selected pin and the user’s location are both visible on the map at the same time. This will give them a sense of location relative to where they are now.

Stretch goal: if pin is centered, then keep it centered as user resizes text panel.

related issues

@Shienny1 Shienny1 changed the title Bug: ICON on map goes to another part of the map when tapped (Mobile) Bug: Sometime result shows a different zip code from what's entered Jul 3, 2024
@Shienny1 Shienny1 added the Priority - Med Needs to be in the next release label Jul 3, 2024
@fancyham fancyham changed the title Bug: Sometime result shows a different zip code from what's entered Text covers obscured map and pin on mobile Jul 4, 2024
@fancyham
Copy link
Collaborator

fancyham commented Jul 4, 2024

Changed title, added some text, added formatting (using markdown https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet) and inserted the screenshot and video into the issue itself (better for easy reference - drag and drop or attach those media files). Also tagged it for devs to look at.

@fancyham fancyham added Role: Front-end Front End Developer Ready for dev lead Issue ready for dev lead to review Priority - High Production requires hotfix and removed Priority - Med Needs to be in the next release labels Jul 4, 2024
@fancyham fancyham changed the title Text covers obscured map and pin on mobile Text panel obscures map and pin on mobile Jul 4, 2024
@fancyham fancyham added the P-Feature: Map Food seeker map view label Jul 8, 2024
@fancyham
Copy link
Collaborator

fancyham commented Jul 8, 2024

@fancyham
Copy link
Collaborator

The map seems to center itself as if the text panel is at its smallest size — and with the current public and dev versions, the text panel is maximized, completely covering the highlighted pins.

The map it seems to ignore the text panel’s existence — the map should instead calculate its visible area and center itself using that.

@fancyham fancyham added this to the Bug fixes milestone Jul 19, 2024
@fancyham fancyham added the Release Note: Bug Something isn't working (Shows on release notes under "Bug Fixes") label Jul 19, 2024
@fancyham fancyham changed the title Text panel obscures map and pin on mobile Text panel obscures map and pin on mobile, off-center on desktop Jul 20, 2024
@Shienny1
Copy link
Member Author

Hi Guys. I checked the devla.foodoasis.net site : this is what I am experiencing still: The icons are nonresponsive at times. And they are still NOT CENTERED in the selected location in the mobile version. Please the clip attached

FOLA-BUG_.PIN.issue.mp4

@fancyham fancyham reopened this Aug 17, 2024
@fancyham
Copy link
Collaborator

fancyham commented Aug 17, 2024

@Shienny1 could you please add that comment to

I’ve reopened this issue so we don’t lose track of it but I believe it’s the same problem as that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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
Projects
2 participants