![]() |
#1
|
||||
|
||||
![]()
Hi. I'm looking for critique on my latest project, Elegance. It's basically a showcase for users where they can post poetry [for now], and eventually photography, writings, and other art. Users are also given a blog, which they may use if they wish. If you've got time, register and tell me what you think! ;) Thanks.
http://elegance.chanlu.org
__________________
|
#2
|
||||
|
||||
![]()
http://elegance.chanlu.org/index.php...ofile&user=%27
Neat SQL injection. I was also allowed to register with every field blank. Tips: - Don't force me to register just to view a profile. - Don't demand I tell you my e-mail, country and date-of-birth just to register. - Nobody intentionally uses "reset" inputs. - Stop using addslashes(). - Rewrite your other URLs. "index.php?id=viewblog&userid=1" is very ugly. Last edited by Creasy; 10-23-2009 at 12:15 AM. |
#3
|
||||
|
||||
![]()
Purple and green do not go together well...
The purple bar on top is annoying, also. I know there's an option to hide it, but it comes up again once I go to another page. I don't really see the use of the bar, since you can login/register by clicking the links on the sidebar. Another thing: why do you need people's emails, birthdays, and countries to register? There really is not point, if the sole purpose of the site is to share poetry and to comment on it. I also don't see why guests can't view profiles and comments. Since your site is small, you want people to look at your site instead of hiding everything from them.
__________________
![]() ||||||||||||||
|
#4
|
||||
|
||||
![]()
The registration thing was a misconception on my part.
Why should I not ask for your email? With an email I can at least try to prevent users from having duplicate accounts. Forcing an unregistered user to register to view a profile was a mistake in my code. I've gotten other requests to remove birthday and date of birth from the registration field. I probably will. Reset will be removed when I take out the other fields... it won't be present in v2. I don't use addslashes() URL rewriting will also be implemented in v2. EDIT @linoone: Sorry, obviously you posted while I was typing this ;) I know that the layout is bad. It's from about 6 months ago. The new one is in progress. I've also answered your other questions above.
__________________
|
#5
|
||||
|
||||
![]()
I joined - it's a neat looking site. I also posted some poetry, but I have one minor complaint - if you input the title, then copy and paste the poetry from say Microsoft Word, the title disappears, and you have to re-enter it. And also, it would be nice if you could comment on the same page as the poem, so that you can read it and quote it while critiquing.
Otherwise, it's pretty nice - well done :) |
![]() |
Thread Tools | |
|
|