steelmanronald06 Posted October 14, 2006 Share Posted October 14, 2006 Sorry for the cheesy title. That is what I am calling the new redesign on my site. Okay, here are the changes:1) New design. Very crisp, IMO. White, grey, and teal but I like it, and alot of members liked it.2) All of the source code has been re-done, using Switch Cases and other stuff.3) New features. Now I know that most all of you won't want to register to see some of these...but it is mainly just admin/mod/author functions, and a private message system. Nothing big there.So just tell me what you think :)http://lampgeekz.netgeekz.net/ Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/ Share on other sites More sharing options...
steviewdr Posted October 14, 2006 Share Posted October 14, 2006 Looks ok and well put togethor.The left menu is way too long for my liking. I would have a collapsable menu or a page specific larger menu.I dont know if you are trying to keep pages sizes small or easy to update - but the site is a bit shy on graphics.Passwords should be in the password format. I dont want anyone beside me seeing me type the password for registering.Oh! -([code]You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'reilly', 'sburke@burkesys.com', 'steve', 'e10adc3949ba59abbe56e057f20f883e', '1'' at line 2[/code]Apparently I was wrong about it been put well togethor!!The details I used for registering were:[quote]steveo'reillysburke@burkesysetc.etc.[/quote]Awwwh - more problems :-/Once I change my surname, registered and logged in, and tried to send a message to myself, using these details:[quote]To: steveSubkject: "asdasd'asdasdMessage: asdasdasd<h1>asdasd</h1>asdasdasd[/quote]I got:[code]You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'asdasd','asdasdasd <h1>asdasd</h1>asdasdasd')' at line 2[/code]I'll let you fix these bugs.-steve Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/#findComment-108761 Share on other sites More sharing options...
steelmanronald06 Posted October 14, 2006 Author Share Posted October 14, 2006 So you got an error on registration and in the subject on the PM system because of the " and the ' characters? okay. I had tested the message part and stuff, and knew that that worked, and html characters is stripped out on the PM system (BTW), so I will fix those. Thanks for testing that :) Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/#findComment-108796 Share on other sites More sharing options...
steelmanronald06 Posted October 16, 2006 Author Share Posted October 16, 2006 la bumpalso on those errors. Still in the process of fixing them. Everything is in Case which makes the problem area easy to find, but within that is OOP which I am not so hot at :) Took forever to get working right, so now it is taking forever to fix la problem ::) Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/#findComment-109647 Share on other sites More sharing options...
Daniel0 Posted October 18, 2006 Share Posted October 18, 2006 Looks very nice, but something is wrong in the menu (perhaps a HTML comment).[attachment deleted by admin] Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/#findComment-110546 Share on other sites More sharing options...
steelmanronald06 Posted October 23, 2006 Author Share Posted October 23, 2006 hrm. I don't see that. anyone else see it :( Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/#findComment-112951 Share on other sites More sharing options...
freakus_maximus Posted October 23, 2006 Share Posted October 23, 2006 Yeah it's there right at the bottom of the "Featured Lessons" on the left is a "-->". Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/#findComment-112968 Share on other sites More sharing options...
448191 Posted October 23, 2006 Share Posted October 23, 2006 The layout can certainly be called an improvement. Congrats.I don't guite agree with the colour scheme though... It's a bit monotone, and the green has a bit boring tone... You could do with a logo, too.Some other, more minor issues:- The size of the login input fields is different in FF and IE. Neither look very good, but if you take the way it looks in FF, and align the fields/labels, it would.- Top navigation is a bit off.- The search input is not vertically aligned. Also the size is not the same in IE and FF. Looks better in FF.- Validation: 3 fatal errors, 13 warnings.- What Daniel0 and freakus_maximus note is visible only in IE.Definite improvement, none the less. Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/#findComment-112999 Share on other sites More sharing options...
steelmanronald06 Posted October 27, 2006 Author Share Posted October 27, 2006 Oh! hmmm..i will look in ie for that one problem. I will do a validation again. :( i hate those things! i will look a the navs agian, might be an IE problem. and i will get the login fields fixed. thanks :) Quote Link to comment https://forums.phpfreaks.com/topic/23912-lampgeekz-source/#findComment-115213 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.