Last modified: 2011-03-09 00:11:21 UTC
Special:SpecialPages should show if such special page is out of use.
Typically Special:WantedPages is off, thus there should be some notice about that on Special:SpecialPages, say:
* Wanted pages (off)
Created attachment 8266 [details]
Functional patch v1
My first attempt to hack MW, feel free to adjust / reject / laugh at it at will.
In short, it moves over to a semantically valid CSS-only markup, then readds the bolding for restricted pages and adds a new class for cache-only pages. Then it adds italics as the default styling for these.
It could just as easily leave them un-styled by default, but I suppose then the new default notes message might look a little odd.
Patch looks good to me. Have not tested yet.
OK, tested the patch. Found a few small issues:
- If a special page is both expensive and restricted, the classes get concatenated with no space in between. I'd suggest using an array instead and imploding it.
- The specialpages-note message is only shown if the list includes restricted pages. The condition should be changed to also show it if there are cached pages.
(Also, ideally the message would be split so that we could only show the parts matching pages actually present in the list, but I think it's good enough as it is.)
-need-review +reviewed per Iimari
Created attachment 8268 [details]
Thanks for the review. Yes, I was going to implement an OR feature for showing the notes, seems reasonable (obviously forgot).
Created attachment 8269 [details]
Oops. Meant to attach this one instead. Sorry all.
Created attachment 8270 [details]
Okay, this is definitely the patch I meant to attach this time. I think. :)
Applied with a few tweaks in r83554. I decided to move the isExpensive() method up to the SpecialPage class and get rid of the method_exists() kluge -- it's IMO more elegant this way.
Oh, and thanks!