Extension:NewPageCSS

Jump to: navigation, search

I notice you've updated the Extension:NewPageCSS page. This extension appears to be a copy of Extension:PageCSS (which is in SVN, but currently broken) with just a couple of minor changes:

  • The author's name contains accented characters; the change made from PageCSS to NewPageCSS is just to HTML-encode these (é is é for example)
  • The parser tag didn't originally return a value - it just stuffed the CSS into the HTML page header and exited. MW 1.18+ requires it return something - so now it returns the empty string "" as it doesn't have anything to put inline in the page body but needs a return value to lose the UNIQ-QINU tags.

Perhaps NewPageCSS should simply be merged *back* into PageCSS as these are small changes to a small extension, not a full rewrite worthy of a new name? I'm not sure if I should raise the question here, though, or through Bugzilla (as the old broken version is in SVN)? --Carlb (talk) 16:53, 16 February 2012 (UTC)

Carlb (talk)16:53, 16 February 2012

Hi Carlb, I just tidied up the extensions page a bit. Yes, I agree that the extension in SVN should be fixed to make this one obsolete. I guess opening a bug is a way or finding someone with commit access to do it. Perhaps Varnent will help on this one. Cheers

[[kgh]] (talk)17:00, 16 February 2012

Opened as bugzilla:34453 with the "new" PageCSS.php source attached. --Carlb (talk) 19:44, 16 February 2012 (UTC)

Carlb (talk)19:44, 16 February 2012

Cool, I will archive NewPageCSS as soon as this is done. Thank you for your help. Cheers

[[kgh]] (talk)19:50, 16 February 2012