[EPIC] Move first paragraph <p> above infobox template on small screens
Closed, ResolvedPublic
Actions

Description

[UX DEBT]

The problem
It takes few scrolls to get to the first sentence of the article. the top of the article doesn't give out overview of the topic quickly. with lead images, there is duplicate information taking up space.

The solution
Move only first paragraph <p> on top of the infobox if there is a lead-image

wireframe

NOW	AFTER

Example

NOW	AFTER

Dependency

Page image extension

Acceptance criteria

should work in beta where page-image extension is deployed

ping @dr0ptp4kt

Related Objects
Search...

Status	Subtype	Assigned	Task
Resolved		Jdlrobson	T147641 [EPIC] Improve top of the article user experience - mobile
Resolved		ovasileva	T143139 [EPIC] Move first paragraph <p> above infobox template on small screens
Resolved		phuedx	T143803 [Spike 4hr]: Explore methods to move first paragraph above infobox
Resolved		phuedx	T150325 Move first paragraph before infobox on stable
Resolved		phuedx	T149852 Lead introduction edge case in MobileFrontend where paragraph precedes list element
Resolved		phuedx	T149830 Re-enable moving the first paragraph before infobox on MF beta
Resolved		Jdlrobson	T149884 Log instances of infoboxes being wrapped in containers
Resolved	PRODUCTION ERROR	phuedx	T151838 [4 hours] Warning: Invalid state error in MobileFormatter
Resolved		Jdlrobson	T162713 [spike 2hrs] Evaluate result of "Log instances of infoboxes being wrapped in containers" (T149884 )
Resolved		Jdlrobson	T163805 Adjustments to how we log troublesome infoboxes and move lead paragraph
Resolved		Jdlrobson	T169001 Remove logging from MobileFormatter#moveFirstParagraphBeforeInfobox
Resolved		phuedx	T170006 MobileFormatter#moveFirstParagraphBeforeInfobox should also move infoboxes wrapped in mw-stack element
Resolved		Niedzielski	T185040 Regression: Lead intro "Found infobox wrapped with container on" warnings are logged on pages with infoboxes inside navboxes
Resolved		Niedzielski	T185106 Cannot access property on non-object in MoveLeadParagraphTransform

Event Timeline

• Nirzar created this task.Aug 16 2016, 6:24 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 16 2016, 6:24 PM

• Nirzar triaged this task as Medium priority.Aug 16 2016, 6:24 PM

• Nirzar updated the task description. (Show Details)Aug 16 2016, 6:26 PM

• Nirzar moved this task from Incoming to 2016-17 Q2 on the Readers-Web-Backlog board.

MBinder_WMF moved this task from 2016-17 Q2 to 2016-17 Q1 on the Readers-Web-Backlog board.Aug 16 2016, 6:29 PM

• Nirzar moved this task from 2016-17 Q1 to 2016-17 Q2 on the Readers-Web-Backlog board.Aug 16 2016, 6:29 PM

Nirzar, is this next in line for UX debt? I noticed this was dropped into sprint +1 on Readers-Web-Backlog.

@dr0ptp4kt yes but it depends too. if the team takes on related pages. i can stall this.

dr0ptp4kt added a project: Reading-Web-Sprint-80-V-for-Vandalism.Aug 18 2016, 6:29 PM

dr0ptp4kt moved this task from 2016-17 Q2 to Product Owner Backlog on the Readers-Web-Backlog board.

@Nirzar Can you flesh this out before Monday's prioritization meeting?

@MBinder_WMF yes.

• Nirzar updated the task description. (Show Details)Aug 21 2016, 9:30 PM

• Nirzar updated the task description. (Show Details)

Jdlrobson created subtask T143803: [Spike 4hr]: Explore methods to move first paragraph above infobox.Aug 24 2016, 4:41 PM

dr0ptp4kt moved this task from Product Owner Backlog to 2016-17 Q2 on the Readers-Web-Backlog board.Aug 25 2016, 6:06 PM

MBinder_WMF removed a subscriber: MBinder_WMF.Aug 25 2016, 6:19 PM

To elaborate on why I added a subtask: T143803

While I agree we should do this, this needs prototyping and further analysis before doing in MobileFrontend which is costly and risky. The Mobile Content service hasn't got this quite right (T143739) so I have little confidence we will.

dr0ptp4kt removed a project: Reading-Web-Sprint-80-V-for-Vandalism.Aug 29 2016, 4:30 PM

dr0ptp4kt moved this task from 2016-17 Q2 to Product Owner Backlog on the Readers-Web-Backlog board.

Jdlrobson closed subtask T143803: [Spike 4hr]: Explore methods to move first paragraph above infobox as Resolved.Sep 9 2016, 5:40 PM

• Nirzar mentioned this in T91712: Move lead paragraph above other article content.Oct 17 2016, 11:19 PM

• Nirzar merged a task: T91712: Move lead paragraph above other article content.

• Nirzar added subscribers: • KHammerstein, • Tbayer, phuedx and 2 others.

• Nirzar removed • Nirzar as the assignee of this task.Oct 19 2016, 7:09 PM

Jdlrobson added a subtask: T149852: Lead introduction edge case in MobileFrontend where paragraph precedes list element.Nov 2 2016, 9:44 PM

Krinkle updated the task description. (Show Details)Apr 29 2017, 12:58 AM

phuedx closed subtask T149852: Lead introduction edge case in MobileFrontend where paragraph precedes list element as Resolved.May 2 2017, 1:11 PM

Jdlrobson renamed this task from Move first paragraph <p> above infobox template on small screens to [EPIC] Move first paragraph <p> above infobox template on small screens.May 21 2017, 11:24 AM

Jdlrobson added a project: Epic.

Jdlrobson added subtasks: T150325: Move first paragraph before infobox on stable, T149884: Log instances of infoboxes being wrapped in containers, T151838: [4 hours] Warning: Invalid state error in MobileFormatter, T162713: [spike 2hrs] Evaluate result of "Log instances of infoboxes being wrapped in containers" (T149884 ), T163805: Adjustments to how we log troublesome infoboxes and move lead paragraph.

Jdlrobson moved this task from Product Owner Backlog to Epics/Goals on the Readers-Web-Backlog board.May 23 2017, 10:08 PM

phuedx mentioned this in T150325: Move first paragraph before infobox on stable.May 25 2017, 4:14 AM