Help needed!
Okay so I made a start to this. I'm probably not going to get time to work on this during this week, but if anyone wants to pick this up and accelerate it through to completion be my guest. I will love you forever. Otherwise I'll continue the work next week.
https://gerrit.wikimedia.org/r/118345 https://gerrit.wikimedia.org/r/118347
On Wed, Mar 12, 2014 at 12:23 PM, Trevor Parscal tparscal@wikimedia.org wrote:
I'm going to start working on some RL modifications to make it possible for skins outside of core to add skinStyles to other modules, which will help with making non-core skins equally capable.
- Trevor
On Wed, Mar 12, 2014 at 12:00 PM, Jon Robson jdlrobson@gmail.com wrote:
Yes this is an orthogonal conversation. If it's that easy for a core change to break a skin outside core, then there are lots of fundamentally wrong things with our skin system, one being the fact that modules added with OutputPage get added to all skins even if they might not be compatible with them. If we want to talk about this I'd encourage you to start a new thread.
On Wed, Mar 12, 2014 at 11:57 AM, Matthew Flaschen mflaschen@wikimedia.org wrote:
On 03/11/2014 05:21 PM, Jon Robson wrote:
If people forget they exist I would say that equates to no one cares about them and no one maintains them.
Isarra was referring to skins outside of core.
There's a difference between core developers forgetting non-core skins exist, and the developers of non-core skins forgetting.
If we had a proper skin API, people wouldn't explicitly need to think
about
Foo non-core skin, but they would need to think about managing the
changes
to the skin API (the same way we manage other API changes).
Matt Flaschen
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
-- Jon Robson
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l
Wikitech-l mailing list Wikitech-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikitech-l