Mailing-List: contact svn-help@forrest.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Forrest Developers List" <dev@forrest.apache.org>
Received-SPF: pass (asf.osuosl.org: local policy)
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r413041 - /forrest/events/forrest-friday/20060609-log.txt
Date: Fri, 09 Jun 2006 13:20:05 -0000
To: svn@forrest.apache.org
From: crossley@apache.org
Message-Id: <20060609132006.07E951A983A@eris.apache.org>

Author: crossley
Date: Fri Jun  9 06:20:05 2006
New Revision: 413041

URL: http://svn.apache.org/viewvc?rev=413041&view=rev
Log:
Update

Modified:
    forrest/events/forrest-friday/20060609-log.txt

Modified: forrest/events/forrest-friday/20060609-log.txt
URL: http://svn.apache.org/viewvc/forrest/events/forrest-friday/20060609-log.txt?rev=413041&r1=413040&r2=413041&view=diff
==============================================================================
--- forrest/events/forrest-friday/20060609-log.txt (original)
+++ forrest/events/forrest-friday/20060609-log.txt Fri Jun  9 06:20:05 2006
@@ -206,3 +206,52 @@
 [22:54] _Gav_: the project skinconf.xml colors will still be used.
 [22:59] _Gav_: Either way W3C Validator is failing on all stylesheets.
 [22:59] _Gav_: Am I right in assuming skinconf.xml and/or skinconf.xsl becomes profile.css ?
+[23:00] _Gav_: If so then I need to add background-color + other changes into either one of those to fix these issues
+[23:01] _Gav_: But I will in any case need to add the same to screen.css because that is failing too, for the same attributes in some cases
+[23:02] _Gav_: # Line : 168 (Level : 1) You have no color with your background-color : #top .header .current
+[23:02] tscherler: mom
+[23:02] _Gav_: Comes from screen.css
+[23:02] tscherler: just poped in
+[23:02] _Gav_: Line : 39 (Level : 1) You have no color with your background-color : #top .header .current
+[23:02] tscherler: dispatcher is a whole different way
+[23:02] _Gav_: comes from profile.css
+[23:02] tscherler: normally I would say
+[23:02] _Gav_: Ok, ignore that for another time then
+[23:03] tscherler: apply changes in screen.css
+[23:03] _Gav_: That was my original plan
+[23:03] tscherler: profile.css is an extension/override of screen.css
+[23:03] tscherler: you are right that (14:59:44) _Gav_: Am I right in assuming skinconf.xml and/or skinconf.xsl becomes profile.css ?
+[23:04] tscherler: like skinconf.xml will be transformed with /pelt/skinconf.xsl
+[23:04] tscherler: the result is used in profile.css.xsl
+[23:04] tscherler: as incoming document
+[23:04] tscherler: so the problem is that
+[23:04] _Gav_: ah ok, so screen.css print.css take priority of values declared in skinconf.x?l
+[23:05] tscherler: adding background color to screen and profile seems to be redundant
+[23:05] tscherler: no
+[23:05] tscherler: look at the source code of
+[23:05] _Gav_: It does seem redundant, but how do we clear these warningd without them
+[23:05] tscherler: skinconf.xml will be transformed with /pelt/skinconf.xsl
+[23:06] tscherler: <link type="text/css" href="skin/basic.css" rel="stylesheet"> <link media="screen" type="text/css" href="skin/screen.css" rel="stylesheet"> <link media="print" type="text/css" href="skin/print.css" rel="stylesheet"> <link type="text/css" href="skin/profile.css" rel="stylesheet">
+[23:06] tscherler: screen comes before profile
+[23:06] tscherler: meaning profile is extending/overriding the values of screen
+[23:07] tscherler: (15:05:55) _Gav_: It does seem redundant, but how do we clear these warningd without them -> you are right, I have no other way ATM
+[23:07] tscherler: that leaves us with
+[23:08] tscherler: skinconf.xml takes precedence over any skin skinconf.xsl 
+[23:08] tscherler: and
+[23:08] tscherler: profile takes precedence over screen
+[23:08] tscherler: ...by extending/overriding
+[23:09] _Gav_: ok yep, so
+[23:09] tscherler: :)
+[23:09] _Gav_: we should not alter skinconf.xml
+[23:10] _Gav_: :)
+[23:10] _Gav_: should be  .... there
+[23:10] tscherler: jupp
+[23:10] tscherler: like in our site-author
+[23:10] tscherler: I recall an issue with default colors
+[23:11] tscherler: we have definied our forrest colors in the sconf.xml
+[23:12] _Gav_: Ok, this issue Im working on is site-author related.
+[23:13] _Gav_: sorry Im == we
+[23:14] _Gav_: So if I add background-colors to screen.css, keeping the changes pelt skin related and not project specific
+[23:15] _Gav_: as profile.css overrides screen.css, the errors should go away from profile.css if declared in screen.css
+[23:15] _Gav_: maybe
+[23:17] _Gav_: If people are happy the changes should go there I'll add them and try it