Page MenuHomePhabricator

Save dialog buttons showing on pages they shouldn't
Closed, ResolvedPublic

Description

The "Review changes" and "Show preview" buttons should not show once you click on them. This regressed around 0.42.0 which leads me to suspect this commit: https://gerrit.wikimedia.org/r/c/oojs/ui/+/686439

Expected:

image.png (312×913 px, 41 KB)

Actual:

image.png (303×914 px, 44 KB)

Event Timeline

Esanders added a subscriber: Jdforrester-WMF.

Reverting that patch locally fixes the issue, so it is indeed the cause of the regression. Thanks to @Jdforrester-WMF for spotting this.

Change 771913 had a related patch set uploaded (by Esanders; author: Esanders):

[oojs/ui@master] Revert "Fix for OO.ui.ActionSet.prototype.get() not returning invisible widgets"

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

Change 772345 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/VisualEditor@master] Fix review button shown when review panel is already active

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

Change 771913 abandoned by Esanders:

[oojs/ui@master] Revert "Fix for OO.ui.ActionSet.prototype.get() not returning invisible widgets"

Reason:

Fixed downstream.

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

Change 772345 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Fix review button shown when review panel is already active

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

Looks like I filed the eventual cause of this a few years ago: T174497

@Esanders One last thing before I move this task. See https://photos.app.goo.gl/13RoWw1MtjoBVG6P8, pay attention to 00:20 - 00:27 - is the "Show preview" button expected to be visible in the preview page? That has been my experience with it but I'd like to confirm that it's how it should be.

Cc: @matmarex

Yes - that is correct behaviour in the old editor, as you can change the wikitext above, and then press "show preview" again to update the preview.

ppelberg claimed this task.