You are viewing a plain text version of this content. The canonical link for it is here.
Posted to svn@forrest.apache.org by cr...@apache.org on 2006/06/10 04:30:07 UTC

svn commit: r413210 - /forrest/events/forrest-friday/20060609-log.txt

Author: crossley
Date: Fri Jun  9 19:30:06 2006
New Revision: 413210

URL: http://svn.apache.org/viewvc?rev=413210&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=413210&r1=413209&r2=413210&view=diff
==============================================================================
--- forrest/events/forrest-friday/20060609-log.txt (original)
+++ forrest/events/forrest-friday/20060609-log.txt Fri Jun  9 19:30:06 2006
@@ -577,3 +577,33 @@
 [11:25] _Gav_: If not that skinconfig.xsl will give it a new default color
 [11:26] _Gav_: and profile.css.xslt takes the colors specified in skinconfig.xsl to create profile.css
 [11:27] _Gav_: skinconf == skinconfig
+[11:33] xley: yes that sounds like the process.
+[11:33] xley: except ...
+[11:34] xley: there are two skinconf.xsl ... webapp/skins/common/skinconf.xsl and for each skin there can be a webapp/skins/*/skinconf.xsl 
+[11:35] xley: i don't quite understand what you need to achieve.
+[11:36] _Gav_: ok, W3C is giving Warnings about not specifying background-color where color is specified and vice-versa
+[11:37] _Gav_: So atm these non-specified background-colors and colors do not exist
+[11:37] _Gav_: So _I_ thought I would need to add them to pelt/skinconf.xsl as site-author is using the pelt skin
+[11:38] _Gav_: Then I would need to add the @value lines also into pelt/profile.css.xslt
+[11:38] _Gav_: to retrieve and add those colors to profile.css
+[11:39] _Gav_: back shortly
+[11:46] xley: why not add them to webapp/skins/common/skinconf.xsl then they are available for all skins.
+[11:56] xley: i see that we would need to add default background-color to the various static *.css too.
+[12:10] _Gav_: yes that is right.
+[12:10] _Gav_: I thought background-colors etc would better be skin related.
+[12:11] _Gav_: but I can put them in common
+[12:12] xley: Ah, okay i think that i see what you mean now - fit in with the default colours of each skin?
+[12:12] _Gav_: yes, thats it.
+[12:12] xley: wait, looking ...
+[12:13] _Gav_: These warnings are a fallback really, so the colors to choose would be the color of the parent container
+[12:14] _Gav_: And unfotunalty 'transparent' can not be used.
+[12:18] _Gav_: I have done this on my minitutorials site. Check out the CSS
+[12:18] _Gav_: http://minitutorials.com/styles/3c​ol_leftNav.css
+[12:18] xley: Hmm, i still think that common/skinconf.xsl might be correct place.
+[12:19] _Gav_: ok
+[12:21] xley: Gav: please see my comments at FOR-891. It is not a Blocker anymore. So i am going to work on other things.
+[12:23] _Gav_: sure, no problem ,I'll carry on with it. Thanks for clarifying.
+[12:24] xley: you are doing the work, so leave it up to you. Remember that there are other people out there with their own skins. They should be basing their stuff on webapp/skins/common
+[12:25] pbol: could you just add *{color: #fff;background-color: #000}
+[12:25] pbol: ...or whatever!
+[12:25] xley: As i suggested. Don't feel that you need to finish this job. We can suffer Warnings for CSS.