Page MenuHomePhabricator
Create Task
Maniphest T143139

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

Assigned To
ovasileva
Authored By
• Nirzar
Aug 16 2016, 6:24 PM
Tags
Referenced Files
F4382849: iPhone 6.png
Aug 21 2016, 9:30 PM
F4382848: iPhone 6 Copy 5.png
Aug 21 2016, 9:30 PM
F4382842: iPhone 6 Copy 3.png
Aug 21 2016, 9:30 PM
F4382843: iPhone 6 Copy 4.png
Aug 21 2016, 9:30 PM
Subscribers
Aklapper
dr0ptp4kt
Jdlrobson
kaldari
• KHammerstein
Mhurd
• Nirzar
View All 10 Subscribers

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

NOWAFTER
iPhone 6 Copy 3.png (2×750 px, 65 KB)
iPhone 6 Copy 4.png (2×750 px, 66 KB)

Example

NOWAFTER
iPhone 6.png (2×750 px, 978 KB)
iPhone 6 Copy 5.png (2×750 px, 835 KB)

Dependency

  • Page image extension

Acceptance criteria

  • should work in beta where page-image extension is deployed

ping @dr0ptp4kt

Related Objects
Search...

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
dr0ptp4kt added a subscriber: dr0ptp4kt.Aug 18 2016, 5:15 PM

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

• Nirzar added a comment.Aug 18 2016, 5:29 PM

@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.
MBinder_WMF added a subscriber: MBinder_WMF.Aug 18 2016, 8:36 PM

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

• Nirzar added a comment.Aug 18 2016, 8:39 PM

@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
Jdlrobson added a subscriber: Jdlrobson.Aug 29 2016, 4:22 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
ovasileva closed this task as Resolved.May 25 2017, 10:24 AM
ovasileva claimed this task.
ovasileva closed subtask T150325: Move first paragraph before infobox on stable as Resolved.
ovasileva added a subscriber: ovasileva.

First paragraph now appearing before infobox

phuedx mentioned this in T169001: Remove logging from MobileFormatter#moveFirstParagraphBeforeInfobox.Jun 27 2017, 5:55 PM
phuedx added a subtask: T169001: Remove logging from MobileFormatter#moveFirstParagraphBeforeInfobox.
Jdlrobson changed the status of subtask T169001: Remove logging from MobileFormatter#moveFirstParagraphBeforeInfobox from Open to Stalled.Jul 7 2017, 6:07 PM
Jdlrobson closed subtask T169001: Remove logging from MobileFormatter#moveFirstParagraphBeforeInfobox as Resolved.Aug 17 2017, 3:14 PM
ovasileva reopened this task as Open.Dec 6 2017, 5:34 PM
ovasileva added a subtask: T170006: MobileFormatter#moveFirstParagraphBeforeInfobox should also move infoboxes wrapped in mw-stack element.
ovasileva closed subtask T170006: MobileFormatter#moveFirstParagraphBeforeInfobox should also move infoboxes wrapped in mw-stack element as Resolved.Jan 10 2018, 9:16 PM
Jdlrobson moved this task from Epics/Goals to Upcoming on the Readers-Web-Backlog board.Jan 12 2018, 5:17 PM
Jdlrobson reopened subtask T170006: MobileFormatter#moveFirstParagraphBeforeInfobox should also move infoboxes wrapped in mw-stack element as Open.Jan 16 2018, 9:09 PM
ovasileva moved this task from Upcoming to Epics/Goals on the Readers-Web-Backlog board.Jan 23 2018, 1:33 AM
phuedx closed subtask T170006: MobileFormatter#moveFirstParagraphBeforeInfobox should also move infoboxes wrapped in mw-stack element as Resolved.Jan 31 2018, 11:15 AM
Jdlrobson closed this task as Resolved.Jan 31 2018, 6:52 PM

I'm calling this done. Failure rate of the lead paragraph move is extremely low and we can work in chores to keep in that way. (https://www.mediawiki.org/wiki/Reading/Web/Chore_list#Review_the_MobileFormatter_lead_paragraph_transform_on_Wikimedia_content)

• Phabricator_maintenance moved this task from Epics/Goals to 2017-18 Q3 on the Readers-Web-Backlog board.Mar 7 2018, 4:44 AM
Jack_who_built_the_house mentioned this in T210540: LeadParagraph: In ruwiki, on mobile, hatnotes sometimes go after the first paragraph.Nov 27 2018, 8:11 PM
XanonymusX mentioned this in T263306: MobileFrontend displays second paragraph as first paragraph on dewiki (Moving article lead is broken).Sep 18 2020, 11:48 PM
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL