Page MenuHomePhabricator

Investigate temp user banner in more skins
Closed, ResolvedPublic

Description

Background

In Vector-22 and Minerva skins, temporary accounts see a banner at the top of the page containing helpful information including their name, links to further information and prompts to create an account:

image.png (209×723 px, 17 KB)

More work is needed in order to enable this banner on more skins. For example, on Vector legacy it overlaps the top navigation elements:

image.png (52×532 px, 11 KB)

However, it would be helpful to have this information available in more situations.

Investigation

Investigate how we could add the temporary user banner in more skins.

See also: T409777: Should notifications work for temporary users on legacy Wikimedia skins?

Event Timeline

Change #1212200 had a related patch set uploaded (by Tchanders; author: Tchanders):

[mediawiki/core@master] WIP Add inline option for the temp user banner

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

Change #1212201 had a related patch set uploaded (by Tchanders; author: Tchanders):

[mediawiki/skins/Vector@master] WIP Add inline temporary user banner to legacy skin

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

As discussed elsewhere, we could add the banner just above where the site notice is displayed, e.g.:

Vector legacy
image.png (366×1 px, 50 KB)
Monobook
image.png (295×1 px, 40 KB)

The patches above show how this could be done.

Moving to "Needs review" for review of the general approach. The patches are still WIP.

Change #1212200 abandoned by Tchanders:

[mediawiki/core@master] WIP Add inline option for the temp user banner

Reason:

Agreed that we will do this a different way

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

Change #1212201 abandoned by Tchanders:

[mediawiki/skins/Vector@master] WIP Add inline temporary user banner to legacy skin

Reason:

Agreed that we will do this a different way

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

Closing this, now we've investigated and discussed it. It can be done with relatively low effort (see POC patches), but we're choosing not to do this for now since it would add to the maintenance load.