Page MenuHomePhabricator

The "All" and "Other" tabs on MobileFrontend watchlist show Flow gibberish topic ID links instead of topic titles
Closed, DeclinedPublic

Description

The "All" and "Other" tabs on MobileFrontend watchlist shows Flow gibberish topic ID links. Also, it doesn't show topic changes in the "Talk" tab.

What should happen:

  • Stuff like Topic:Se14547daaj65kpa shouldn't be shown. Instead it should show the human-readable topic title.
  • Topic changes should be shown in the Talk tab. Currently they are omitted.
  • Possibly, changes to a flowified talk pages shold be shown only as topic changes. Currently "User talk:Amire80" is shown, and if it's the same change as the topic change, it's unnecessary.

Event Timeline

Amire80 created this task.Mar 23 2015, 9:19 AM
Amire80 updated the task description. (Show Details)
Amire80 added a subscriber: Amire80.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 23 2015, 9:19 AM
Amire80 renamed this task from gibberish in mobile fronted with Flow to the "All" tab on MobileFrontend watchlist shows Flow gibberish topic ID links.Mar 23 2015, 9:27 AM
Amire80 updated the task description. (Show Details)
Amire80 set Security to None.
Amire80 added subscribers: DannyH, Jdlrobson.

Come to think of it, ideally the notifications for Flow on mobile should be so good that a watchlist for talk pages shouldn't be needed at all :)

MaxSem moved this task from 2014-15 Q4 to 2016-17 Q4 on the Readers-Web-Backlog board.
EBernhardson triaged this task as Normal priority.Mar 23 2015, 5:45 PM
EBernhardson added a subscriber: EBernhardson.

Part of the fix for this T59154, it is not a complete solution but i think it would help to at least have sensible titles shown without having to visit each place that shows titles.

I think we just need to play Whac-a-Mole with these places. Adding words to the unique ID won't really make this mobile page a lot better. It should say "Topic title on Board name".

QuimGil renamed this task from the "All" tab on MobileFrontend watchlist shows Flow gibberish topic ID links to The "All" and "Other" tabs on MobileFrontend watchlist show Flow gibberish topic ID links instead of topic titles.Nov 15 2015, 2:41 PM
QuimGil updated the task description. (Show Details)

As a first basic fix, couldn't the mobile watchlist use the same code used by the mobile notifications, where the topic appears in human-readable form?

This might not be the perfect UX solution, but it looks like low-hanging fruit (?) to solve the current problem, which is grave. When new mobile users see "Topic:Sspenndn8ro02k81", there goes their credibility about Flow.

Is this low-hanging fruit solution good first bug? Could it be a Google-Code-In-2015 task?

DannyH removed a subscriber: DannyH.Nov 16 2015, 11:38 PM
Jdlrobson lowered the priority of this task from Normal to Lowest.Dec 3 2015, 11:46 PM

No time to work on this right now but patches welcome although this might be a little overcomplicated . T109277 might be a better use of time here.

QuimGil added a comment.EditedDec 4 2015, 12:09 AM

Can someone explain in simple terms why the human readable titles that appear in mobile notifications are so difficult to show in the mobile watchlist?

PS: I know not many users check ca.wiki, mediawiki.org and office.wiki watchlists from their mobile device regularly, but this is my true story. :) This bug is the most visible broken aspect of my Wikimedia mobile experience.

Jdlrobson moved this task from Backlog to Tracking on the MobileFrontend board.Mar 16 2016, 10:38 PM
Jdlrobson closed this task as Declined.May 22 2017, 10:11 AM

We don't have capacity to reimplement the watchlist anymore then we currently have. The problem with the current implementation is it directly hits the database rather than making use of an api method provided by core to access to all rows in the watchlist. It thus overlooks things like preferences. Continuing like this is unsustainable.

I will make sure this particular feature is added as a requirement to T109277 which aims to increase code quality of the special watchlist code in core to allow it to be skinned for mobile. At very least that work should give us an api/model of the data in the watchlist and simply override the presentation.

Please bear with me a week or so to do this.
Ps. Not sure if declining or merging as duplicate was appropriate here.

Any status is OK as long as the intention is to get rid of the gibberish.