From: Jamie C. <jca...@we...> - 2003-05-29 02:48:55
|
On Thu, 2003-05-29 at 06:43, Sandro Dentella wrote: > Playing around with look & themes I needed to add a class=header in the > table of the 'header' function, could it be possible to have it in the main > release too? it will not impact on anything adding a bit of new possibilities. > > > if (defined(&theme_prebody)) { > &theme_prebody(@_); > } > if (@_ > 1) { > print "<table class=header width=100%><tr>\n"; > ^^^^^^^^^^^ > if ($gconfig{'sysinfo'} == 2 && $remote_user) { > > > For the curious ones, the result, using gehroghal theme configurator is here: > http://www.e-den.it/files/webmin.png That is quite a good idea, as it would allow the creation of themes that effect the standard header just by defining their own pre- and post- header HTML, rather than overriding the whole Perl function. I'll put this class in, along with any others that you can suggest for the header and footer. - Jamie |