Last modified: 2014-06-18 16:49:28 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 T49326, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47326 - <a> / <b> / <i> issues.
<a> / <b> / <i> issues.
Status: NEW
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Low normal
: ---
Assigned To: C. Scott Ananian
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-17 17:44 UTC by C. Scott Ananian
Modified: 2014-06-18 16:49 UTC (History)
3 users (show)

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


Attachments

Description C. Scott Ananian 2013-04-17 17:44:35 UTC
Currently <a> tags are being broken by boldface/italics changes.  See https://gerrit.wikimedia.org/r/57690 and bug 44476.

We should fix that, hopefully fixing cases like ''[[Lunar Prospector]]'''s at the same time (see https://gerrit.wikimedia.org/r/57748 ).

See also mediawiki.DOMPostProcessor:minimizeInlineTags, which currently ignores <a> tags.  Probably the first task to do is turn this off and see what happens; we think this code isn't doing much right now (see bug 42803).
Comment 1 Gabriel Wicke 2013-04-19 20:10:22 UTC
A DOM-based solution makes sense, but I doubt that the current minimizeInlineTags helps much as it will only re-shuffle inline tags starting at the same text / content offset. This is not sufficient in cases like the example in bug 42803.

Fortunately, selective serialization mostly hides this issue for most untouched content as the break-up is deterministic. The visual editor tries to give priority to links, so in new content this should also not be an issue.
Comment 2 Gabriel Wicke 2013-05-02 04:25:05 UTC
Somewhat related, but harder to 'fix': bug 47963
Comment 3 Andre Klapper 2013-07-04 10:34:18 UTC
[Parsoid component reorg by merging JS/General and General. See bug 50685 for more information. Filter bugmail on this comment. parsoidreorg20130704]
Comment 4 Gabriel Wicke 2014-06-18 16:49:28 UTC
Another test case reported by jackmcbarn:

''<table><tr><td>'''foo'''</td></tr></table>'''

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


Navigation
Links