Last modified: 2014-02-12 23:47:57 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 T50760, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48760 - Unexpected <br> tags in mobile diffs
Unexpected <br> tags in mobile diffs
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
beta (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Jon
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-05-24 01:06 UTC by Juliusz Gonera
Modified: 2014-02-12 23:47 UTC (History)
9 users (show)

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


Attachments

Description Juliusz Gonera 2013-05-24 01:06:49 UTC
This test case doesn't pass:

		[
			$( '<div><del>Here we go.</del><ins>Here we go. Ha!</ins></div>' ),
			'<span>Here we go.</span><ins> Ha!</ins>'
		]


Failed assertion: Test case 7, expected: <ins> Ha!</ins>, but was: <span>Here we go</span><ins>. Ha!&lt;</ins><del>.&lt;</del><span>br&gt;</span>
Comment 1 Jon 2013-05-29 20:34:47 UTC
This seems to be restricted to adding new sentences.

go.<br> is treated as a word.
https://en.m.wikipedia.org/wiki/Special:MobileDiff/557405110...557405575
Comment 3 Gerrit Notification Bot 2013-06-13 17:58:47 UTC
Related URL: https://gerrit.wikimedia.org/r/68456 (Gerrit Change Ia2185b655c1d84ef29781e8702290a719ebe8865)
Comment 4 Gerrit Notification Bot 2013-06-14 00:07:14 UTC
https://gerrit.wikimedia.org/r/68456 (Gerrit Change Ia2185b655c1d84ef29781e8702290a719ebe8865) | change APPROVED and MERGED [by jenkins-bot]

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


Navigation
Links