Last modified: 2014-11-19 18:32:52 UTC
Right now, the way that the save button is disabled when you click it is that MWSaveDialog listens to the button's click event, in response to that it fires the 'save' event, ViewPageTarget gets that event, reaches into the save dialog and disables the button. This is only one of many insane things about the totally dysfunctional MWSaveDialog - ViewPageTarget relationship; the whole way these two interact should be refactored.
This'd be lovely to get fixed properly. Alex, could you take a look at this at some point? I imagine Roan'd love to be involved in the refactor, though. :-)
Mostly fixed in Gerrit change #135945 but Roan said there are still some outstanding issues.
Yeah, that patch wasn't really intended to fully fix this.
Change 140868 had a related patch set uploaded by Alex Monk: Move some stuff from ViewPageTarget to MWSaveDialog https://gerrit.wikimedia.org/r/140868
Change 140868 abandoned by Alex Monk: Move some stuff from ViewPageTarget to MWSaveDialog https://gerrit.wikimedia.org/r/140868