[dev] Re: [patch] Cleanup giapeto blocks with titles

Chuck Hagenbuch chuck at horde.org
Sun Mar 6 10:32:47 PST 2005


Quoting Roel Gloudemans <roel at gloudemans.info>:

> When you gice a giapeto block a title it is displayed with this title 
> as a portal block.
>
> This patch cleans up this code a bit, so that it is more easily 
> configurable. It was a table and a div, but this proved to be a pain 
> to create neat blocks. This patch changes it to just a table, which 
> is much more easy to configure in terms of css.

How about just using a wrapping div (giapetoblock), an inner div or 
just <p> (giapetoheader), and another div (giapetocontent) - looks like 
the table isn't needed at all. Also, please don't change the indent on 
the CSS.

> (PS. Check out http://www.gloudemans.info. It sports a new design and 
> is still a 100% horde site)

Very nice! Those headers/tabs just css styles, or are there code 
modifications behind some of that?

-chuck

-- 
"But she goes not abroad in search of monsters to destroy." - John 
Quincy Adams


More information about the dev mailing list