Page MenuHomePhabricator

Donor badge experiment: Final design review and adjustments for donor badge
Closed, ResolvedPublic2 Estimated Story Points

Description

Background

This task is for final design tweaks for the donor badge experience

Requirements

  • @Sneha is able to review and list out final design changes
  • Changes are made to the styling of the donor badge and the other treatment UI elements
  • Final design sign-off on the donor badge

Testing steps

Recommended to test on test.wikipedia.org

  1. In incognito window visit https://thankyou.wikipedia.org/wiki/Thank_You/en?country=US
    1. Visit https://test.wikipedia.org/wiki/Main_Page?mpo=donor-delight-badge:treatment-b-simple or
    2. https://test.wikipedia.org/w/index.php?title=Main_Page&mpo=donor-delight-badge%3Atreatment-c-delightful

Acceptance criteria for done

  • Final design sign-off on the donor badge accomplished
    • Simple (B) & Delightful (C): In both treatment groups, hovering on the badge removes the white border (outline). The white border should not disappear instead there was this on tap style explored by @Volker_E that would be nice to have. Note: Went with header button style for now.
    • Simple (B) & Delightful (C): Popover is over viewport, slightly too wide
    • Simple (B) & Delightful (C): Popover (arrow pointing) is slightly too far down
    • Simple (B) & Delightful (C): On confirmation dialog the white badge border is not there. It should look the same as the badge on the header with a shadow outline so that the white border becomes visible.
    • Simple (B) & Delightful (C): badge itself mistakenly received CSS custom properties instead of Less vars (originated in static file and not replaced in handover)
    • Delightful (C): the first time pop up pointing to badge is not appearing not reproducible
    • Delightful (C): "Remove badge" button to use progressive normal style
    • Delightful (C): Unify on "Remove" for all badge removing activities - https://gerrit.wikimedia.org/r/c/1300171

Event Timeline

bwang renamed this task from Final design review and adjustments for donor badge to Donor badge experiment: Final design review and adjustments for donor badge.May 28 2026, 4:56 PM
bwang triaged this task as Medium priority.
bwang updated the task description. (Show Details)

Some things we need to fix:

  1. In both versions, hovering on the badge removes the border. The white border should not disappear instead there was this on tap style explored by @Volker_E that would be nice to have.
  2. For delight version - the first time pop up pointing to badge is not appearing.
  3. On confirmation dialog the white badge border is not there. It should look the same as the badge on the header with a shadow outline so that the white border becomes visible.
  4. "Hide badge" button right now is using a style that blends with the article page and makes the button less visible. Could we use progressive normal style or is there a different style we use for floating buttons?
Volker_E updated the task description. (Show Details)

@Sneha Thanks, I'd add:
Delight (C)

  • the popup is a bit too far away from the badge still
  • the badge itself, not the one in the popup mistakenly received custom vars instead of Less vars, which could (minimally) impact anon users in dark mode (drop shadow being invisible)

At best, we're putting all these changes in smallest amount of patches to reduce backporting overhead.

@Sneha I'd also ask once more about the "Yes, hide the badge" label. It might

  1. lead to translation overflow in one of our audience wikis (German: "Ja, verstecke den Badge")
  2. We should unify on "remove badge" in both treatments – B already uses remove.
  3. We don't use full sentences in our labels, it should be active first person: "Remove badge". The importance is delivered with destructive treatment, not via "Yes" in the label.

Agree?

Change #1300171 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/WikimediaCustomizations@master] Donor Delight Badge, styles: Amending to final design review feedback

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

@Volker_E sounds good. I am attaching a mock for alignment.

Hide badge prompt.png (384×765 px, 108 KB)

@Volker_E and I had a quick async discussion and we have decided to replace "hide" to "remove" based on Volker's suggestion. Adding here for documentating decisions.

Change #1300186 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/WikimediaCustomizations@master] Donor Delight Badge: Unify on "Remove badge" language across treatments

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

@Volker_E sounds good. I am attaching a mock for alignment.

Hide badge prompt.png (384×765 px, 108 KB)

@Sneha @Volker_E What should the body text be? this is whats in the patch now but it read a bit funny to me https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikimediaCustomizations/+/1300186

"This heart is a thank you badge for your donation. Removing it from your view won't let you restore it."

