GEOLYTIX / xyz

An open source javascript framework for spatial data and application interfaces.
MIT License
86 stars 25 forks source link

_default view on mobile devices #1269

Open dbauszus-glx opened 1 month ago

dbauszus-glx commented 1 month ago

There seems to be a couple of issues in regard to the default display on mobile devices.

We can test this via browser stack.

iphone 14 v16 seems to look alright but we need to find a solution for the growing button column display.

image

RobAndrewHurst commented 4 weeks ago

We can mark this as complete?

dbauszus-glx commented 4 weeks ago

No.... The growing button column needs to be addressed. Will change to RFC.

simon-leech commented 4 weeks ago

image

The helpModal does not stay within the viewport of the mapview when you use mobileview.

simon-leech commented 4 weeks ago

image Clicking on a location in the cluster popup does not do anything - it should open that location in the location panel.

simon-leech commented 4 weeks ago

Not sure if it would work but perhaps we need to move the 'default' buttons. zoom in / zoom out / zoom to area / admin / logout to the right of the screen in a separate panel. That would remove 5 buttons from the plugin button panel? Or we need to just add extra buttons next to the others in the mobile view?

dbauszus-glx commented 4 weeks ago

I think the solution needs to involve some form of scrolling. Splitting the control based on the viewport will be hard and just kick the can down the line for another year when there's too many buttons on either side.

simon-leech commented 2 weeks ago

Admin view is rubbish on mobile. Trying to set roles opens up the drop-down off the screen and your keyboard takes up the whole table too.

dbauszus-glx commented 2 weeks ago

@simon-leech Can you create an "admin view on mobile" issue for this?

simon-leech commented 2 weeks ago

@dbauszus-glx deleted the previous comment and moved to this issue for clarity

https://github.com/GEOLYTIX/xyz/issues/1337

simon-leech commented 1 week ago

Also noticed the tab view isn't accessible on mobile. This isn't great as you can click infoj checkboxes that should do something and they do nothing, but it isn't clear to the user.

We need to allow tabview on mobile to resolve

cityremade commented 1 day ago
Screenshot 2024-07-02 at 14 20 53