[Bug] mobile-html: back link text doesn't match reference link text for reference groups
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	JoeWalsh
	Mar 3 2020, 11:23 AM

Description

Steps to Reproduce

Visit an article with reference groups (https://apps.wmflabs.org/en.wikipedia.org/v1/page/mobile-html/Neferefre, https://apps.wmflabs.org/en.wikipedia.org/v1/page/mobile-html/United_States)
Expand the references section

Expected Results

Back link titles match reference link titles (truncated according to the designs in https://phabricator.wikimedia.org/T244627

Screen Shot 2020-03-03 at 6.22.36 AM.png (1×2 px, 426 KB)

Actual Results

Back link titles are all numeric

Screen Shot 2020-03-03 at 6.22.07 AM.png (1×2 px, 426 KB)

Details

	Subject	Repo	Branch	Lines +/-
	Ensure reference back link matches reference link	mediawiki/services/mobileapps	master	+885 -777

Customize query in gerrit

Related Objects

Mentioned In: T272097: Parsoid Cite doesn't know about special group names like "lower-alpha" etc. and treats them like other group names like "xyz", "books", etc.
Mentioned Here: T51538: Cite: Improve compatibility between Parsoid's port and the PHP extension
T244627: Update reference styling for Mobile HTML work

Event Timeline

JoeWalsh created this task.Mar 3 2020, 11:23 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 3 2020, 11:23 AM

JoeWalsh triaged this task as High priority.Mar 3 2020, 11:23 AM

JoeWalsh moved this task from Needs triage to Kanban on the Product-Infrastructure-Team-Backlog-Deprecated board.

JoeWalsh edited projects, added Product-Infrastructure-Team-Backlog-Deprecated (Kanban); removed Product-Infrastructure-Team-Backlog-Deprecated.

JoeWalsh moved this task from To Do to Code Review on the Product-Infrastructure-Team-Backlog-Deprecated (Kanban) board.

Change 576308 had a related patch set uploaded (by Joewalsh; owner: Joewalsh):
[mediawiki/services/mobileapps@master] Ensure reference back link matches reference link

https://gerrit.wikimedia.org/r/576308

gerritbot added a project: Patch-For-Review.Mar 3 2020, 12:11 PM

The .textContent of some reference links are incorrect. For example, references that display as [a] have a .textContent of [lower-alpha 1]. The text is changed for display in CSS and is unreadable from JS. This will be fixed in Parsoid by T51538

Change 576308 merged by jenkins-bot:
[mediawiki/services/mobileapps@master] Ensure reference back link matches reference link

https://gerrit.wikimedia.org/r/576308

• bearND moved this task from Code Review to To Deploy on the Product-Infrastructure-Team-Backlog-Deprecated (Kanban) board.Mar 6 2020, 7:11 PM

• bearND removed a project: Patch-For-Review.

• bearND moved this task from To Deploy to Sign off on the Product-Infrastructure-Team-Backlog-Deprecated (Kanban) board.Mar 10 2020, 5:35 PM

• bearND closed this task as Resolved.Apr 2 2020, 6:17 PM

ssastry mentioned this in T272097: Parsoid Cite doesn't know about special group names like "lower-alpha" etc. and treats them like other group names like "xyz", "books", etc..Jan 14 2021, 11:29 PM

	F31658352: Screen Shot 2020-03-03 at 6.22.36 AM.png
	Mar 3 2020, 11:23 AM

	F31658362: Screen Shot 2020-03-03 at 6.22.07 AM.png
	Mar 3 2020, 11:23 AM

[Bug] mobile-html: back link text doesn't match reference link text for reference groupsClosed, ResolvedPublicActions