On 03/04/2013 10:02 AM, Ecaterina Moraru (Valica) wrote: > Hi devs, > > Our documentation > http://platform.xwiki.org/xwiki/bin/view/DevGuide/VerticalForms is > requesting to use .buttonwrapper class as container for button > controls. > When I added this in the standard was because we were also supporting > Toucan Skin and in Toucan this was needed in order to have images on > buttons (it was an older skin and we needed to support IE6 and IE7). > Since we don't support Toucan anymore and now the CSS has advanced > that you can insert additional style by using the ':after' or > ':before' pseudo-classes I don't think there is no reason to keep an > additional container. > > I send this mail because Marius created a very sweet Button Group > Widget in 4.5.1 > http://www.xwiki.org/xwiki/bin/view/ReleaseNotes/ReleaseNotesXWiki451#HButtonGroupWidget > and he is using the .buttonwrapper container. I don't think we should > promote an additional container anymore. > > Right now Colibri's CSS demand the 'buttonwrapper' in the style it has > both for inputs or for links. This could be changed for Colibri by > rewriting a bit those rules or by adding new ones (more specific but > would preserve backward compatibility). > > I don't know if these changes should be kept for a new skin or added > now, so let me know what you think. >
+1. I was also thinking recently that the .buttonwrapper class doesn't make sense anymore, but I was hesitant in proposing this change. +1 for not adding the .buttonwrapper from now on. +1 for cleaning Colibri selectors/rules. -0 for cleaning up the existing instances of .buttonwrapper, since Toucan is still "recent" enough so we shouldn't completely break support for it in 5.x. Let's at least wait until we get a new skin to have an alternative to Colibri. -- Sergiu Dumitriu http://purl.org/net/sergiu _______________________________________________ devs mailing list [email protected] http://lists.xwiki.org/mailman/listinfo/devs
