Repro:
<mapframe text="India (80,20)" width=700 height=400 zoom=3 longitude="0" latitude="0">
{
"type": "Feature",
"geometry": { "type": "Point", "coordinates": [80,20] }
}
</mapframe>After disabling jQuery 3, the map works.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Fix jQuery3 deferred issues | mediawiki/extensions/Kartographer | master | +37 -45 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Krinkle | T127328 Optimise critical rendering path | |||
| Resolved | Krinkle | T124742 Upgrade to jQuery 3 | |||
| Resolved | Krinkle | T168086 jQuery 3 catches all exceptions within promises | |||
| Resolved | Krinkle | T168744 Kartographer is broken by jQuery 3 |
@MaxSem Do you have more information as to why it fails? It is unclear to me what the next action item is here. Should this block enabling jQuery 3 on Beta?
Currently there's very little I can do to move it forward, as enabling it on Beta is the next step to finding any remaining problems.
enabling it on Beta is the next step to finding any remaining problems
You can test it in Vagrant.
Do you have more information as to why it fails?
It emits an error that's cryptic due to T168086: jQuery 3 catches all exceptions within promises, however it's still present without JQ3 when Kartographer works.
FYI, Julien is not working on this stuff anymore, so assigning it to him will do nothing. @Krinkle, I think you can flip the switch on betalabs as long as discovered problems are addressed and the site remains a useful testing platform in the long run.
Same as Karthotherian - noone. I volunteer on occasion when I have some time at my new job, so does Max. A few more amazing individuals have volunteered their dev time to keep it afloat. @Pnorman is working on the new map style and fixes some data-related problems. @Gehel watches over the servers. This is what I have observed by watching the tickets & the public channels.
Change 362420 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/Kartographer@master] Fix jQuery3 deferred issues
Change 362420 merged by jenkins-bot:
[mediawiki/extensions/Kartographer@master] Fix jQuery3 deferred issues
Hi @Esanders, thanks for working on this issue. With the gerrit check-in that you've created, will there be any additional work that needs to be done to make Kartographer compatible with jQuery 3?
I've deployed the jQuery 3 upgrade to Beta cluster and verified verified that the example from this bug report, when saved at https://en.wikipedia.beta.wmflabs.org/w/index.php?title=Sandbox&oldid=363309, works without issue now.
Thanks, @Krinkle - we'll let the patch ride the train and we should be all good sometime during the week of July 11, 2017. :)