Last modified: 2014-05-30 19:12:24 UTC
The id and class names for Special:PageMigration (a GSoC project) need to be fixed/altered to meet the current standards followed by MediaWiki. Refer: https://gerrit.wikimedia.org/r/#/c/133720/17/specials/SpecialPageMigration.php
Hi, what does ""as per the standards followed"" mean? What is the bug here? It's unclear from reading this ticket what the actual problem is. :( Also see https://www.mediawiki.org/wiki/How_to_report_a_bug for general info.
Hi Andre, by standards, I mean the general guidelines and principles followed on MediaWiki. If you look into the code, I have just given trivial names which came to my mind. This needs to be fixed to make the code production-level.
Created attachment 15475 [details] From current code to fix: "html template" which shows the tree structure/html elements in context to understand the meaning BPositive produced this for Nikerabbit to comment how he wants the classes and IDs to be named.
Yeah, these IDs are quite generic. Can easily clash with things and seem out of place in a MediaWiki context. Relevant conventions: https://www.mediawiki.org/wiki/Manual:Coding_conventions/CSS#Naming
Some reading: http://csswizardry.com/2013/01/mindbemding-getting-your-head-round-bem-syntax/ Even if we don't use that syntax (which I think we should in this case), the mindset of part-of and kind-of is very helpful.
Change 135750 had a related patch set uploaded by BPositive: ID and class names altered with mw-tpm-sp prefix https://gerrit.wikimedia.org/r/135750
Change 135750 merged by jenkins-bot: ID and class names fixed for Special:PageMigration with mw-tpm-sp prefix https://gerrit.wikimedia.org/r/135750
Assuming fixed, further issues to be reported separately.