Page MenuHomePhabricator

IP Masking: Remove access to Preferences for Temporary accounts
Closed, ResolvedPublic

Description

User story & summary:

As a temporary user, I don't want to be linked to Preferences if I can't adjust them, because that would be a poor user experience.

Background & research:

This task is important because it has been decided that:

This task covers any UI adjustments needed to accommodate those requirements.

Questions:

It seems like the main links to Preferences have already been removed. Are there any other areas where Preferences are linked that we need to address for Temporary accounts?

Links to Preferences:

Main user menu.
The link to Preferences has already been removed for Temporary accounts (when testing https://de.wikipedia.beta.wmflabs.org/).

Screenshot 2023-06-12 at 3.32.35 PM.png (426×1 px, 71 KB)

Special:Notifications
The link to Preferences has already been removed for Temporary accounts (when testing https://de.wikipedia.beta.wmflabs.org/).

Screenshot 2023-06-12 at 3.35.48 PM.png (528×1 px, 110 KB)

Note that the entire menu control with Preferences and Mark all read can be removed for Temp accounts:
image.png (1×2 px, 720 KB)

Notifications or Alerts drop down list
The link to Preferences has already been removed for Temporary accounts (when testing https://de.wikipedia.beta.wmflabs.org/).

Screenshot 2023-06-12 at 3.38.50 PM.png (454×1 px, 67 KB)

Ideally the ">" before the All notifications button should also be removed.
image.png (1×2 px, 645 KB)
|
image.png (1×2 px, 517 KB)

Design:

Figma designs

Acceptance Criteria:

Given I'm logged into a temporary account,
When I reach a screen that includes a link to Preferences,
Then those options are either not shown to me (or it's clear that I need to create an account to adjust Preferences)

AND

Determine if there are other user account menus or Special pages that link to Preferences

Event Timeline

Related: https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Echo/+/897854/
(I'm not sure if there is any work needed here, but figured it was a use case we needed to check and test for).

Etonkovidova subscribed.

Checked on dewiki betalabs - all specs are done; the access to Preferences are not present for temp users.

For a quick Design review - should the following be done?

Ideally the ">" before the All notifications button should also be removed.

image.png (1×2 px, 645 KB)
|
image.png (1×2 px, 517 KB)

Presently ">" is still present:

Screen Shot 2023-07-19 at 5.09.41 PM.png (484×1 px, 67 KB)
Screen Shot 2023-07-19 at 5.09.32 PM.png (512×1 px, 64 KB)

Checked on dewiki betalabs - all specs are done; the access to Preferences are not present for temp users.

For a quick Design review - should the following be done?

Ideally the ">" before the All notifications button should also be removed.

image.png (1×2 px, 645 KB)
|
image.png (1×2 px, 517 KB)

Presently ">" is still present:

Screen Shot 2023-07-19 at 5.09.41 PM.png (484×1 px, 67 KB)
Screen Shot 2023-07-19 at 5.09.32 PM.png (512×1 px, 64 KB)

I am fine to let this one go (ie not remove the ">") if @KStoller-WMF prefers to keep it as lightweight as possible for MVP.

KStoller-WMF claimed this task.

In T338908#9031157, @RHo wrote:
I am fine to let this one go (ie not remove the ">") if @KStoller-WMF prefers to keep it as lightweight as possible for MVP.

Let's keep this simple and leave the ">". Thanks for the thorough check, @Etonkovidova !