Some things we need to fix:

  1. In both versions, hovering on the badge removes the border. The white border should not disappear instead there was this on tap style explored by @Volker_E that would be nice to have.
  2. For delight version - the first time pop up pointing to badge is not appearing.
  3. On confirmation dialog the white badge border is not there. It should look the same as the badge on the header with a shadow outline so that the white border becomes visible.
  4. "Hide badge" button right now is using a style that blends with the article page and makes the button less visible. Could we use progressive normal style or is there a different style we use for floating buttons?

@Sneha

  1. its working for me, perhaps its not appearing cuz you already saw the popup when you were testing treatment b. paste this into the console to reset the experience to the first time: mw.user.clientPrefs.set( 'minerva-badge', '0' );
  2. this is what im seeing for treatment C. isnt this the progressive normal style already?
    Screenshot 2026-06-10 at 4.15.07 PM.png (572×318 px, 20 KB)

@Volker_E sounds good. I am attaching a mock for alignment.

@Sneha @Volker_E What should the body text be? this is whats in the patch now but it read a bit funny to me https://gerrit.wikimedia.org/r/c/mediawiki/extensions/WikimediaCustomizations/+/1300186

"This heart is a thank you badge for your donation. Removing it from your view won't let you restore it."

Your copy proposal @bwang made more sense than the former: "This heart is a thank you badge for your donation. The badge cannot be restored after it is removed." I'd suggest we go with that.

@bwang looks like were looking at different builds. Based on your screenshot for the floating button it looks good.

Change #1300842 had a related patch set uploaded (by Anne Tomasevich; author: VolkerE):

[mediawiki/extensions/WikimediaCustomizations@wmf/1.47.0-wmf.6] Donor Delight Badge: Unify on "Remove badge" language across treatments

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

Change #1300186 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@master] Donor Delight Badge: Unify on "Remove badge" language across treatments

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

Volker_E set the point value for this task to 2.

Change #1300171 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@master] Donor Delight Badge, styles: Amending to final design review feedback

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

Change #1300896 had a related patch set uploaded (by Anne Tomasevich; author: VolkerE):

[mediawiki/extensions/WikimediaCustomizations@wmf/1.47.0-wmf.6] Donor Delight Badge, styles: Amending to final design review feedback

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

Change #1300842 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@wmf/1.47.0-wmf.6] Donor Delight Badge: Unify on "Remove badge" language across treatments

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

Change #1300896 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@wmf/1.47.0-wmf.6] Donor Delight Badge, styles: Amending to final design review feedback

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

