Return-Path: Delivered-To: apmail-forrest-dev-archive@www.apache.org Received: (qmail 71858 invoked from network); 21 Mar 2007 21:39:42 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 21 Mar 2007 21:39:42 -0000 Received: (qmail 62987 invoked by uid 500); 21 Mar 2007 21:33:00 -0000 Delivered-To: apmail-forrest-dev-archive@forrest.apache.org Received: (qmail 62057 invoked by uid 500); 21 Mar 2007 21:32:57 -0000 Mailing-List: contact dev-help@forrest.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: dev@forrest.apache.org List-Id: Delivered-To: mailing list dev@forrest.apache.org Received: (qmail 60548 invoked by uid 99); 21 Mar 2007 21:31:56 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Mar 2007 14:31:56 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received-SPF: neutral (herse.apache.org: local policy) Received: from [81.169.146.190] (HELO mo-p07-ob.rzone.de) (81.169.146.190) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 21 Mar 2007 14:31:47 -0700 Received: from [192.168.1.33] (51.Red-88-17-7.dynamicIP.rima-tde.net [88.17.7.51]) by post.webmailer.de (mrclete mo43) (RZmta 5.2) with ESMTP id D03b80j2LLMq0f for ; Wed, 21 Mar 2007 22:31:25 +0100 (MET) Subject: Re: svn commit: r520650 - in /forrest/trunk/whiteboard/plugins: org.apache.forrest.plugin.input.doap/ org.apache.forrest.plugin.internal.dispatcher/ org.apache.forrest.plugin.output.solr/ org.apache.forrest.themes.core/themes/common/fo/ org.apache.forrest.... From: Thorsten Scherler To: dev@forrest.apache.org In-Reply-To: <4601120E.4030404@pcotech.fr> References: <20070320231748.EEDA71A9846@eris.apache.org> <4600F480.5060906@pcotech.fr> <4600FB5A.3070802@apache.org> <4600FFAE.7090800@pcotech.fr> <1174471790.6521.26.camel@cartman> <4601120E.4030404@pcotech.fr> Content-Type: text/plain; charset=utf-8 Date: Wed, 21 Mar 2007 22:31:24 +0100 Message-Id: <1174512684.6143.14.camel@cartman> Mime-Version: 1.0 X-Mailer: Evolution 2.8.1 Content-Transfer-Encoding: 8bit X-RZG-AUTH: z4gQVF2k7GZD04reRGeuHfQ= X-RZG-CLASS-ID: mo07 X-Virus-Checked: Checked by ClamAV on apache.org On Wed, 2007-03-21 at 12:07 +0100, Cyriaque Dupoirieux wrote: > le 21/03/2007 11:09 Thorsten Scherler a écrit : > > Hmm, since we decided to change the name of tiles to panels (see the > > thread on dev) I did expect that this commit would have addressed this > > issue as well. > > > > Let me explain why the commit is not really how we want it. > > > > On Tue, 2007-03-20 at 15:50 +0000, Ross Gardler wrote: > > > >> Why is common-fo.vt.xml in html directory rather than the fo directory? > >> > > > > This should go into the "panels" directory of the theme directory since > > it is an "old" tiles. > > > > Panels do not care which contracts they contain. Meaning that for > > internal processing in the dispatcher pelt-css.vt.xml will not be > > treated different then common-fo.vt.xml. > > > > The fo/css in the naming only shows which aggregation of contract and > > hook (in short panel) are addressed. > > > > Panels do not belong in the html nor in the fo nor in css nor in .... > > directory but in a directory on its on like e.g. > > themes/pelt/panels > > themes/common/panels > > > Humm, I agree with the definition of panels and with the fact that vt > should be changed. > I don't see the benefit to store them in a specific directory, because I > have no example where a panel can be used for two different formats. One can think of one, but you have a point that panels are likely to be format dependent in their usage (not by definition). > Why don't we name them with the .panel extension and store them in the > format directory ? Imagine you have a bigger side and have heaps of panels, don't you think that keeping them in the root folder of a theme (e.g. theme/common) can become pretty messy? > If we look at the css common directory we have a mix of different kind > of files : > theme/common/css > |-- basic.css (a real source file to be used > as it is) > |-- branding-generic-css.ft (a contract...) > |-- ... > |-- profiling.css.xslt (a stylesheet - which I'm not > sure is very useful...) > |-- ... > > It doesn't strike me to have panels in there : > |-- common.panel > That mean we would introduce another extension that we need to add to the svn configs. I would rather use *.panel.xml. wdyt? salu2 -- Thorsten Scherler thorsten.at.apache.org Open Source Java & XML consulting, training and solutions