Page MenuHomePhabricator

Add a link: Yes button flickers when toggling states
Closed, ResolvedPublicBUG REPORT

Assigned To
Authored By
mewoph
May 5 2021, 5:31 PM
Referenced Files
F34443960: flicker3_button.gif
May 6 2021, 6:10 PM
F34443956: flicker_button.gif
May 6 2021, 6:10 PM
F34442196: addlink_yesToggle.gif
May 5 2021, 5:40 PM
F34442188: addlink_flicker.gif
May 5 2021, 5:31 PM

Description

List of steps to reproduce (step by step, including full links if applicable):

  • Enable link recommendations
  • Accept recommendation
  • Un-accept recommendation

What happens?:
The button flickers before showing toggled/untoggled state.

What should have happened instead?:
The button state should be updated without flickering.

addlink_flicker.gif (1,200×880 px, 317 KB)

Event Timeline

Change 685538 had a related patch set uploaded (by MewOphaswongse; author: MewOphaswongse):

[mediawiki/extensions/GrowthExperiments@master] Add a link: don't reset button states when toggling

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

Change 685538 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Add a link: don't reset button states when toggling

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

Etonkovidova subscribed.

Checked in betalabs - below is animated gifs to compare (betalabs with the fix vs production without the fix)

betalabs(click to animate)testwiki (click to animate)
flicker_button.gif (405×266 px, 27 KB)
flicker3_button.gif (453×275 px, 37 KB)