Page MenuHomePhabricator

Watchlist Expiry: Improve existing behavior of watching via star on mobile [medium]
Closed, ResolvedPublic

Description

As a user, I want to watch via star on mobile, so that I can easily watch a page for a short period of time .

Acceptance criteria:

  • The user should be able to still watch a star permanently with existing behavior (i.e., click on star to watch permanently)
  • The notification at the bottom should be changed to read: [page name] and its talk page have been added to your watchlist permanently
  • The dropdown should be labeled with "Watchlist time period" and display the following options:
    • Permanent (as default)
    • 1 week
    • 1 month
    • 3 months
    • 6 months
  • If a temporary expiry was chosen, the following behavior should occur:
    • The full star becomes half star (covered in T261970)
    • Display a success message: [page name] and its talk page have been added to your watchlist [time period, such as "one week"]
    • Adjust the watch state of the page to have an expiry
  • If a temporary expiry is not chosen (i.e., the user keeps permanent selection):
    • They will see original message notification
    • The original dismissal behavior of the pop-up should remain in place
  • Note: UX improvements will be covered in a separate ticket (T261997), and the bug related to only seeing the half star upon refresh will be covered in a separate ticket (T261970)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Note: We can find an example of this bottom sheet when someone tries to watch a page when logged off in mobile view. See example below.

Screen Shot 2020-05-05 at 1.17.10 PM.png (697×1 px, 279 KB)

ifried renamed this task from Watchlist Expiry: Add star icon UI to mobile [WIP] to Watchlist Expiry: Add star icon UI to mobile .May 7 2020, 5:10 PM
ARamirez_WMF renamed this task from Watchlist Expiry: Add star icon UI to mobile to Watchlist Expiry: Add star icon UI to mobile [medium].May 7 2020, 5:22 PM
ARamirez_WMF moved this task from Needs Discussion to Up Next (June 3-21) on the Community-Tech board.
dmaza removed dmaza as the assignee of this task.Jul 7 2020, 10:00 PM
dmaza moved this task from In Development to Ready on the Community-Tech (Kanban-2019-20-Q4) board.
dmaza subscribed.

We should copy the above Slack conversation to this ticket, if it's useful, because not everyone has access to the WMF Slack workspace.

Change 622203 had a related patch set uploaded (by MusikAnimal; owner: MusikAnimal):
[mediawiki/skins/MinervaNeue@master] Show half star for temporarily watched pages

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

Change 622203 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Show half star for temporarily watched pages

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

ifried renamed this task from Watchlist Expiry: Add star icon UI to mobile [medium] to Watchlist Expiry: Improve existing behavior of watching via star on mobile [medium].Sep 3 2020, 10:00 PM
ifried updated the task description. (Show Details)
ifried updated the task description. (Show Details)
ifried updated the task description. (Show Details)

Everything in the acceptance criteria should be done, with exception of "The full star becomes half star" as soon as you watch temporarily (T261970). So really the only work I did was to show the half-star on page load accordingly (e.g. watch temporarily, refresh, and you should see a half star).

Change 626531 had a related patch set uploaded (by MusikAnimal; owner: MusikAnimal):
[mediawiki/skins/MinervaNeue@REL1_35] Show half star for temporarily watched pages

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

Change 626531 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@REL1_35] Show half star for temporarily watched pages

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

dom_walden subscribed.
  • The user should be able to still watch a star permanently with existing behavior (i.e., click on star to watch permanently)

Sure.

  • The notification at the bottom should be changed to read: The user will first see the following message at the top: [page name] and its talk page have been added to your watchlist permanently

Sure.

  • The dropdown should be labeled with "Watchlist time period" and display the following options:
    • Permanent (as default)
    • 1 week
    • 1 month
    • 3 months
    • 6 months

Sure.

Sometimes, if the screen is wide enough or the font small enough, there is no gap between the two sentences, so it reads:
...have been added to your watchlist permanently.Watchlist time period:

mobile_no_gap.png (162×1 px, 10 KB)

I have not seen this happen on any phones or tablets I have tested so far (either portrait or landscape orientation). Still, it affects desktop Minerva as well so raised T262934.

  • If a temporary expiry was chosen, the following behavior should occur:
    • The full star becomes half star (covered in T261970)

As Dayllan mentioned, initially the star appears incorrectly oriented:

incorrectly_oriented_star.png (2×1 px, 263 KB)

After refreshing the page, it is correctly oriented:
refresh_correctly_oriented_star.png (2×1 px, 213 KB)

In RTL languages, the star is "reversed", e.g.:
hebrew_star.png (2×1 px, 201 KB)

I don't know if this is intentional, but it seems OK to me (and consistent with some other UI elements such as the VE loading bar). Although I have not seen this behaviour for any of the other skins (as far as I remember).

I also notice that there is no spinning transition when going from permanent to temporary, like there is going from unwatched to permanent (or vice versa). I don't know if that will be covered in T261970 (or if that should be added to the acceptance criteria).

Also, when you first visit a page you are already temporarily watching, the star does not spin like it does when the page is permanently watched. Again, this might be covered in T261970.

  • Display a success message: [page name] and its talk page have been added to your watchlist [time period, such as "one week"]

Sure.

    • Adjust the watch state of the page to have an expiry
  • If a temporary expiry is not chosen (i.e., the user keeps permanent selection):
    • They will see original message notification
    • The original dismissal behavior of the pop-up should remain in place

It dismisses after ~6 seconds after it was last interacted with (hovering over, selecting expiry, etc.)

It can be dismissed by clicking/tapping on it.

Test environment: various versions of https://en.wikipedia.beta.wmflabs.org, most recent MediaWiki 1.36.0-alpha (1c56c9f) 2020-09-15T01:49:13

Test devices:

  • Samsung S9 Chrome
  • iPad 7th Safari
  • Samsung S4 Chrome (low PPI screen)
ifried closed this task as Resolved.EditedSep 15 2020, 10:12 PM

I have tested this, and I am able to watch via star on mobile (see screenshot examples below). The issues related to the star shading will be handled separately. I am marking this as Done.

Selecting a temporary time period in mobile:

Screen Shot 2020-09-15 at 6.14.41 PM.png (630×352 px, 100 KB)

Half star in mobile:

Screen Shot 2020-09-15 at 6.13.49 PM.png (640×861 px, 81 KB)