[CWM] Critical mobile UX bugs

[lin-d-hop]

Descriptions

In initial mobile testing a number of critical UX issues have come up. These all need fixing.

• Directory, Search and results don't scroll up quite far enough, leaving some content behind the nav
• Pop-up doesn't fit on the screen well, resulting in weird behaviour and a difficulty to view it, particularly on smaller screens
• Add apply filter button and don't open the results panel until this is clicked. The UX on this needs to be clear ie dont make it so you ahve to scroll to see the button.
• Search results not closing when result is clicked (doing this in [CWM] Centre map position to marker on click (marker or result) #64)

Acceptance Criteria

The above issues are fixed and deployed.

[ms0ur1s]

Task 1: Added 80px padding to the bottom on the content panel (height of the nav), This bug only appears on my phone and I can't recreate it on my machine using dev tools or the browser plugin I have, so it needs live testing.

Task 2: Reduced mobile pop-up height from 500px to 450px.

PR for the first three issues

#73

[rogup]

@ms0ur1s I'll take task 4 and do it as part of my work on #64

[lin-d-hop]

Guys! Impressive!
@rogup @ms0ur1s

[lin-d-hop]

Can I confirm this has been deployed? Still seeing these issues on my phone :(

[ms0ur1s]

@ms0ur1s I'll take task 4 and do it as part of my work on #64

Thanks man ❤

[rogup]

Can I confirm this has been deployed? Still seeing these issues on my phone :(

Yes, deployed. I'm seeing the changes live - mabybe some browser caching on your end?

[ms0ur1s]

Andriod and IOS positional difference for reference

Android Chrome

IOS Safari

IOS Chrome

[ms0ur1s]

PR for the apply filters box positioning bug

[ColmDC]

Popups behaving worse on my mobile now. Always some parts offscreen, and if I attempt to pan the map, they disappear. I am on a Galaxy S8.

[rogup]

@ColmDC I can't repro this. Maybe it was due to the confusion of the earlier deploy not having updated code. Is it still happening for you, and in what location on the map?

[ColmDC]

I'm still seeing it on deploy-2024-11-19-41-g2e5b07b.

[ColmDC]

Popups behaving worse on my mobile now.

I take back statement about behaving worse. There are so many cases. So don't consider this a regression. I think making it possilbe for the user to pan to bring the popup into full view themselves will mean they can always see the data.

In element I made the suggestion

"I'm wondering if we can disable the popup disappearing when panning, as when it does clip, the user can at least manually move the popup into view?"

But I know that behaviour was introduced to solve #78

Rohit suggested then that "Maybe it could temporarily be hidden whilst the map is being panned and then be reshown". This might work as long as it didn't then attempt to recentre once enabled again.

[ColmDC]

#78 (comment)