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 changeset description to the popup #278

Open
DaveF63 opened this issue Dec 23, 2016 · 6 comments
Open

Add changeset description to the popup #278

DaveF63 opened this issue Dec 23, 2016 · 6 comments

Comments

@DaveF63
Copy link

DaveF63 commented Dec 23, 2016

Hi
When checking recently edited items in my area I often want to know the reason for the edits. Could the changeset description be added to the popup?

op turbo popup

To compensate so its size isn't increased the blank lines beneath 'Tag' & 'Meta' could be removed & the 'Way' font size could be reduced.

@tyrasd
Copy link
Owner

tyrasd commented Dec 26, 2016

Good idea.

There's one complication, tough: In order to display changeset comments, one would need to fire up additional requests to the main OSM API (since the result from the Overpass API doesn't directly include that information). But it should certainly be doable.

PS: I'm not really concerned about the popup getting too long. ^-^

@HolgerJeromin
Copy link
Contributor

Should the Overpass API request have that information? Perhaps that can be added.

@tyrasd
Copy link
Owner

tyrasd commented Dec 26, 2016

There's a (somewhat) related ticket on the Overpass API repository: drolbr/Overpass-API#189

@mmd-osm
Copy link
Contributor

mmd-osm commented Dec 27, 2016

Should the Overpass API request have that information? Perhaps that can be added.

Given the effort and complexity of adding changeset metadata to Overpass API, I don't expect this to happen anytime soon. That data is not available as part of the minutely diffs, and needs to be fetched from a different source, the data needs to be managed on disk, language extensions defined, implemented and tested.

@HolgerJeromin
Copy link
Contributor

Ah, ok.

@DaveF63
Copy link
Author

DaveF63 commented Jan 10, 2017

PS: I'm not really concerned about the popup getting too long. ^-^

The reason I suggested that was I wouldn't want the boxes to disappear off the visible window such as occurs on sites like Osmose.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants