-
Notifications
You must be signed in to change notification settings - Fork 35
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 OSM Element Tags widget #2499
✨ Add OSM Element Tags widget #2499
Conversation
35f489a
to
d6d1a11
Compare
c048626
to
e4398af
Compare
4cdc531
to
d4ed1cf
Compare
@CollinBeczak thanks for the comments. I pushed my final update, I will let you review this. Feel free to comment anything. |
0cbb694
to
eace1cd
Compare
eace1cd
to
c835b26
Compare
Looks good to me! The design at the moment works and looks good. |
Thanks for contributing this @homersimpsons! It'll be deployed to production in our next release; I'll let you know when that happens. |
This is now available on maproulette.org 🙂 |
Thank you for noticing me. I tried it out, but I may have made a mistake somewhere because:
|
The change that resolves the issue you are seeing is in this pr: #2510. I will let you know when we are planning to deploy the change when I find out. |
Implements #2453
I based my devlopments on "OSMHistoryWidget", I may not have cleaned up everything yet.
Unresolved questions:
Screenshots
(With original name, not updated to "OSM Element Tags" name)