"George Orwell III" posted a comment on MediaWiki.r111647.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/111647#c31997

Commit summary for MediaWiki.r111647:

(Bug 31417) Content-holding div needs an ID.
Since the introduction of the class mw-content-ltr/rtl, it has been used to get 
the page text which is very bad because there are two class names and it is not 
always on the same place (if at all). This ID will always hold the bodytext 
(excluding categories, ArticleFeedback, ...).

George Orwell III's comment:

A lot of people on en.wikisource were wondering how to use the ms-content-ltr 
container to replace an interim container scheme used for dynamic layouts of 
transcluded content once it went through the ProofReading process. This 
additional ID generated skin-wide would greatly help cut down on the amount of 
local .js currently needed (not to mention help stop abusing #contentSub where 
the interim container is being dynamically moved to).

Any chance this revision slated for 1.20wmf with the new ID attribute 
generation could be hurried along? Patching the current wikisource deployed 
version sooner rather than later sure could go a long way in undoing our 
dynamic layout mess moving forward. TIA - GOIII

_______________________________________________
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to