Page MenuHomePhabricator

Updates to "Last edited" element on mobile
Open, NormalPublic

Description

Description

There are two main issues with the "Last edited" element on mobile:

  • the tappable area for the element is unclear (only the "Last edited ... ago" text is tappable, not the chevron or the rest of the element)
  • the element includes a link to the user page of the user who made the most recent update, which is not useful (read T163853#3281928)

Design notes

  • entire element (edge-to-edge) should be tappable, and lead to the history page
  • remove the link to the user page

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 25 2017, 10:37 PM
Jdlrobson triaged this task as Normal priority.Apr 26 2017, 9:54 PM
Jdlrobson added a project: patch-welcome.
Jdlrobson moved this task from Incoming to Triaged but Future on the Readers-Web-Backlog board.
TheDJ added a subscriber: TheDJ.May 21 2017, 11:38 AM

Yeah, probably want to make that entire bar clickable really. I recently noticed this too.

I also noticed I accidentally clicked the editor name. I think that's useless. The last editor's userpage is not something that you often would want to go to (you want to show who it is, but that doesn't necessarily mean you need to interact with that person right away. You can still easily get to him from the history page, so I think just having the entire bar move you to the history page is more sensible.

Jdlrobson updated the task description. (Show Details)
Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptJun 6 2017, 4:56 PM
Jdlrobson added a subscriber: Jdlrobson.

Nirzar this needs some input

This "easy" task has no codebase project assigned so I assume this is about the MobileFrontend codebase?

Jdlrobson moved this task from Backlog to Tracking on the MinervaNeue board.Oct 16 2017, 10:25 PM
Jdlrobson moved this task from Tracking to Bugs on the MinervaNeue board.

@alexhollender feel free to merge this into T193993 if that makes sense, but let's work out a way to address the issues associated with this component.

Niedzielski renamed this task from Last edited arrow and cell should be clickable to Last edited history arrow, icon, and cell should be clickable.Nov 29 2018, 4:19 PM
Niedzielski removed Nirzar as the assignee of this task.
Niedzielski updated the task description. (Show Details)
Niedzielski added a subscriber: Nirzar.
alexhollender renamed this task from Last edited history arrow, icon, and cell should be clickable to Updates to "Last edited" element on mobile.Jun 4 2019, 4:55 PM
alexhollender updated the task description. (Show Details)
alexhollender updated the task description. (Show Details)Jun 4 2019, 5:03 PM
alexhollender added a subscriber: ovasileva.

@ovasileva @Jdlrobson even if we bring AMC features to all logged-in users this will still be an issue for logged-out users. It seems worth fixing to me. Moving out of design column since there is no more design work needed (the task description has been updated to clarify what should be done).

@ovasileva @Jdlrobson even if we bring AMC features to all logged-in users this will still be an issue for logged-out users. It seems worth fixing to me. Moving out of design column since there is no more design work needed (the task description has been updated to clarify what should be done).

sounds good.

Given that AMC may be deployed to all users in time, do we want to remove this link instead?

Per discussion: not a priority. Moving to backlog. We'll know more once AMC has been out for a while.