Page MenuHomePhabricator

Unexpected <br> tags in mobile diffs
Closed, ResolvedPublic

Description

Author: jgonera

Description:
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>


Version: unspecified
Severity: normal

Details

Reference
bz48760

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 1:27 AM
bzimport added a project: MobileFrontend-beta.
bzimport set Reference to bz48760.
bzimport created this task.May 24 2013, 1:06 AM

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

Related URL: https://gerrit.wikimedia.org/r/68456 (Gerrit Change Ia2185b655c1d84ef29781e8702290a719ebe8865)

https://gerrit.wikimedia.org/r/68456 (Gerrit Change Ia2185b655c1d84ef29781e8702290a719ebe8865) | change APPROVED and MERGED [by jenkins-bot]