[geeklog-devel] Config page validation errors

[=MF=Geiss]

Tom,

Thanks for the feedback. I appreciate you taking the time to respond. :-)

I will look at trying to bring out the color in the th and even rows. I 
want to try to keep it somewhat neutral (so it doesn't overpower the 
content), but I agree, it can blend in a little more.

I've already created a new topic_gl icon, and can do the same in the 
future for other icons. At the moment though, as they reside outside the 
layout/nouveau folder structure, I'm taking a hands off approach. :-)

As far as the forum css and stuff goes, it shouldn't be too hard to 
re-skin. Everything is pretty neutral to begin with, and those blue 
colors that need adjusting are pretty easy to identify. If Blaine wants 
to include it, I'm sure I can do it fairly quickly.

Thx!

Eric

Web Site Master wrote:

> I think everyone is busy ;-)

>

> I really like the new theme, the only thing I am not sure on is the color

> choice for the background of the titles on pages like Site Statistics.  It

> seems a little off when comparing it to the blue of the logo (to much grey

> in it??)

>

> I think it will be a good choice for the default theme for geeklog.net (this

> was mentioned in a previous post). It feels more modern to me than the

> professional theme. The only issues I see there is that the Forum css would

> have to be updated to match a bit better and a new icon set for the topics

> and maybe the forum as well would help.

>

> Tom

>

>

>

>

> -----Original Message-----

> From: geeklog-devel-bounces at lists.geeklog.net

> [mailto:geeklog-devel-bounces at lists.geeklog.net] On Behalf Of =MF=Geiss

> Sent: December-18-07 10:01 PM

> To: Geeklog Development

> Subject: [geeklog-devel] Config page validation errors

>

> Hi all,

>

> While validating my theme I came across the following issue (among 

> others)...

>

> In system/classes/config.class.php on line 437 the onClick needs to be 

> changed to all lowercase like this:

> "(<a href='#' onclick='unset(\"{$name}\");'>X</a>)");

>

> Can someone with CVS commit rights please make that change? Thanks in 

> advance! :-)

>

> Also, there are a handful of errors due to using an "id" more than once. 

> This should be switched to "class" since the selector is used repeatedly 

> in the markup. Of course the corresponding style.css "#" will need to be 

> changed to ".", or at least duplicated as such, if it is used elsewhere 

> appropriately as an "id".

>

> There are also some changes that needed to be made in the 

> configuration.thtml and config_element.thtml files to be compliant. Col 

> end tags, etc.

>

> I haven't received any feedback on the nouveau theme from anyone. I'm 

> not sure if it is because everyone is busy, or because people have 

> stopped caring. At any rate, it is available in an SVN repository at 

> http://geiss.getmyip.com:81/svn/nouveau/trunk. Read only login/password 

> is everyone/everyone.

>

> Thx!

>

> Eric

>

> _______________________________________________

> geeklog-devel mailing list

> geeklog-devel at lists.geeklog.net

> http://eight.pairlist.net/mailman/listinfo/geeklog-devel

>

> __________ NOD32 2732 (20071219) Information __________

>

> This message was checked by NOD32 antivirus system.

> http://www.eset.com

>

>

> _______________________________________________

> geeklog-devel mailing list

> geeklog-devel at lists.geeklog.net

> http://eight.pairlist.net/mailman/listinfo/geeklog-devel

>

>

>