Page MenuHomePhabricator
Create Task
Maniphest T58612

Add map widget for coordinate selection
Open, LowPublicFeature
Actions

Description

UploadWizard contains a coordinate field, but leaves it to the user to figure out the coordinates. A map widget where they can just click at the right point would make the task of adding a location much easier.

This map could also be used to suggest coordinates, especially in image contests which use a fixed set of locations (and uploaders arrive through a campaign link) - in such cases the available locations could be shown on the map, along with their descriptions, and the user could be forced to choose one of them.

(This is a potential OPW project: https://www.mediawiki.org/wiki/Outreach_Program_for_Women/Round_7#UploadWizard:_OSM_map_embedding )

Version: unspecified
Severity: enhancement

Details

Reference
bz56612
SubjectRepoBranchLines +/-
[WIP] Embedding Static Map to UploadWizardmediawiki/extensions/UploadWizardosm3+97 -33
Customize query in gerrit

Related Objects

Event Timeline

• bzimport raised the priority of this task from to Low.Nov 22 2014, 2:40 AM
• bzimport added a project: UploadWizard.
• bzimport set Reference to bz56612.
Tgr created this task.Nov 5 2013, 9:43 AM
gerritbot added a comment.Nov 10 2013, 7:22 AM

Change 94587 had a related patch set uploaded by Nemo bis:
OPW Microtask:adding button to show osm map

https://gerrit.wikimedia.org/r/94587

gerritbot added a comment.Nov 17 2013, 9:40 PM

Change 94587 merged by jenkins-bot:
Introducing a button to show coordinates in OSM

https://gerrit.wikimedia.org/r/94587

Aklapper added a comment.Feb 12 2014, 4:22 PM

Patch was merged a while ago - is there more work left to do here (if yes: please reset the bug report status to NEW or ASSIGNED), or can you close this ticket as RESOLVED FIXED?

gerritbot added a comment.Feb 12 2014, 5:55 PM

Change 107892 had a related patch set uploaded by Gergő Tisza:
Embedding Static Map to UploadWizard

https://gerrit.wikimedia.org/r/107892

gerritbot added a comment.Feb 12 2014, 5:56 PM

Change 109063 had a related patch set uploaded by Gergő Tisza:
Adding Leaflet scripts to UploadWizard

https://gerrit.wikimedia.org/r/109063

gerritbot added a comment.Feb 12 2014, 5:56 PM

Change 109263 had a related patch set uploaded by Gergő Tisza:
Embedding leaflet maps based on OSM data to UploadWizard

https://gerrit.wikimedia.org/r/109263

gerritbot added a comment.Feb 28 2014, 6:08 PM

Change 116127 had a related patch set uploaded by Inchikutty:
Embedding Static Map to UploadWizard

https://gerrit.wikimedia.org/r/116127

gerritbot added a comment.Feb 28 2014, 6:08 PM

Change 116128 had a related patch set uploaded by Inchikutty:
Adding Leaflet scripts to UploadWizard

https://gerrit.wikimedia.org/r/116128

gerritbot added a comment.Feb 28 2014, 6:08 PM

Change 116129 had a related patch set uploaded by Inchikutty:
Embedding leaflet maps based on OSM data to UploadWizard

https://gerrit.wikimedia.org/r/116129

gerritbot added a comment.Feb 28 2014, 11:18 PM

Change 116127 merged by jenkins-bot:
Embedding Static Map to UploadWizard

https://gerrit.wikimedia.org/r/116127

gerritbot added a comment.Feb 28 2014, 11:37 PM

Change 109063 abandoned by Gergő Tisza:
Adding Leaflet scripts to UploadWizard

Reason:
Abandoned, development will continue on a branch:
https://gerrit.wikimedia.org/r/#/c/116128/

https://gerrit.wikimedia.org/r/109063

gerritbot added a comment.Feb 28 2014, 11:39 PM

Change 107892 abandoned by Gergő Tisza:
Embedding Static Map to UploadWizard

https://gerrit.wikimedia.org/r/107892

gerritbot added a comment.Mar 3 2014, 1:57 AM

Change 116128 merged by jenkins-bot:
Adding Leaflet scripts to UploadWizard

https://gerrit.wikimedia.org/r/116128

gerritbot added a comment.Mar 7 2014, 8:21 AM

Change 116130 had a related patch set uploaded by Nemo bis:
UploadWizard: OSM embedding: Geocoding, reverse geocoding

https://gerrit.wikimedia.org/r/116130

gerritbot added a comment.Mar 10 2014, 9:09 AM

Change 116129 merged by jenkins-bot:
Embedding leaflet maps based on OSM data to UploadWizard

https://gerrit.wikimedia.org/r/116129

gerritbot added a comment.Mar 10 2014, 10:18 AM

Change 109263 abandoned by Inchikutty:
Embedding leaflet maps based on OSM data to UploadWizard

Reason:
Abandoning,as further development is done on a branch:
https://gerrit.wikimedia.org/r/#/c/116129/

https://gerrit.wikimedia.org/r/109263

gerritbot added a comment.Mar 10 2014, 7:57 PM

Change 116130 merged by jenkins-bot:
UploadWizard: OSM embedding: Geocoding, reverse geocoding

https://gerrit.wikimedia.org/r/116130

gerritbot added a comment.Mar 16 2014, 4:48 PM

Change 118978 had a related patch set uploaded by Inchikutty:
Improving zoom of embedded map, on address search[WIP]

https://gerrit.wikimedia.org/r/118978

gerritbot added a comment.Apr 4 2014, 11:23 PM

Change 118978 merged by jenkins-bot:
Improving zoom of embedded map, on address search

https://gerrit.wikimedia.org/r/118978

gerritbot added a comment.Jun 25 2014, 10:27 AM

Change 141920 had a related patch set uploaded by Inchikutty:
Embedding Static Map to UploadWizard

https://gerrit.wikimedia.org/r/141920

gerritbot added a comment.Jun 25 2014, 3:56 PM

Change 141979 had a related patch set uploaded by Inchikutty:
Adds leaflet scripts to UploadWizard

https://gerrit.wikimedia.org/r/141979

gerritbot added a comment.Jun 25 2014, 4:02 PM

Change 141987 had a related patch set uploaded by Inchikutty:
Embeds leaflet map based on OSM data to UploadWizard

https://gerrit.wikimedia.org/r/141987

gerritbot added a comment.Jun 26 2014, 6:24 PM

Change 142301 had a related patch set uploaded by Inchikutty:
UploadWizard: OSM embedding: Geocoding, reverse geocoding

https://gerrit.wikimedia.org/r/142301

Aklapper added a comment.Aug 17 2014, 11:22 AM

Status here: 4 patches by Anu/Inchikutty still awaiting review...

gerritbot added a comment.Aug 20 2014, 2:06 PM

Change 144324 had a related patch set uploaded by Inchikutty:
Adds OSM parameter to coordinate template

https://gerrit.wikimedia.org/r/144324

gerritbot added a comment.Aug 21 2014, 9:51 AM

Change 155511 had a related patch set uploaded by Inchikutty:
[WIP]Add unit tests to mw.GeoMap.js and some fixes

https://gerrit.wikimedia.org/r/155511

• Gilles added a project: Multimedia.Nov 24 2014, 3:11 PM
gerritbot added a comment.Jan 14 2015, 10:37 AM

Change 184869 had a related patch set uploaded (by Inchikutty):
[WIP] Embedding Static Map to UploadWizard

https://gerrit.wikimedia.org/r/184869

Patch-For-Review

Jdforrester-WMF moved this task from Untriaged to Backlog on the Multimedia board.Sep 4 2015, 6:12 PM
Restricted Application added subscribers: Steinsplitter, Matanya. · View Herald TranscriptSep 4 2015, 6:12 PM
Aklapper added a project: Discovery-ARCHIVED.Oct 2 2015, 11:26 AM

The patches by Anu/Inchikutty are still awaiting review and are the oldest unreviewed patches for UploadWiazrd.

Are there plans to move this forward? Could the Discovery-ARCHIVED /Maps provide input on / review of these patches and the approach?

Tgr added a comment.Oct 2 2015, 6:46 PM

I plan to review it once the map service is enabled on Commons. Legal issues around using an external service were probably the most problematic part of the project.

matmarex mentioned this in T119764: Clean the code review queue of extensions/UploadWizard.Nov 27 2015, 9:27 PM
gerritbot subscribed.Dec 1 2015, 5:34 PM

Change 184869 abandoned by MarkTraceur:
[WIP] Embedding Static Map to UploadWizard

Reason:
These are going away for now, revive these if someone is available to work on them (or if you decide to rebuild them from scratch, that's cool too)

https://gerrit.wikimedia.org/r/184869

EBernhardson moved this task from Needs triage to Maps on the Discovery-ARCHIVED board.Feb 11 2016, 11:26 PM
Aklapper added a comment.Jun 17 2016, 9:01 AM
In T58612#1698124, @Tgr wrote:

I plan to review it once the map service is enabled on Commons.

@Tgr: That time has come, it seems. :)

jeremyb added subscribers: jeremyb, Yurik.Oct 17 2016, 2:12 AM
jeremyb claimed this task.Oct 18 2016, 4:32 AM
jeremyb added a subscriber: Inchikutty.
Yurik added a comment.EditedOct 18 2016, 5:58 AM

The patch should be implemented using Kartographer API
Update: I meant that we have recently launched the Kartographer API, and the pending patch should be updated to take advantage of that. We must not embed resources from external sites such as staticmap.openstreetmap.de into production code.

Yurik added projects: Maps, Maps-Sprint.Oct 19 2016, 9:17 PM
Yurik set Security to None.
Nikki subscribed.Nov 6 2016, 10:07 PM
MaxSem added a subscriber: Bmueller.Jan 5 2017, 7:02 PM
MarkTraceur unsubscribed.Jan 27 2017, 10:22 PM
• Deskana removed a project: Maps-Sprint.Feb 3 2017, 6:40 PM
• Jhernandez removed a project: Discovery-ARCHIVED.Jul 10 2018, 4:40 PM
• Mholloway moved this task from All map-related tasks to Tracking on the Maps board.Aug 28 2018, 3:19 PM
Tgr mentioned this in T218057: Determine workflow to selectively purge potentially privacy-sensitive EXIF fields, such as geocoordinates, from a Wikimedia Commons file.Mar 12 2019, 4:37 AM
Aklapper added a comment.Jan 3 2020, 12:28 AM

@jeremyb: Hi! This task has been assigned to you a while ago. Could you maybe share an update? Do you still plan to work on this task, or do you need any help?

If this task has been resolved in the meantime: Please update the task status (via Add Action... → Change Status in the dropdown menu).
If this task is not resolved and only if you do not plan to work on this task anymore: Please consider removing yourself as assignee (via Add Action... → Assign / Claim in the dropdown menu): That would allow others to work on this (in theory), as others won't think that someone is already working on this. Thanks! :)

Aklapper removed jeremyb as the assignee of this task.Feb 8 2020, 12:32 PM
Aklapper removed a project: Patch-For-Review.

@jeremyb: I am resetting the assignee of this task because there has not been progress lately (please correct me if I am wrong!).
Resetting the assignee avoids the impression that somebody is already working on this task. It also allows others to potentially work towards fixing this task.
Please claim this task again when you plan to work on it (via Add Action... → Assign / Claim in the dropdown menu) - it would be welcome! Thanks for your understanding!

Aklapper changed the subtype of this task from "Task" to "Feature Request".Feb 4 2022, 11:13 AM
Aklapper removed subscribers: • LuisV_WMF, • Fabrice_Florin.
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL