Last modified: 2013-06-23 15:50:24 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 T51608, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 49608 - VisualEditor: HTML comments are dropped from transclusion calls
VisualEditor: HTML comments are dropped from transclusion calls
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Data Model (Other open bugs)
unspecified
All All
: Highest major
: VE-deploy-2013-06-20
Assigned To: Ed Sanders
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-15 02:59 UTC by Matthew Flaschen
Modified: 2013-06-23 15:50 UTC (History)
6 users (show)

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


Attachments

Description Matthew Flaschen 2013-06-15 02:59:00 UTC
https://en.wikipedia.org/w/index.php?title=Vidin&diff=559844598&oldid=559844102 shows several dirty diff issues, including spurious template bars, space changes, and corrupt links.

However, this bug is specifically about the HTML comments that are dropped, "the name of a location map as per http://en.wikipedia.org/wiki/Template:Location_map" and "the position of the pushpin label: left, right, top, bottom, none".  These comments often have important messages to other editors, so they can not be stripped.
Comment 1 Timeshifter 2013-06-15 04:13:41 UTC
At the time of this discussion by VE developers, 
[[Wikipedia:VisualEditor/Feedback/Archive 2013 05#General_observations]], hidden notes/comments were preserved. Now all hidden HTML notes on a page (or at least those in templates) are removed when the page is edited by VE.

Page edit notices and section edit notices will not work as a replacement since hidden notes can be instructional for specific references, tables, infoboxes, navboxes, sentences, paragraphs, etc.. Page notices and edit notices are for a whole page or section. For more info: 
[[Wikipedia:VisualEditor/Feedback#HTML_notes]] - or wherever it ends up later in the archives. 

If HTML notes are going to be replaced by some new hidden notation system, they still need to be preserved in the meantime. They should be treated similarly to how VE handles tables, infoboxes, and other stuff that VE can not currently edit. VE should not be doing anything within templates. Templates are too complex for VE to meddle with in the slightest way. VE should not even remove spaces in templates. 

If VE ends up with another hidden note tool, then a bot may have to go around to convert all existing hidden HTML notes to the new VE tool. But why bother? 

Why not keep the HTML notes, and use some kind of popup tooltip in VE that pops up when one puts the mouse cursor over a hidden note icon in VE edit mode? So one can read the hidden note in the popup. Kind of like how reference tooltips work.
Comment 2 James Forrester 2013-06-16 22:07:35 UTC
Confirm that these are being re-broken, but seemingly online in template calls (where we can't alienate them?). Ed, can you confirm at your end if this is a DM issue or a Parsoid one?

I was able to edit around an HTML comment without altering it inline (as expected) in https://www.mediawiki.org/w/index.php?title=VisualEditor:TestComments&diff=712245&oldid=712244 - but changes to the HTML comment in the second template fail to be detected as a change (?). Note that the first template can't be edited as the preceding block comment it created as an mw:Placeholder (per bug 47403) and (wrongly?) merged with the following transclusion.
Comment 3 Timeshifter 2013-06-16 23:14:27 UTC
See also bug 49655 and this (where Ssastry discusses it): 
Wikipedia:VisualEditor/Feedback#Removal_of_comments_in_Infobox
Comment 4 Timeshifter 2013-06-16 23:18:00 UTC
[[Wikipedia:VisualEditor/Feedback#Removal_of_comments_in_Infobox]]

See also bug 42124.
Comment 5 James Forrester 2013-06-16 23:38:46 UTC
(In reply to comment #3)
> See also bug 49655 and this (where Ssastry discusses it): 
> Wikipedia:VisualEditor/Feedback#Removal_of_comments_in_Infobox

The place for "see also" bugs is the "see also" section at the top right of the bug - have put it there. :-) Bug 42124 is not relevant.
Comment 6 Ed Sanders 2013-06-17 10:32:36 UTC
Can't reproduce in master. The comment appears in the template dialog and can be edited. With experimental code disabled the template is completely untouched.
Comment 7 James Forrester 2013-06-17 23:24:27 UTC
This is now fixed; as an example, see https://en.wikipedia.org/w/index.php?title=Bleak_House&diff=560362551&oldid=560338958 as an edit made with VisualEditor that leaves the comments in the templates as they were.
Comment 8 Elitre 2013-06-23 14:09:18 UTC
Not sure if it's the same thing, but I just did this today http://it.wikipedia.org/w/index.php?title=Google&diff=59622535&oldid=59377529 and it discarded commented text which, as noted above, should be there for a reason :)
Comment 9 Ed Sanders 2013-06-23 15:50:24 UTC
That's a separate bug, will raise as such.

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


Navigation
Links