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

Create a separate package for the icons #149

Open
tordans opened this issue Mar 17, 2023 · 9 comments
Open

Create a separate package for the icons #149

tordans opened this issue Mar 17, 2023 · 9 comments
Assignees
Labels
icon Icon requests and other icon related issues

Comments

@tordans
Copy link

tordans commented Mar 17, 2023

The id-tagging-schema project started using the Röntgen icons and with it several OSM editor projects now have pull the icons and package those that are used in presets as part of their apps.

My understanding is, that it is not easy to import just the icon set. In ideditor/schema-builder#75 (comment) the request was made to have a separate package for just the icons.

//cc @enzet: It would be superbly nice to have a separate versioned package of just the pure icons, in order to use them more easily outside of map-machine. Do you think something like that would be feasible?

I created this issue to give this more prominence and track it more easily.

For more details on the technical side @tyrasd, @bryceco, @westnordost and other id-tagging-schema-consumers can give more insights.

@matkoniecz
Copy link

I tried to use one of icons in iD preset but I am unable to guess its identifier

screen03

From the second row.

@tyrasd
Copy link

tyrasd commented Mar 31, 2023

it's the fort (https://github.com/openstreetmap/iD/blob/develop/svg/roentgen/fort.svg)

@enzet
Copy link
Owner

enzet commented Aug 2, 2023

Sorry guys, I've been out of the project for a while.

@tordans, thank you for creating the issue. I'm glad that the icon set is living his live now outside of the Map Machine project.

@tyrasd, thank you for writing a readme about the icons and for deliberately placing the license. I was thinking about creating a separate repository for Röntgen icons, however, I switched to the new project, iconscript, which makes the icon set even more generated. So, my final plan was to put everything in three separate repositories:

  • Map Machine for the map renderer and mapping scheme,
  • Röntgen for SVG templates and iconscript files,
  • iconscript for the icon renderer.

As for the icons, it was planned to create a separate web page and make them downloadable through web interface as SVG files.

Is this plan suits you or do you want the final SVG files to be in the GitHub repository in order to easily track commits or something like that?

@enzet enzet self-assigned this Aug 2, 2023
@enzet enzet added the icon Icon requests and other icon related issues label Aug 2, 2023
@tordans
Copy link
Author

tordans commented Aug 3, 2023

Is this plan suits you or do you want the final SVG files to be in the GitHub repository in order to easily track commits or something like that?

Generally speaking, a good model for iD/Rapid/GoMap etc. would be to have a Github repo for the icons that follows the structure that https://github.com/rapideditor/temaki uses, especially:

Alternatively, you could look into contributing the icons to temaki and we all use this one place. However, I would start by getting input by the maintainers if some of the icons are not a good fit for temaki; if so, having something separate is easer overall.

@westnordost
Copy link

Wow, looks pretty complex distribution for just some icons.

For StreetComplete (which also uses iD icons), most convenient would be if there was a publicly accessible directory somewhere from which they could be downloaded by name.

@enzet
Copy link
Owner

enzet commented Aug 3, 2023

@tordans Sounds good!

  • A separate repo with icons directory—no problem.
  • I'll definitely use semantic versioning, however I want to mix it with versions for individual icons, so that icons directory may contain, say, ford_v1.svg and ford_v2.svg icons (or something like that).
  • I know almost nothing about npm so I have to research this part. If you can provide me with links to some useful information, this will be great.
  • Web page is definitely on the roadmap. I even have demo version but it is outdated and doesn't have download button.

Contributing to Temaki doesn't seem to me like good idea, because the overall style of Röntgen is slightly different and the process of creating and changing icons is more complex.

@westnordost I definitely overengineered it. 🥲

Yes, there will be a web site with icons as well as a directory in a GitHub repository.

@tordans
Copy link
Author

tordans commented Aug 4, 2023

I know almost nothing about npm …

Me neither but I am told for simple use cases it is straight forward. @tyrasd can maybe give a few hints…

@tordans
Copy link
Author

tordans commented Aug 4, 2023

…however I want to mix it with versions for individual icons, so that icons directory may contain, say, ford_v1.svg and ford_v2.svg icons (or something like that).

Personally I would be happy to alway get the one icon from the current package. Instead of using v1,v2 maybe just add a new icon instead. So there could be "tower.svg" and "tower-windows.svg" for a version that shows windows as well…

@enzet
Copy link
Owner

enzet commented Aug 6, 2023

@tordans Yeah, I thought about it and it seems like semver should deal with this problem. Do I understand it right, that if I want to drastically change the shape of the icon, I should just bump the major version of the whole package and this will be OK?

For example, is it OK if those icons have the same name diving_platform.svg, but come from versions 1.0, 2.0, 2.1 respectivelly?

diving_platform_icons

@enzet enzet moved this to In Progress in Project support Aug 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
icon Icon requests and other icon related issues
Projects
Status: In Progress
Development

No branches or pull requests

5 participants