Remove .hiddenStructure from CSS
Closed, ResolvedPublic


Author: happy_melon

It's been deprecated for over three years now, yet we're still forced to manually override this to stop people using the damn thing. If people want to continue using a terrifying hack that royally screws over people with older browsers (vide Brion, they can add the definition themselves, it's not exactly difficult. But given how many other friendlier ways there are of hiding content, it's crazy to leave this in there just because it's been there since forever.

@@ 487:

  • .hiddenStructure {
  • display: none;
  • }

Version: unspecified
Severity: minor

bzimport added a subscriber: Unknown Object (MLST).
bzimport set Reference to bz17009.
bzimport created this task.Via LegacyJan 13 2009, 10:58 PM
demon added a comment.Via ConduitJan 15 2009, 8:46 PM

Using it for its original purpose (in meta templates?) was bad. Leaving it as a general CSS class declaration for hiding things shouldn't hurt anyone...?

bzimport added a comment.Via ConduitJan 16 2009, 7:52 PM

Wiki.Melancholie wrote:

Please note that this class is being used on wikis here and there.
Removal will suddenly unhide stuff on wikis out there making use of this.

Lejonel added a comment.Via ConduitOct 10 2009, 8:24 PM

If it is removed it should also be removed in css for other skins (modern, vector, and maybe other).

If it is not removed, maybe it should be moved to a skin independent css file. Currently the class is missing for old skins like Standard and CologneBlue.

bzimport added a comment.Via ConduitOct 27 2009, 2:34 AM

chinchi29 wrote:

If any wiki still wants to use this class, the sysops can put the class back in the local common.css.

Dodoiste added a comment.Via ConduitOct 23 2010, 2:46 PM

(In reply to comment #4)

If any wiki still wants to use this class, the sysops can put the class back in
the local common.css.

Yes. The biggest wikis already wiped out most of the hiddenstructure usages anyway.

The thing is, since the class still exists editors keep using it. And it's a pain to have to check the templates on a regular basis to ensure nobody tried using it again. And really often, we find yet nother buch of templates that we have to rewrite completely.

So please, let's remove it once and for all.

bzimport added a comment.Via ConduitMar 30 2011, 4:44 PM wrote:

Removed in r85017.

He7d3r added a comment.Via ConduitAug 1 2012, 6:38 PM

For the record: it is really difficult to get rid of it. We still have at least 400 pages on Portuguese Wikipedia using the old class...

Add Comment