[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
>
>
>