Last modified: 2014-11-17 10:35:15 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T40271, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 38271 - Review and deploy SVGEdit extension to Wikimedia wikis
Review and deploy SVGEdit extension to Wikimedia wikis
Status: NEW
Product: Wikimedia
Classification: Unclassified
Extension setup (Other open bugs)
unspecified
All All
: Normal enhancement with 7 votes (vote)
: ---
Assigned To: Brion Vibber
https://www.mediawiki.org/wiki/Extens...
:
: 62562 (view as bug list)
Depends on:
Blocks: svg 12603 31235 commons 54213 54221
  Show dependency treegraph
 
Reported: 2012-07-10 00:27 UTC by MZMcBride
Modified: 2014-11-17 10:35 UTC (History)
18 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description MZMcBride 2012-07-10 00:27:11 UTC
This is a follow-up to bug 5899 ("Allow on-wiki editing of SVG images").

Please review and deploy the SVGEdit extension (<https://www.mediawiki.org/wiki/Extension:SVGEdit>) to Wikimedia wikis.
Comment 1 Sam Reed (reedy) 2012-07-10 08:36:03 UTC
Extension requires moving to Git
Comment 2 MZMcBride 2012-07-10 20:31:53 UTC
(In reply to comment #1)
> Extension requires moving to Git

Okay, copying Chad about that.

A demonstration is currently available here: <https://toolserver.org/~brion/svg-edit/svg-editor.html>. Looks pretty neat!
Comment 3 Nemo 2012-07-27 08:34:36 UTC
(In reply to comment #2)
> (In reply to comment #1)
> > Extension requires moving to Git
> 
> Okay, copying Chad about that.

Chad, is this in the queue?
Also cc'ing Harry because I don't understand how this would interact with TranslateSVG and he probably has some insights to share.
Comment 4 Jarry1250 2012-07-27 10:05:30 UTC
Well, interaction with TranslateSvg *per se* shouldn't be a problem, the issue is really its (by which I mean SVGEdit and not the extension) support for certain parts of the SVG1.1 specification on which TranslateSvg relies (namely <switch> tags).

At the moment, if you dump in a <switch> tag, it will render it correctly, which is nice. However, the text is not editable, and the thing isn't movable.

If you add a group around it, it becomes movable.

In other words, I think this is a fixable issue - SVGEdit *can* handle <switch>, it just doesn't think it can.

SVGEdit is written in JS, right? In which case, I could probably take a look myself at some point.
Comment 5 Chad H. 2012-07-27 10:50:44 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > (In reply to comment #1)
> > > Extension requires moving to Git
> > 
> > Okay, copying Chad about that.
> 
> Chad, is this in the queue?

It's already been done in the last round of moves.

https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/SVGEdit.git

(Removing myself from the CC list again since the part I need to do is done and I don't really care about SVGs)
Comment 6 Sumana Harihareswara 2012-08-30 17:29:11 UTC
Harry (Jarry1250), do you have time to look at this soon?  Thanks.
Comment 7 Jarry1250 2012-08-31 22:59:51 UTC
(In reply to comment #6)
> Harry (Jarry1250), do you have time to look at this soon?  Thanks.

Yeah, I can probably take a look tomorrow. I'd be more certain except someone tried to SQL-inject one of my websites today.
Comment 8 Jarry1250 2012-09-01 15:40:59 UTC
Upstream bug filed: https://code.google.com/p/svg-edit/issues/detail?id=1006 . Probably shouldn't block this, except perhaps from the perspective of svg-edit "sanitizing" an SVG and thereby breaking it, which could be confusing for the user.
Comment 9 Sumana Harihareswara 2012-09-17 03:16:01 UTC
Since this is user-facing, then it looks like SVGEdit awaits a design review https://www.mediawiki.org/wiki/WMF_Project_Design_Review_Process before we can move forward with deploying it on Wikimedia sites.  Would Harry, Brion, or someone else like to lead that?

Thanks.
Comment 10 MZMcBride 2012-09-17 03:36:32 UTC
(In reply to comment #9)
> Since this is user-facing, then it looks like SVGEdit awaits a design review
> https://www.mediawiki.org/wiki/WMF_Project_Design_Review_Process before we can
> move forward with deploying it on Wikimedia sites.  Would Harry, Brion, or
> someone else like to lead that?

You're citing a draft of a page that was written in a single edit in April 2011 (about sixteen months ago). I find it difficult to believe that the page is authoritative or that anyone intended it to be.
Comment 11 Sumana Harihareswara 2012-09-17 03:40:53 UTC
I'm inferring from http://lists.wikimedia.org/pipermail/design/2012-September/000138.html that Brandon wants folks to follow the process as outlined at https://www.mediawiki.org/wiki/WMF_Project_Design_Review_Process -- cc'ing Brandon to check.
Comment 12 Marc Laporte 2012-11-16 22:45:14 UTC
December 4th, 2012, there will be a community conference call. One of the topics is "SVG-edit on Wikipedia: What are the steps to get there?"

Looking forward to it:
https://code.google.com/p/svg-edit/wiki/CommunityConferenceCall#Wikipedia

Thanks!

Marc Laporte
SVG-edit
Comment 13 Nemo 2012-11-16 22:49:07 UTC
(In reply to comment #12)
> December 4th, 2012, there will be a community conference call. One of the
> topics is "SVG-edit on Wikipedia: What are the steps to get there?"

You should probably announce it on [[mw:Meetings]] etc.
Comment 14 Brion Vibber 2012-11-16 22:49:40 UTC
I'll be on the call, unless my jury duty interferes. :P
Comment 15 Marc Laporte 2012-11-16 23:02:00 UTC
(In reply to comment #13)
> (In reply to comment #12)
> > December 4th, 2012, there will be a community conference call. One of the
> > topics is "SVG-edit on Wikipedia: What are the steps to get there?"
> 
> You should probably announce it on [[mw:Meetings]] etc.

Done
https://www.mediawiki.org/wiki/Meetings/2012-12-05
Comment 16 Jarry1250 2012-12-04 17:53:25 UTC
Okay, so I was on the call today. What I took away from:
* svg-edit has a large body of diverse users and contributors. There are many use cases out there.
* many users have to hack the UI to get it a little more user friendly, but there does seem to be progress in this area. For example, CWM-tool is an open source attempt make a very simple GUI
* another caller, Emmanuel, said he was going to be trying out the existing extension for his new project.

In short, it's a very cool project. For my money, the choice is between jumping in now or jumping in later. I don't think it's very practical at present to have us reskinning--as some of the callers said, they were scared that because they'd hacked it, if they tried to update all they'd end up with was a horrible mess. Updatability > skinning for us, I suspect, especially if better skinning support comes soon.

Overall, I think we ought to consider a Labs deployment for svg-edit, and I'll look into getting that bug with <switch> support fixed.
Comment 17 Marc Laporte 2012-12-05 00:48:08 UTC
Hi!

Thanks for being on the call today. We had over 30 people, and the meeting lasted over 3 hours!

Notes and a link to the recording are here:
https://code.google.com/p/svg-edit/wiki/CommunityConferenceCall

Harry and Brion have commit access. We look forward to fixes for anything blocking progress towards this high-priority goal.

Best regards,

M ;-)
Comment 18 Waldir 2013-01-17 01:00:00 UTC
(In reply to comment #16)
> For example, CWM-tool is an open source attempt make a very simple GUI
TO save someone else the googling: the name is actually CWM Draw Tool: http://code.google.com/p/cwm-drawtool/
Comment 19 Marc Laporte 2013-01-21 18:51:41 UTC
The next SVG-edit community conference call is on 2013-02-12:

And one of the main topics is Wikipedia & the round-trip test suite:
https://code.google.com/p/svg-edit/wiki/CommunityConferenceCall2#Wikipedia

Thanks!
Comment 20 Andre Klapper 2013-02-25 19:02:55 UTC
Marc / Brion (bug assignee): Is there anything that volunteers can help with to get this forward? Was there any outcome from the community conference call?
Comment 21 Brion Vibber 2013-02-25 19:53:53 UTC
The main thing we want to do is get compatibility testing automated -- round-tripping problems when editing existing files is the biggest problem with SVG-Edit for Wikipedia potential usage.

I have a proof-of-concept:
https://github.com/brion/svg-edit-test

(this currently only runs in Firefox because other browsers taint <canvas>es that have SVGs drawn to them.)

Needs some more work:
* add a back-end data store to save original files, round-tripped files, renderings, and subtractive comparison images
* add a front-end test runner that automatically churns through thousands of images
* add a front-end dashboard that shows images with large difference scores and allows easy searching

Probably we want to set this up on a Wikimedia Labs instance.

Then we can start identifying round-trip failures cases and seeing about fixing them.



In addition or as an alternative, we could update SVGEdit extension to current code and either limit it to new files or files that were created with SVGEdit, or add a warning that editing may damage files.
Comment 22 Marc Laporte 2013-03-02 16:58:33 UTC
Hi!

The SVG-edit testing suite has already lead to a first bug fix!

We have 9 new committers in the last 2 months:
http://www.ohloh.net/p/svg-edit/contributors?query=&sort=newest

I think it's a great time to have clear objectives which will come from this dashboard. And for all the 'old' committers, once we have a dashboard, I will write to all of them to ask them to help.

Best regards,

M ;-)
Comment 23 Marc Laporte 2013-03-02 17:01:09 UTC
The 3rd SVG-edit 'Community Conference Call' is confirmed for April 9th, 2013 16h00 UTC: http://code.google.com/p/svg-edit/wiki/CommunityConferenceCall3

Wikipedia / dashboard is one the agenda.

Thanks!

M ;-)
Comment 24 Quim Gil 2013-03-23 23:35:43 UTC
This feature request is being proposed at

https://www.mediawiki.org/wiki/Talk:Mentorship_programs/Possible_projects#Polishing_this_list_for_GSOC.2713_24957

and I'm considering whether to add it or not to 

https://www.mediawiki.org/wiki/Summer_of_Code_2013#Project_ideas

If you think this is worth a try, I have some questions:

Could someone post a description of the work that needs to be done at http://www.mediawiki.org/wiki/Mentorship_programs/Possible_projects ?

Is there a potential mentor willing to help potential students interested in
this project?

Is there a reasonable support from the Commons community to
incorporate this feature if it's developed and meets the quality criteria?
Comment 25 Marc Laporte 2013-07-29 21:34:09 UTC
Some very interesting progress by Mike Baynton:

Please see comments here:
https://code.google.com/p/svg-edit/wiki/Wikipedia

And: http://home.mbaynton.com/svg-edit-test/svg-edit-test/launcher.php

Best regards,

M ;-)
Comment 26 Nemo 2013-09-17 09:24:42 UTC
Question: is SVG scripting (mentioned in <https://strategy.wikimedia.org/wiki/Proposal:Inline_SVG_preference>) in scope for this bug? If not, can someone please create a separate ticket for that in the most relevant component? Thanks.
Comment 27 Bawolff (Brian Wolff) 2013-09-17 15:08:28 UTC
(In reply to comment #26)
> Question: is SVG scripting (mentioned in
> <https://strategy.wikimedia.org/wiki/Proposal:Inline_SVG_preference>) in
> scope
> for this bug? If not, can someone please create a separate ticket for that in
> the most relevant component? Thanks.

Its not. (Also it seems very unlikely we would ever allow scripted svgs [scary security issues]. Declaritive animation with smil seems at face value more likely to be a possibility, but would still have a lot of issues to go through)
Comment 28 Brion Vibber 2013-09-19 15:16:46 UTC
Note there are some security issues reported recently with the base svg-edit (in the embedding API and in the editor itself); let's make sure those get fixed!
Comment 29 Marc Laporte 2013-10-03 15:29:20 UTC
SVG-edit 4th 'Community Conference Call' confirmed for October 10th, 2013 16h00 UTC: https://code.google.com/p/svg-edit/wiki/CommunityConferenceCall4

A major topic for Wikipedia is the recent progress on Round-Trip tests:
https://code.google.com/p/svg-edit/wiki/RoundtripTests

Thanks!
Comment 30 Marc Laporte 2013-10-10 22:20:27 UTC
The call recording is available at:
http://code.google.com/p/svg-edit/wiki/CommunityConferenceCall4

Many topics were discussed, but with respect to Wikipedia, next steps:

1- Security concerns must be addressed
2- SVG-edit could first be added for new drawings (until we have a reliable way to deal with existing drawings)
3- We should get the Multimedia list involved
http://lists.wikimedia.org/pipermail/multimedia/2013-July/000000.html

Thanks!
Comment 31 Bawolff (Brian Wolff) 2014-03-13 06:15:56 UTC
*** Bug 62562 has been marked as a duplicate of this bug. ***
Comment 32 Brett Zamir 2014-03-13 07:17:41 UTC
As far as item #1 on "Security concerns must be addressed", all issues on SVG-Edit should now be addressed in trunk, but I haven't gotten any takers to do testing yet.
Comment 33 Juan de Vojníkov 2014-03-15 08:31:47 UTC
I can test it once its deployed:-)
Comment 34 Brett Zamir 2014-03-15 09:33:54 UTC
Trunk is deployed live on the SVG Edit site, albeit not as part of any MW extension there:

http://svg-edit.googlecode.com/svn/trunk/editor/svg-editor.html
http://svg-edit.googlecode.com/svn/trunk/editor/embedapi.html

(Note that the dialog about cookies can be avoided)
Comment 35 Brett Zamir 2014-04-08 02:42:27 UTC
Version 2.7 of SVG Edit is now released.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links