Last modified: 2012-09-18 13:27:16 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 T42325, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 40325 - mw.util.$content changed for Vector
mw.util.$content changed for Vector
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
ResourceLoader (Other open bugs)
1.20.x
All All
: Normal normal (vote)
: ---
Assigned To: Daniel Friesen
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-18 10:02 UTC by JulesWinnfield-hu
Modified: 2012-09-18 13:27 UTC (History)
5 users (show)

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


Attachments

Description JulesWinnfield-hu 2012-09-18 10:02:32 UTC
According to [[mw:ResourceLoader/Default modules#$content]], and up until 13 September 2012, the mw.util.$content should mean the #bodyContent for the Vector skin on huwiki. Now is the #content, causing unwanted behavior.
Comment 1 Liangent 2012-09-18 10:09:27 UTC
I prefer the idea that this should be a documentation bug. IMHO it should be the element who directly holds parser output HTML from wikitext of a page.
Comment 2 Daniel Friesen 2012-09-18 10:15:38 UTC
No it shouldn't. $content has only ever been used by core and the extensions I could find to refer to the overall content area (including ui, etc...).

If you want the actual content text use `$( '#mw-content-text ')`.

Updated the documentation.
Comment 3 Krinkle 2012-09-18 13:27:16 UTC
Indeed. #bodyContent is never the right selection, it never was.

* The raw page output is #mw-content-text. This ID is fixed by the MediaWiki output handler and not overwritten by skins.
* The full page output is a few levels up, and depends on the skin. In Vector it is #content. Because this is variable, mw.util.$content helps.
* #bodyContent is somewhere in between.

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


Navigation
Links