Page MenuHomePhabricator
Create Task
Maniphest T70367

Mobile Watchlist does not take into account preference value of expand/aggregation into account
Closed, DeclinedPublic
Actions

Description

By default, the mobile watch list uses the "Expand watchlist to show all changes, not just the most recent" mode of the watch list, even if not so selected in the preferences of the user.

This is inconsistent with the website and might be confusing. Also it means that busy pages take up an extraordinary amount of items in the watch list, making it easy to miss changes on other pages.

Version: unspecified
Severity: normal
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=68368
https://bugzilla.wikimedia.org/show_bug.cgi?id=68369

Details

Reference
bz68367

Related Objects
Search...

Event Timeline

• bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:37 AM
• bzimport added a project: MobileFrontend-General-or-Unknown.
• bzimport set Reference to bz68367.
• bzimport added a subscriber: Unknown Object (MLST).
TheDJ created this task.Jul 22 2014, 9:56 AM
• bzimport added a comment.Jul 22 2014, 10:00 AM

bingle-admin wrote:

Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/KpdIlK5i

gerritbot added a comment.Sep 19 2014, 5:45 PM

Change 149326 had a related patch set uploaded by Jdlrobson:
WIP: Make mobile watchlist use core code

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

Jdlrobson edited projects, added Web-Team-Backlog; removed MobileFrontend-General-or-Unknown.Dec 9 2014, 11:42 PM
Jdlrobson set Security to None.
Jdlrobson moved this task from Incoming to 2016-17 Q4 on the Web-Team-Backlog board.Dec 10 2014, 12:16 AM
Jdlrobson removed a project: Patch-For-Review.Dec 23 2014, 10:15 PM
Jdlrobson added a project: MediaWiki-Watchlist.

Patch was abandoned. We need to refactor core Watchlist code to support mobile use case.

Jdlrobson moved this task from 2016-17 Q4 to 2014-15 Q3 on the Web-Team-Backlog board.Jan 15 2015, 9:09 PM
• Moushira subscribed.Feb 13 2015, 2:26 PM
• Tbayer subscribed.Apr 12 2015, 12:48 AM
• Jhernandez triaged this task as Low priority.Aug 17 2015, 11:55 AM
• Jhernandez edited projects, added MobileFrontend; removed Web-Team-Backlog.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 17 2015, 11:55 AM
• Jhernandez added a parent task: T109277: [EPIC]: Use core watchlist code for mobile experience.Aug 17 2015, 12:01 PM
MaxSem unsubscribed.Aug 17 2015, 6:34 PM
MBinder_WMF added a project: Web-Team-Backlog.Apr 27 2016, 4:28 PM
MBinder_WMF moved this task from 2014-15 Q3 to Incoming on the Web-Team-Backlog board.Apr 27 2016, 4:35 PM
MBinder_WMF moved this task from Incoming to Triaged but Future on the Web-Team-Backlog board.Jul 4 2016, 7:49 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.

kaldari unsubscribed.May 23 2017, 10:41 PM
Jdlrobson mentioned this in T109277: [EPIC]: Use core watchlist code for mobile experience.May 25 2017, 10:34 PM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and 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