Hi Arik, On Sun, 2005-04-10 at 19:55 +0200, Arik Kfir wrote: > I've noticed that the skinconf.xml contains all the skin colors. > Generally, that is the place they belong, but I see some problems with > this:
<snip something we agree on/> > What I would suggest is to introduce an additional (optional) file > called "skin-colors.xml" which will contain the site's color > definitions. It should be optional of course. > Yeah that would make sense. > What's more, with this configuration, we will be able to provide > several files - each containing the "skin-colors.xml" file to use (the > forrest.properties can have a property that points to it, with a > sensible default) so I can have several color configurations and > simply "pick out" the one I want by changing a property in > forrest.properties. > > What do you think? > I reckon I would not store it in the forrest.properties because the idea is to be able to change this color settings with a cforms tool (or something similar) in webapp mode. That brings us a step forward in the direction of a skinbot. ;-) I reckon it should be stored in the view (the link). Generally speaking I am +1 to split skinconf.xml apart. Since this changes will be post 0.7, anyway, maybe we should generally discuss what belongs in skinconf.xml and what in e.g. contractconf.xml salu2 > > Thorsten Scherler wrote: > > Gidday, > > > > here is a proposal for a new skinconf format. > > http://svn.apache.org/viewcvs/forrest/trunk/plugins/org.apache.forrest.plugin.viewHelper/src/documentation/skinconf.proposal.xml?view=auto&rev=160783 > > > > This is based on the recent user discussion (skinconf text elements in > > group.svg) which showed again that our skinconf needs to be more > > extensible without touching the dtd. > > > > e.g. > > <forrest:property contract="copyright"> > > <!-- The following are used to construct a copyright statement --> > > <year>2005</year> > > <vendor>The Apache Software Foundation.</vendor> > > <copyright-link>http://www.apache.org/licenses/</copyright-link> > > </forrest:property> > > > > The idea is to allow all xml within a forrest:property. > > > > WDYT? > > > > salu2 > > -- thorsten "Together we stand, divided we fall!" Hey you (Pink Floyd)