Le 03/10/2013 16:37, [email protected] a écrit :
> Le 3 octobre 2013 à 15:20, Marc-Etienne Vargenau <[email protected]> a écrit :> > > Le 02/10/2013 23:26, Franck Villaume a écrit : > > Le 01/10/2013 13:26, Marc-Etienne VARGENAU a écrit : > >> Author: vargenau > >> Date: 2013-10-01 13:26:20 +0200 (Tue, 01 Oct 2013) > >> New Revision: 18089 > >> > >> Modified: > >> trunk/src/www/project/admin/tools.php > >> Log: > >> Use <h2> > >> > >> Modified: trunk/src/www/project/admin/tools.php > >> =================================================================== > >> --- trunk/src/www/project/admin/tools.php 2013-10-01 11:24:11 UTC > >> (rev 18088) > >> +++ trunk/src/www/project/admin/tools.php 2013-10-01 11:26:20 UTC > >> (rev 18089) > >> @@ -112,7 +112,8 @@ > >> echo '<tr class="top">'; > >> echo '<td class="halfwidth">'; > >> -echo $HTML->boxTop(_('Active Tools').''); > >> +echo '<div class="info-box">' . "\n"; > >> +echo '<h2>' . _('Active Tools') . '</h2>'; > >> ?> > > > > Hi Marc-Etienne, > > > > this one is probably a good idea but it's not. > > HTML->boxTop is themable. > > Here your modification drop the theme support. > > > > Can you revert or I will ? > > Hello Frank, > > I added the <h2> with my accessibility hat on. > > You can style the info-box with CSS, I sent an > example in the Funky theme. > > Best regards, > > Marc-Etienne > > -- And ??? You drop theme support. I understand the h2 approach. As I said before, it was a good idea.You should probably develop a accessibility theme and not drop theme support. I've seen your proposition using css.... Once again, the funky css is only valid with specific theme.Please, I beg you. Find another solution.
Commit reverted. Regards, Franck -- TrivialDev Founder http://trivialdev.com
<<attachment: franck_villaume.vcf>>
_______________________________________________ Fusionforge-general mailing list [email protected] http://lists.fusionforge.org/cgi-bin/mailman/listinfo/fusionforge-general
