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

Unneccesary underscores added to source #4474

Closed
hikemaniac opened this issue Oct 26, 2017 · 6 comments
Closed

Unneccesary underscores added to source #4474

hikemaniac opened this issue Oct 26, 2017 · 6 comments
Labels
bug A bug - let's fix this! good first issue Best for first-time contributors. No experience necessary!

Comments

@hikemaniac
Copy link
Contributor

When writing a changeset comment and adding a source, iD will automatically add underscores to the source tag. I believe that this is actually an unneccesary feature; most editors (e.g.: JOSM) will apply the source tag without the underscores and the wiki says nothing about it. As this is bad for the readability, can we remove this feature?

source

source field

@bhousel
Copy link
Member

bhousel commented Oct 26, 2017

Thanks for reporting, I agree..

@bhousel bhousel added bug A bug - let's fix this! good first issue Best for first-time contributors. No experience necessary! Hacktoberfest labels Oct 26, 2017
@octagonal
Copy link

I'll have a look

@bhousel
Copy link
Member

bhousel commented Oct 26, 2017

@octagonal this is an easy one!

  • open data/presets/fields/source.json
  • add the property "snake_case": false
  • npm run build

This should tell iD to not stick underscores in the value for this field.

@octagonal
Copy link

Thanks @bhousel! This is a really delightful app though, I'm glad I came across it!
Do you have any other good first issue type issues?

@bhousel
Copy link
Member

bhousel commented Oct 26, 2017

Thanks @bhousel! This is a really delightful app though, I'm glad I came across it!
Do you have any other good first issue type issues?

Thanks for the kind words @octagonal!

With Hacktoberfest going on, people tend to close the good first issues really quickly. Most of the issues involving "presets" are fairly straightforward, because the presets in iD are just stored in .json files. If you are following the project you will see a few of these appear every week. I'll add some info to #4441 and you can do that one!

Another great thing to do would be to visit https://openstreetmap.org and do some mapping with the iD editor. Find a place on OpenStreetMap that you are familiar with, click "Edit", go through the tutorial, and start improving the map around there. You can also visit http://tasks.hotosm.org/ if you want to work on mapping for a humanitarian purpose. You won't get Hacktoberfest credit, but it's a great way to get familiar with the editor, if getting deeper into the code is something you are interested in! We'd love the help ❤️

@octagonal
Copy link

octagonal commented Oct 27, 2017

@bhousel thanks for the pointers on how to get started! I'll have a look at #4441 this weekend.

if getting deeper into the code is something you are interested in!

I am indeed. Especially the map display and tooling part seems like a fun and challenging problem space. (Just for reference, I have a few years of coding experience and want to start contributing much more to open source projects 😄)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug - let's fix this! good first issue Best for first-time contributors. No experience necessary!
Projects
None yet
Development

No branches or pull requests

3 participants