forrest-svn mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject svn commit: r413210 - /forrest/events/forrest-friday/20060609-log.txt
Date Sat, 10 Jun 2006 02:30:07 GMT
Author: crossley
Date: Fri Jun  9 19:30:06 2006
New Revision: 413210



Modified: forrest/events/forrest-friday/20060609-log.txt
--- 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
 [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_:​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.

View raw message