| • alexhollender_WMF | |
| May 27 2021, 12:52 PM |
| F34489173: Screen Shot 2021-06-10 at 12.42.43 AM.png | |
| Jun 10 2021, 7:44 AM |
| F34478631: Screen Shot 2021-06-02 at 10.42.53 AM.png | |
| Jun 2 2021, 5:43 PM |
| F34478624: Screen Shot 2021-06-02 at 10.40.19 AM.png | |
| Jun 2 2021, 5:43 PM |
| F34478629: Screen Shot 2021-06-02 at 10.42.41 AM.png | |
| Jun 2 2021, 5:43 PM |
| F34478627: Screen Shot 2021-06-02 at 10.42.16 AM.png | |
| Jun 2 2021, 5:43 PM |
| F34472810: talk.png | |
| May 28 2021, 1:49 AM |
| F34471936: image.png | |
| May 27 2021, 12:52 PM |
| F34471933: image.png | |
| May 27 2021, 12:52 PM |
Currently, as a carryover from the behavior in Legacy Vector, when you have a new message on your Talk page the Talk link is removed from the menu and shown as an alert (yellow box) in the site header.
 |  |
The Talk link should remain in the user menu regardless of whether or not the alert is present.
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T283811#7147841 |
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Add talk notification instead of transforming the talk link | mediawiki/extensions/Echo | master | +9 -16 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | ovasileva | T266536 [EPIC] Consolidate user links into a single menu | |||
| Resolved | ovasileva | T274428 User Links: Handling of talk page notification | |||
| Resolved | ovasileva | T283811 'Talk' link should always be in user menu , even when new user messages bar is shown |
Change 695648 had a related patch set uploaded (by Nray; author: Nray):
[mediawiki/extensions/Echo@master] POC: Add talk notification instead of transforming the talk link
I think the next question to figure out is whether we want this to only affect modern Vector or whether we want this in other skins as well (e.g legacy Vector, etc). We could, for example, get rid of the transformation logic in other skins and always have the talk link show e.g. in legacy Vector it could look like:
The logic in Echo to do this would be easy (see above patch), but we would have to be careful to not break anything that depends on #pt-mytalk [1] and expects that to be a notification. That's the tricky part.
[1] https://codesearch.wmcloud.org/search/?q=pt-mytalk&i=nope&files=&excludeFiles=&repos=
@alexhollender we took a stab at estimating this, and came out large due to an open question.
This will result in two talk page links in the menu, with different labels. Currently Nick proposes to do this for all skins. Here is how it would look for legacy Vector:
@alexhollender does this make sense or do we only want to do this for modern Vector?
@alexhollender just to make it explicit, it sounds like you are okay with changing the default for all skins to show this link twice?
yes, I feel relatively confident that such a change won't rock the boat too much for legacy vector, monobook, and modern users. Though it might make sense to wait for @ovasileva to return before we finalize this?
This seems fine to me and can't envision any objections from the communities. @MMiller_WMF, @RHo - let us know if you have any concerns with this new treatment.
Thanks @ovasileva - no concerns from me. New users wouldn't come across legacy skins too often anyway, and it seems prudent to have this become an extra extra 'notification' link rather than the transformation of a global menu item for everyone.
Change 695648 merged by jenkins-bot:
[mediawiki/extensions/Echo@master] Add talk notification instead of transforming the talk link
Status: ✅ PASS
Environment: beta
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA
Test Artifact(s):
✅ AC1: The Talk link should remain in the user menu regardless of whether or not the alert is present.
