References parsing JS doesn't recognize cluster of ref links if they have space between links.
Closed, ResolvedPublic
Actions

Assigned To

None

Authored By

	Mhurd
	Jul 30 2014, 12:02 AM

Description

It works fine if the refs looks like this "[1][2][3]"

But it doesn't if they look like this "[1] [2] [3]"

Version: Alpha
Severity: normal

Details

Reference: bz68831

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:34 AM

• bzimport added a project: Wikipedia-iOS-App-Backlog.

• bzimport set Reference to bz68831.

• bzimport added a subscriber: Unknown Object (MLST).

Mhurd created this task.Jul 30 2014, 12:02 AM

One way to fix this is by replacing nextSibling/previousSibling with nextElementSibling/previousElementSibling.
The drawback is that it also finds some elements a bit farther away.

Got a sample page to test with? Probably easyish to fix per Bernd's note

I created one on test wiki, but not sure if they really happen in a real world wiki.
https://test.wikipedia.org/wiki/Reference_Test, refs [4]-[7]

Done on the iOS refs branch in: https://gerrit.wikimedia.org/r/#/c/148247/ -- see the goLeft/goRight functions

• KLans_WMF moved this task from Needs Triage to Resolved/Invalid/Declined/Legacy on the Wikipedia-iOS-App-Backlog board.Mar 16 2015, 9:34 PM

• Phabricator_maintenance removed a subscriber: yuvipanda.Jun 7 2017, 6:58 PM

References parsing JS doesn't recognize cluster of ref links if they have space between links.Closed, ResolvedPublicActions

Description

Details

Event Timeline

References parsing JS doesn't recognize cluster of ref links if they have space between links.
Closed, ResolvedPublic
Actions