Last modified: 2012-10-23 17:11:17 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T43267, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 41267 - [Regression] mediawiki.action.edit.preview is not loaded at Commons despite "Use live preview" (wpuselivepreview) is activated in [[Special:Preferences]]
[Regression] mediawiki.action.edit.preview is not loaded at Commons despite "...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
1.20.x
All All
: High normal (vote)
: ---
Assigned To: Krinkle
: code-update-regression
Depends on:
Blocks: 41298
  Show dependency treegraph
 
Reported: 2012-10-22 09:28 UTC by Rainer Rillke @commons.wikimedia
Modified: 2012-10-23 17:11 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Rainer Rillke @commons.wikimedia 2012-10-22 09:28:29 UTC
Tested with IE 8 and latest RC of FF.

Current behaviour:
When clicking the "preview" or "show diff" button, the page reloads.

Evidence:
...after the DOM is ready...
...executed in JavaScript console...
> mw.loader.getState('mediawiki.action.edit.preview');
'registered'

Expected behaviour:
When clicking the "preview" or "show diff" button, live preview just replaces some elements in the DOM. No page reload.

> mw.loader.getState('mediawiki.action.edit.preview');
'ready'

How to reproduce:
Set your interface language to de or en, enable "Use live preview" in [[Special:Preferences]], go to a page, edit a section, click preview.

Note:
Using manually
mw.loader.load('mediawiki.action.edit.preview');
works perfectly: After it is loaded, LivePreview works as expected.
Comment 1 Rainer Rillke @commons.wikimedia 2012-10-22 09:32:45 UTC
Commons runs 1.21wmf2

Also tested at dw.wp which runs 1.21wmf1 and there it works. So my guess is that the bug slipped-in between the two versions.
Comment 2 Rainer Rillke @commons.wikimedia 2012-10-22 09:38:05 UTC
Used Vector skin to test
Comment 3 Andre Klapper 2012-10-22 16:28:36 UTC
(Regression => keyword code-update-regression)
Comment 4 Sam Reed (reedy) 2012-10-22 16:50:43 UTC
In the 1.20 release-notes:

* Live preview functionality has been improved and moved into the
 'mediawiki.action.edit.preview' module. The old 'mediawiki.legacy.preview' module
  has been removed.

Looking at core, the only usage of 'mediawiki.action.edit.preview' is in the resource definition

Original commit:
https://github.com/mediawiki/core/commit/4d4bfc27a1bedf6b0d1670707ac812fe36fe85ad

EditPage.php now:

		if ( $wgUser->getOption( 'uselivepreview', false ) ) {
			$wgOut->addModules( 'mediawiki.legacy.preview' );
		}

It got killed by https://github.com/mediawiki/core/commit/bb51a58e575b13eb8b6c496482f7066a7081f676

https://gerrit.wikimedia.org/r/29339
Comment 5 Rainer Rillke @commons.wikimedia 2012-10-23 16:10:27 UTC
https://bugzilla.wikimedia.org/show_activity.cgi?id=41267

I am confused. Please tell me what is "right" now?

Using [Regression] in bug title or Keyword "code-update-regression" or both?
Comment 6 Andre Klapper 2012-10-23 16:16:34 UTC
I guess different people have different habits, so I change my mind and say: Do what you want with the bug subject, but having the keywrod set would be really great." Sorry for any confusion.
Comment 7 Krinkle 2012-10-23 17:11:17 UTC
(In reply to comment #5)
> https://bugzilla.wikimedia.org/show_activity.cgi?id=41267
> 
> I am confused. Please tell me what is "right" now?
> 
> Using [Regression] in bug title or Keyword "code-update-regression" or both?

They keyword is what counts. The reason various people prefer to (sometimes) include (parts of) keywords or components (e.g. "[Regression]", "VisualEditor: .." etc.) is because most interfaces with bugzilla content lack proper indication of components and tags, so we put them in the subject so that they can be seen directly (e.g. in irc notifications for bug changes and in e-mail notifications[1]).

[1] Yes, for some of the fields there are mail headers sent, but that is only useful for querying e-mails, not when browsing them in any mail applications plain list view.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links