Mentioned in SAL (#wikimedia-operations) [2026-06-11T20:17:31Z] <jdrewniak@deploy1003> Started scap sync-world: Backport for [[gerrit:1300842|Donor Delight Badge: Unify on "Remove badge" language across treatments (T427313)]], [[gerrit:1300843|[A11y] Donor Badge: Remove Badge button disappears too quickly (T428646)]], [[gerrit:1300896|Donor Delight Badge, styles: Amending to final design review feedback (T427313)]]

Mentioned in SAL (#wikimedia-operations) [2026-06-11T20:34:51Z] <jdrewniak@deploy1003> annet, jdrewniak: Backport for [[gerrit:1300842|Donor Delight Badge: Unify on "Remove badge" language across treatments (T427313)]], [[gerrit:1300843|[A11y] Donor Badge: Remove Badge button disappears too quickly (T428646)]], [[gerrit:1300896|Donor Delight Badge, styles: Amending to final design review feedback (T427313)]] synced to the testservers (see https://wikitech.wikimedia.org/wiki/Mwdebug

Mentioned in SAL (#wikimedia-operations) [2026-06-11T20:51:42Z] <jdrewniak@deploy1003> Finished scap sync-world: Backport for [[gerrit:1300842|Donor Delight Badge: Unify on "Remove badge" language across treatments (T427313)]], [[gerrit:1300843|[A11y] Donor Badge: Remove Badge button disappears too quickly (T428646)]], [[gerrit:1300896|Donor Delight Badge, styles: Amending to final design review feedback (T427313)]] (duration: 34m 10s)

After reviewing T425459: Donor badge experiment: Setup donor badge, I decided to add my notes here to keep UI issues in one place.

Checked in enwiki beta

Issues:
(1) there is typography mismatch between Figma and codex (a known issue; see also https://phabricator.wikimedia.org/T419395#12003722)
The popover displays small font vs medium font specified in Figma:

enwiki betaFigma
#minerva-badge-popover h3 {font-size: var(--font-size-x-small,0.75rem)
Screenshot 2026-06-11 at 7.24.43 PM.png (924×1,460 px, 178 KB)
Screenshot 2026-06-11 at 7.44.52 PM.png (776×1,570 px, 488 KB)

(2) Accessibility check identifies some issue with the element:

Screenshot 2026-06-11 at 7.31.08 PM.png (918×992 px, 136 KB)

(3) I did not see the popover for 'donor-delight-badge' group - when the heart icon clicked, the animation starts immediately. I tested it on enwiki beta and on https://test.wikipedia.org/w/index.php?title=Main_Page&mpo=donor-delight-badge%3Atreatment-c-delightful (click on the gif below)

heart yellow_animation.gif (448×787 px, 1 MB)

Thanks @Etonkovidova.
(1) and (2) are easily fixable. I have actually wondering about the same thing on too small font-size, but when looking up Figma, it seemed the right tokens. Pls just shortly verify @Sneha.
With (3) I'm a bit confused, as this has been behaving differently before. I'd also appreciate @bwang eyes on it.

Thanks @Etonkovidova.
(1) and (2) are easily fixable. I have actually wondering about the same thing on too small font-size, but when looking up Figma, it seemed the right tokens. Pls just shortly verify @Sneha.

Thanks, @Volker_E!

Just to clarify
Regarding (1) issue - the comment I mentioned (https://phabricator.wikimedia.org/T419395#12003722) refers to a similar problem that Growth team had. There is a follow-up comment from @DTorsani-WMF (https://phabricator.wikimedia.org/T419395#12004453) - not quite sure if it fixes the tokens mismatch and some additional patch(es) are needed.

Regarding (2) - as I mentioned it's an old accessibility issue, not introduced by any recent changes. I just thought since we are working on Watch label, probably it'd be good to fix it as well.

With (3) I'm a bit confused, as this has been behaving differently before. I'd also appreciate @bwang eyes on it.

@bwang
Re-checked - turned out the behavior is trickier than I expected. I guess it's by design?

  • First time on a page (beta or testwiki)I use mw.user.clientPrefs.set( 'minerva-badge', '1' );
  • then, I use

mw.testKitchen.overrideExperimentGroup('donor-delight-badge', 'treatment-c-delightful');

The badge appears and the popover appears:

Screenshot 2026-06-12 at 9.14.28 AM.png (922×1,306 px, 243 KB)

Then, after tapping - the animation starts and "Remove badge" floating button appears at the button:

Screenshot 2026-06-12 at 9.39.40 AM.png (880×1,848 px, 1 MB)

After that, I don't see the the popover anymore.

Change #1301392 had a related patch set uploaded (by VolkerE; author: VolkerE):

[mediawiki/extensions/WikimediaCustomizations@master] Donor Delight Badge: Add accessible label and hide popover from AT

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

Change #1301392 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@master] Donor Delight Badge: Add accessible label and hide popover from AT

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

Change #1303517 had a related patch set uploaded (by Jdlrobson; author: VolkerE):

[mediawiki/extensions/WikimediaCustomizations@wmf/1.47.0-wmf.6] Donor Delight Badge: Add accessible label and hide popover from AT

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

Change #1303517 merged by jenkins-bot:

[mediawiki/extensions/WikimediaCustomizations@wmf/1.47.0-wmf.6] Donor Delight Badge: Add accessible label and hide popover from AT

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

Mentioned in SAL (#wikimedia-operations) [2026-06-17T22:14:08Z] <jdlrobson@deploy1003> Started scap sync-world: Backport for [[gerrit:1303517|Donor Delight Badge: Add accessible label and hide popover from AT (T427313)]]

Mentioned in SAL (#wikimedia-operations) [2026-06-17T22:31:50Z] <jdlrobson@deploy1003> jdlrobson: Backport for [[gerrit:1303517|Donor Delight Badge: Add accessible label and hide popover from AT (T427313)]] synced to the testservers (see https://wikitech.wikimedia.org/wiki/Mwdebug). Changes can now be verified there.

Mentioned in SAL (#wikimedia-operations) [2026-06-17T22:45:09Z] <jdlrobson@deploy1003> Finished scap sync-world: Backport for [[gerrit:1303517|Donor Delight Badge: Add accessible label and hide popover from AT (T427313)]] (duration: 31m 01s)

The last AC is done (checked on `testwiki wmf.8):

Simple (B) & Delightful (C): On confirmation dialog the white badge border is not there. It should look the same as the badge on the header with a shadow outline so that the white border becomes visible.

Screenshot 2026-06-23 at 1.54.05 PM.png (894×692 px, 162 KB)

Sneha claimed this task.
Sneha added a subscriber: HFan-WMF.