Skip to content

Icons: add points of interest#963

Merged
wpkelso merged 3 commits intomainfrom
danirabbit/icons-poi
Feb 28, 2026
Merged

Icons: add points of interest#963
wpkelso merged 3 commits intomainfrom
danirabbit/icons-poi

Conversation

@danirabbit
Copy link
Member

@danirabbit danirabbit commented Feb 26, 2026

Adds the poi icons from Maps. Needed if we want to have consistent location completion in Maps, Photos, Tasks, and Calendar. Uses the icon names we get from Geocode.Place

While we're here, put all the category icons in a better path'd resource so that things are a bit more organized

Somewhat in service of #962

@danirabbit danirabbit requested a review from a team February 26, 2026 18:23
Copy link
Member

Choose a reason for hiding this comment

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

I think this should use similar shading to poi-school. It feels much lighter than the rest of the set.

Copy link
Member Author

Choose a reason for hiding this comment

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

Tried to make this a bit more solid

Copy link
Member

Choose a reason for hiding this comment

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

I think that works!

Copy link
Member

@wpkelso wpkelso left a comment

Choose a reason for hiding this comment

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

Everything looks good to me!

Copy link
Member

Choose a reason for hiding this comment

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

I think that works!

@wpkelso wpkelso enabled auto-merge (squash) February 28, 2026 03:32
@wpkelso wpkelso merged commit 9f6456b into main Feb 28, 2026
5 checks passed
@wpkelso wpkelso deleted the danirabbit/icons-poi branch February 28, 2026 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants