OneBusAway / onebusaway-ios

OneBusAway for iOS, written in Swift.
Other
89 stars 35 forks source link

small UI problems with route labels on stop icons on map #132

Open alanb2718 opened 4 years ago

alanb2718 commented 4 years ago

The route labels on the stop icons on the map are great, but there are a couple of UI bugs as of 0c8e4795c79fb9c65fdf836bbd6b1a6abc71800f.

If there are too many routes to display under the stop icon, the "..." is not always shown. See screenshot with the 3rd Ave & Pine St (NW) stop. I got confused by this trying to find that stop last night, since it had routes under it but not the 62 that I was looking for. IMG_3087

The route labels are omitted if zoomed out, but they should be omitted a little more aggressively -- right now they can overlap other routes. See 2nd screenshot. IMG_3090

ualch9 commented 4 years ago

We could hide labels based on if the annotation views are overlapping each other.

I also played around with clustering. It's a big change compared to just hiding labels.

feedback welcome

aaronbrethorst commented 4 years ago

I’ve played around with clustering before, and I’ve never been able to get it work satisfactorily in OBA. Stops get merged too aggressively and it ends up being a real usability impediment.

I’m much more open to hiding labels. I’ve played around with this too, and was also unable to get it to work in a way that made me happy about its usability, but I’d be delighted to see you crack it, Alan.

ualch9 commented 3 years ago

Simulator Screen Shot - iPhone 12 Pro - 2021-06-29 at 09 26 37

Although visually overlapping, this doesn't hinder functionality. mapkit lets you select both, selecting one on the first tap, then selecting the other on the second tap.

Attached video:

https://user-images.githubusercontent.com/22162410/123834791-a8a88600-d8bc-11eb-807c-25dbf145c0d7.mov

Stops 375, 2349.