[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2011-03-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

Max Semenik  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||WONTFIX

--- Comment #13 from Max Semenik  2011-03-08 09:05:50 
UTC ---
Closing per Krinkle. The only use case presented is non-admin styling of
templates, hovewer all popular templates on large wikis are protected anyway.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2011-03-08 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

Krinkle  changed:

   What|Removed |Added

 CC||krinklem...@gmail.com

--- Comment #12 from Krinkle  2011-03-08 08:49:56 UTC ---
Suggesting LATER or WONTFIX per the mentioned problems and complexity with
parser cache and linking tables.

If the usecase is style, contact your wiki admin and ask him to add some
css-classes to MediaWiki:Common.css.

The root tag of all MediaWiki-pages contains (among others) a class for the
current skin.



so, something like the following should work fine if you really need something
in the wiki to be skin-dependend. Add the following to MediaWiki:Vector.css

/* Make the table of contents white-on-black in Monobook! */
#toc { background-color: black; color: white; }

or if you have several different colors for different skins, it may be easier
to keep them together, instead the following in MediaWiki:Common.css

/* Funky wikitables */
.mediawiki .wikitable td { background-color: black; /* default */ }
.skin-vector .wikitable td { background-color: #8da7d6 }
.skin-monobook .wikitable td { background-color: #00FF00 }

Or if you *really* need to switch the actual article contents (usecase
example?) even this would work:

Template:SkinSwitch

{{{vector|}}}
{{{monobook|}}}
{{{default|}}}

MediaWiki:Common.css
.skinswitch { display: none; }
.skinswitch-default { display: block; }

.skin-vector .skinswitch-default,
.skin-monobook .skinswitch-default { display: none; }

.skin-vector .skinswitch-vector,
.skin-monobook .skinswitch-monobook { display: block; }

User:Example

{{SkinSwitch
|vector=Hello fellow Vectorians,
|monobook=Howdy Monobooky,
|default=Hi there,
}}

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2011-01-13 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

Daniel Friesen  changed:

   What|Removed |Added

 CC||mediawiki-bugs@nadir-seen-f
   ||ire.com

--- Comment #11 from Daniel Friesen  
2011-01-14 00:02:39 UTC ---
This would fragment the cache even more. Imho something like this that
fragments the cache with little extra purpose should be isolated to an
extension.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2010-11-07 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

--- Comment #10 from Platonides  2010-11-07 21:00:17 UTC 
---
That may seem logical to you, but the fact is, it would use your default skin.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2010-11-06 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

--- Comment #9 from MC10  2010-11-07 05:13:55 UTC ---
(In reply to comment #7)
> How would this respond when there's no skin? The only situations I can think 
> of
> would be action=parse in the API and action=render via index.php, though 
> parsed
> system messages might also be affected. 

#default should be triggered if there is no skin, since it will not match any
of the test cases.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2010-11-02 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

--- Comment #8 from Platonides  2010-11-02 17:19:39 UTC 
---
> It may be possible to use MediaWiki:.css, but as a non-admin, it is a 
> bit
> more difficult to make changes.
I am aware of such inconvenience, but despite of it, still seems a better
system.


> not it's in a #ifeq: or not. If it's not
> detected by anti-vandalism tools, it may be a bit harder to remove, but it
> shouldn't be too difficult for someone who searches up "" to
> remove it.

Until you get that profanity included in a template included only for a few
uncommon skins from a template three levels deep. Which makes me thing,
including links or templates under a {{CURRENTSKIN}} branch would produce
another instance 
into bug 14404.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2010-11-01 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

Alex Z.  changed:

   What|Removed |Added

 CC||mrzmanw...@gmail.com

--- Comment #7 from Alex Z.  2010-11-02 00:43:13 UTC ---
How would this respond when there's no skin? The only situations I can think of
would be action=parse in the API and action=render via index.php, though parsed
system messages might also be affected. 

I agree with Platonides though, if the main use is styling, we have
skin-specific CSS for this already. And when it comes to the English Wikipedia
"they probably won't use it too much" is almost never a safe assumption.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2010-11-01 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

--- Comment #6 from MC10  2010-11-02 00:32:49 UTC ---
Actually, shouldn't those already be detected by anti-vandalism tools/bots? It
doesn't exactly matter whether or not it's in a #ifeq: or not. If it's not
detected by anti-vandalism tools, it may be a bit harder to remove, but it
shouldn't be too difficult for someone who searches up "" to
remove it.

It may be possible to use MediaWiki:.css, but as a non-admin, it is a bit
more difficult to make changes.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2010-11-01 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

--- Comment #5 from Platonides  2010-11-01 18:57:11 UTC 
---
I am not convinced about the usecase for adding such parserfunction in core.

You want to give a slightly different style to a table (eg. footballer infobox)
depending on the skin. Given how these things expand, this would ultimately
lead to hundred of pages with skin variants.
However, there's another way to get the same effect which doesn't require
splitting the cache: use a class in the infobox, then customize the styles in
MediaWiki:Vector.css, MediaWiki:Monobook.css...

I don't think there will be too much use for message customization either. Take
into account that internal messages can provide the skin as a parameter if
needed, so only wiki instruction would need it.

However, it reduces reviewing. An addition of
{{#ifeq:{{CURRENTSKIN}}|cologneblue |  }} inside a biography
would be hardly noticed unless viewing the wikitext, while still providing the
vandal a way to advertise it using the useskin parameter.

Skin style customization via the skin css page, although harder for non-admins,
is much safer.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2010-11-01 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

Roan Kattouw  changed:

   What|Removed |Added

 CC||platoni...@gmail.com,
   ||roan.katt...@gmail.com

--- Comment #4 from Roan Kattouw  2010-11-01 12:49:57 
UTC ---
(In reply to comment #3)
> As an aside, this feature sounds like something that would have a (possibly
> negative) affect on how caching works, but I don't really know enough about 
> how
> parser cache works to comment on that.
The parser cache code would have to be tweaked a bit such that a different
version if cached for each skin if {{CURRENTSKIN}} is used on the page. I don't
think that's terribly hard to do; Platonides would know, I think, CCing him. As
for performance: I don't think it'd have a large impact as long as
{{CURRENTSKIN}} isn't used on lots and lots of pages.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 25729] Add a {{CURRENTSKIN}} variable to wikitext

2010-11-01 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=25729

Bawolff  changed:

   What|Removed |Added

 CC||bawolff...@gmail.com
Summary|Add a #switchskin   |Add a {{CURRENTSKIN}}
   |ParserFunction  |variable to wikitext

--- Comment #3 from Bawolff  2010-11-01 11:05:06 UTC ---
Since they're essentially asking for the same feature (or to solve the same
usecase), and everyone who's commented seems to like the magic word method, its
probably fine to just rename this. (which I just did)


As an aside, this feature sounds like something that would have a (possibly
negative) affect on how caching works, but I don't really know enough about how
parser cache works to comment on that.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l