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

Add bus and rail POIs for stops and stations #845

Merged
merged 6 commits into from
Apr 6, 2023
Merged

Conversation

Pengor
Copy link
Member

@Pengor Pengor commented Mar 30, 2023

Description

This adds bus stops/stations and rail (including subway and tram) stops/stations. I think there are a lot of directions we could go with the design of the icons for public transportation but a simple head-on view of a bus and a train made sense to me as being both recognizable and consistent with iconography used by various transportation agencies.

I tried to keep the two icons differentiated enough to be recognizable, but the nature of them both being head-on views of a larger vehicle makes them inherently look rather similar.

In the future I could see additional icons for trams/streetcars and subways, but in many places in North America the distinction between light rail, subways, and trams isn't always crystal clear to those unfamiliar with a transit system, so I don't think it's necessary at this point in time. If we eventually have clickable POIs then that info could alternatively be surfaced at that point.

Addresses #436 and #790.

I went with making these appear starting at z16 as the density of bus stops and train stations is approximately equivalent to the density of cafe and bar POIs in downtown areas I referenced while working on this (Seattle and NYC).

Renders

Before (1x display density):

transit_1x_before

After (1x display density):

transit_1x_after

Before (2x display density):

transit_2x_before

After (2x display density):

transit_2x_after

Fun fact: Third Avenue in Seattle (shown in these renders) is supposedly the busiest bus corridor in North America!

@Pengor Pengor added enhancement New feature or request points of interest labels Mar 30, 2023
@Pengor Pengor self-assigned this Mar 30, 2023
This was linked to issues Mar 30, 2023
src/layer/poi.js Outdated Show resolved Hide resolved
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We'll need to find a way to stylistically distinguish stations from stops in both bus and railway features. OsmAnd does this by giving stations (e.g. bus stations, light rail stations, train stations) circular icons, and stops (e.g. bus stops, tram stops) square icons.

I think the bus icon is fine as it is, and we could adopt a similar backgroundless style for tram stops. But I'd like to see bus stations visually distinguished somehow, perhaps with a circle background, and we can apply that same visual element to railway stations.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree that this would be a good feature (unsurprisingly, since I mentioned a similar tack in https://github.com/ZeLonewolf/openstreetmap-americana/issues/790#issuecomment-1475432421).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this makes sense, I've drawn up some circle-enclosed icons and will commit these alongside changes to render icons by subclass instead of class:

poi_bus_circle
poi_rail_circle

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in 8b7012b

@wmisener
Copy link
Collaborator

Thanks for putting this together! I think the icons are nice: those bus headlights are quite stylish, and while I think down the road it would be best to distinguish light rail, subways, and conventional rail, this icon is nicely ambiguous for the time being IMO.

My first impression from your screenshots is that the bus icons seem quite large. They have a lot of visual weight in my eyes, which I'm not sure matches their importance (and this comes from someone who takes the bus every day to work!). I wonder if it would be better if the bus icons were smaller than 20x20px, so they'd be smaller than the cafes and whatnot. I'm not sure this style has ever done that before so maybe that needs some discussion as a general stylistic choice. I think train stations can stay the same size as they are, though as Clay suggested it might be good to distinguish tram stops from other stations by inscribing the icon in a circle for stations.

@Pengor Pengor force-pushed the pengor-transit-pois branch from 759f77b to 8b7012b Compare April 5, 2023 05:47
@Pengor
Copy link
Member Author

Pengor commented Apr 5, 2023

Updated now that separate POI icons can be rendered by subclass. There are icons for bus and rail stations enclosed in a circle. I think these are sufficiently different looking from one another but happy to incorporate additional feedback!

Examples

Light rail stations, bus stops, and streetcar stops:
Downtown_Seattle_Transit

Bus station and light rail station:
Bus_and_train_stations

Bus station and bus stops:
Bus_stops_and_station

@claysmalley
Copy link
Member

Looking pretty good altogether.

Just a nitpick—I think the bus and tram icons would look a little better if they were shrunk to match the size of the inner part of the station icons. Otherwise the stops kind of visually dominate the stations, the opposite of what I'd expect.

@Pengor
Copy link
Member Author

Pengor commented Apr 5, 2023

Shrank the "stop" icons for bus and rail to match the size of the inner parts of the circular icons:

image

image

Copy link
Member

@claysmalley claysmalley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

Screenshot from 2023-04-06 10-05-13

@Pengor Pengor merged commit 4de5a78 into main Apr 6, 2023
@Pengor Pengor deleted the pengor-transit-pois branch April 6, 2023 14:12
This was referenced Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request points of interest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Render railway stations Bus stops
3 participants