Hello and welcome to our community! Is this your first visit?
Register
Enjoy an ad free experience by logging in. Not a member yet? Register.
Page 1 of 2 12 LastLast
Results 1 to 15 of 24
  1. #1
    Regular Coder
    Join Date
    Sep 2002
    Location
    California
    Posts
    511
    Thanks
    1
    Thanked 1 Time in 1 Post

    First 'full scale' site...opinions?

    Hey

    Ive been meaning to make myself a site here for a long time now, so I went out and bought a domain name and some space over at ithium and this is what I came up with. All of it was coded by hand by me...except for the image rotator..but everything else is mine =). I did this as a way to test my newly found love for php/sql.

    http://www.thecornerofmymind.com

    Just tell me what you guys think, It took me about 5 days of work to do, its php/mysql driven with a admin section in where i can post new blogs/news and see whos been going over my site, I also run all user submitted information thru a few filters to remove all html code, and to escape sql code to prevent injections. Probably my favorite part of the whole site is the user -> admin messenging thing. Its not 100% done YET, you'll notice that the 'tutorials' link doesnt work...thats because im focusing efforts on the site look and function first, all ive got left is to add some custom BB codes to take the place of the removed html capabilities. But tell me what you think, ive showed it to a few friends who all loved it but perhaps a more unbiased opinion is needed.
    When traveling down the road of life, Try not to drag your face in the gravel.

  • #2
    Regular Coder
    Join Date
    Oct 2004
    Location
    England
    Posts
    282
    Thanks
    0
    Thanked 0 Times in 0 Posts
    It's a good start but there is going to be a lot of criticism here i think

    There's a lot of grey, i'd say too much
    You're using times new roman? Need I say more?
    The news doesn't align with the left and right hand columns in firefox
    The header doesn't look right being central like that, and it stands out quite a bit from the rest of the site as it's the only bit with colour
    All the text seems to be the same size throughout the whole site, different sizes make things stand out, it's a good to be a bit more variable


  • #3
    Regular Coder
    Join Date
    Sep 2002
    Location
    California
    Posts
    511
    Thanks
    1
    Thanked 1 Time in 1 Post
    Ya, there is a bit of grey, I thought that it would make it more...clean, the banners provide a nice contrast, i dont find myself looking at them too long, but I do glance up at them occasionally...keeps the eyes movings...makes people less bored I suppose.

    The news doesn't align with the left and right hand columns in firefox
    You mean horizontally or vertically?, its not supposed to align horizontally, but then vertical alignment has been a bit of a problem for some time as it differs from IE to Firefox...so thats kind of a middle point between the differences.
    When traveling down the road of life, Try not to drag your face in the gravel.

  • #4
    Regular Coder
    Join Date
    Sep 2002
    Location
    California
    Posts
    511
    Thanks
    1
    Thanked 1 Time in 1 Post
    I notice a good handful of peope are coming to the site thru this thread...but nobody leaves any commentary?...anyone have anything else to say?
    When traveling down the road of life, Try not to drag your face in the gravel.

  • #5
    Regular Coder
    Join Date
    May 2005
    Location
    Michigan, USA
    Posts
    566
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by Bry Man
    I notice a good handful of peope are coming to the site thru this thread...but nobody leaves any commentary?...anyone have anything else to say?
    Yes it is horrible. Is that a bit to honest? But luckily for you I took the time to make it better. I did not really like you attitude in the last post though so I made it work good enough, and the rest of it you are free to play with. One thing you should really fix is making col1, col3 into Ordered Lists instead of div's like they currently are.

    Tested FF 1.0.6, O 8.02, IE 6.0 SP2 (XP SP2) (Did not validate it)


    *snip* (Removed the code)
    Last edited by SpirtOfGrandeur; 09-08-2005 at 01:53 AM. Reason: Removed the code...
    Note: I do not test code. I just write it off the top of my head. There might be bugs in it! But if any thing I gave you the overall theory of what you need to accomplish. Also there are plenty of other ways to accomplish this same thing. I just gave one example of it. Other ways might be faster and more efficient.

  • #6
    Regular Coder
    Join Date
    Sep 2002
    Location
    California
    Posts
    511
    Thanks
    1
    Thanked 1 Time in 1 Post
    I was simply asking a question, if anyones packing an attitude its you. you can take your self righteous high n mighty attitude and code and shove it if thats how your gonna act when someone asks you a question. I just want commentary, WHY is it horrible....provide more than just...it sucks please, WHY does it suck...
    When traveling down the road of life, Try not to drag your face in the gravel.

  • #7
    Regular Coder
    Join Date
    May 2005
    Location
    lincolnshire
    Posts
    155
    Thanks
    0
    Thanked 0 Times in 0 Posts

    great code! but

    i think the site is lacking in character, and sadly looks like a site made 5 years ago!, im really sorry but im being honest!

    I can see ur php skills outweigh the creativity on this site!, it needs a little color, or soem nice graphics!

    sorry if this is so negative, its not a bad site just rather it needs a little character!, if u wanted a template you could download one, add some spunk!

    I just think it looks like a tabled site!, sorry this is crap response!
    littletoaster - web design dwsystems -pc repair

    "If the diver thought of the shark, he would never lay hands on the pearl"

  • #8
    Regular Coder
    Join Date
    May 2005
    Location
    Michigan, USA
    Posts
    566
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by Bry Man
    I was simply asking a question, if anyones packing an attitude its you. you can take your self righteous high n mighty attitude and code and shove it if thats how your gonna act when someone asks you a question. I just want commentary, WHY is it horrible....provide more than just...it sucks please, WHY does it suck...
    Did you even try the code? I was going to make it a link, but could not get any server space to show it off.

    Number one you have broken HTML. You have ending </a> tags all over that do not need to be there. Using a fixed PX font is not the way websites are supposed to be created. I have large fonts installed on my machine therefore 12px looks horrible. Maybe 12 pt? (Was Fixed) How about the horrible JS that is 110% not needed? (Was Fixed) How about the difference between vertical heights between the side cols and the middle col? (Was Fixed) How about the use of '&nbsp;' instead of padding? (Was Fixed) And I definatly like the use of '<br />' where there is no need to be breaking. (Was Fixed) The font is hard on the eyes. (Was Fixed) You had about 8% redundant CSS. (Was Fixed). '<center>' and '<font>' tags are deperciated and should not be used. (Was Fixed)

    I think that is all that I am going to post... o wait, yea you should change col1 and col3 to be lists. And the color scheme could use some updating. And the layout is nothing special. It is hard to read on my 61' HDTV even with FF scrolling the text to the largest size. (Due to color and font) Instead of writing all that up, I went ahead and coded it for you. Was that so wrong? Your right maybe it was.

    For your sake I hope you saved it. So that you could learn something...
    Last edited by SpirtOfGrandeur; 09-08-2005 at 01:55 AM.
    Note: I do not test code. I just write it off the top of my head. There might be bugs in it! But if any thing I gave you the overall theory of what you need to accomplish. Also there are plenty of other ways to accomplish this same thing. I just gave one example of it. Other ways might be faster and more efficient.

  • #9
    Regular Coder
    Join Date
    Sep 2002
    Location
    California
    Posts
    511
    Thanks
    1
    Thanked 1 Time in 1 Post
    @SpirtOfGrandeur

    Actually I didnt save it...why...becuse its worthless to me, to put it nicely your quite full of yourself, Did you not pay attention to the fact taht the difference in height was intentional? Its there to provide a break in the flow...we dont want a hard block to look at do you, maybe the sites you make are like that but not mine. What usless JS? do you have any idea what that does...if you did you would know its not usless. Why use padding, &nbsp is there to take the place of content whencontent is not available...that style is used for content as well, having padding with content would be god awful on that bar. Ending </a> tags are the for a reason THEY END <a> Tags! lol.....nothing you did was of ANY use to me, sorry but you dont seem to know what your talking about.

    @little_toaster

    Thank you for your commentary, I know its a bit drab with the grayscale scheme, I was aiming more on getting the site operationaly and then I was gonna make a new scheme for it, I kinda liked the gray for a little bit cuz it wasnt supposed to be a birght and happy site...its a me thing i guess, i like to try and make the site look good using only code, ive never been much of a graphics guy even though ive taken several graphical design classes I prefer the beauty of code over pixel for some reason lol...at least you have the decency to not be all self righteous and not provide reason for your actions, as I said...this is my first 'full scale' site writtin in pretty much 100% php with a mysql backend, I think the line count is over 4000 lines...i guess your right...my php skills are better than my design lol...thanks again for your comment.
    When traveling down the road of life, Try not to drag your face in the gravel.

  • #10
    Regular Coder
    Join Date
    May 2005
    Location
    Michigan, USA
    Posts
    566
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by Bry Man
    Did you not pay attention to the fact taht the difference in height was intentional? Its there to provide a break in the flow...we dont want a hard block to look at do you, maybe the sites you make are like that but not mine.
    If that is what you intend to do then fine. Because there is such a big space between the four parts of your site it is confusing to the normal user. If you want to try something a bit different why dont you make col1 and col3 be lower... then again this is just some peoples taste. If you like it keep it.

    Quote Originally Posted by Bry Man
    What usless JS? do you have any idea what that does...if you did you would know its not usless.
    Actually I do. It changes the color of the &raquo; to a light blue. There is no need for JS to do this because you can do it in CSS in 1/2 the code. Or actually in about 1/4 the code. And people with JS turned off will not see the color where people with CSS turned on will.


    Quote Originally Posted by Bry Man
    Why use padding, &nbsp is there to take the place of content whencontent is not available...that style is used for content as well, having padding with content would be god awful on that bar.
    Actually having padding with content on that bar looked 100 times more managable using CSS. There is never a need to start and/or end a string with a &nbsp;. You can use padding for that. And believe it or not it reads better in the code.

    Quote Originally Posted by Bry Man
    Ending </a> tags are the for a reason THEY END <a> Tags! lol.....nothing you did was of ANY use to me, sorry but you dont seem to know what your talking about.
    Just took this off your site... Where are the starting <a> tags? It appears as though you meant to put a link in the bottom (maybe an admin edit link) and did not remove the </a>.

    Code:
    <div class="EntryInfo">Submitted On: 09.06.05</a></div><br />
    
    <div class="EntryTitle">New Blog Entry!</div>
    <div class="EntryContent">
    <br /><center>New Blog entry added, click <a href="./blog.php?action=accepted">Here</a> to be taken to the Blog.</center><br />
    </div>
    <div class="EntryInfo">Submitted On: 09.05.05</a></div><br />
    <div class="EntryTitle">A New Month....two days late</div>
    <div class="EntryContent">
    Hey, New month means time to clear the crap for the last month...except the blogs, which ive gotta devise a way to store old entrys according to month...or leave em as they are....hmm. Ive still gotta write a stupid pagination script too...so much to do. Then ill do a sweep of the code and make it more uniform, and adjust it so that the site has a bit more detail.
    </div>
    Note: I do not test code. I just write it off the top of my head. There might be bugs in it! But if any thing I gave you the overall theory of what you need to accomplish. Also there are plenty of other ways to accomplish this same thing. I just gave one example of it. Other ways might be faster and more efficient.

  • #11
    Regular Coder
    Join Date
    Sep 2002
    Location
    California
    Posts
    511
    Thanks
    1
    Thanked 1 Time in 1 Post
    heh you were right on one thing
    <div class="EntryInfo">Submitted On: 09.06.05</a></div><br />

    <div class="EntryTitle">New Blog Entry!</div>
    <div class="EntryContent">
    <br /><center>New Blog entry added, click <a href="./blog.php?action=accepted">Here</a> to be taken to the Blog.</center><br />
    </div>
    <div class="EntryInfo">Submitted On: 09.05.05</a></div><br />
    <div class="EntryTitle">A New Month....two days late</div>
    <div class="EntryContent">
    Hey, New month means time to clear the crap for the last month...except the blogs, which ive gotta devise a way to store old entrys according to month...or leave em as they are....hmm. Ive still gotta write a stupid pagination script too...so much to do. Then ill do a sweep of the code and make it more uniform, and adjust it so that the site has a bit more detail.
    </div>
    those shouldnt be there....i dont remember ever even putting them in there.....
    When traveling down the road of life, Try not to drag your face in the gravel.

  • #12
    Registered User
    Join Date
    Jul 2005
    Posts
    153
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by SpirtOfGrandeur
    Did you even try the code? I was going to make it a link, but could not get any server space to show it off.

    Number one you have broken HTML. You have ending </a> tags all over that do not need to be there. Using a fixed PX font is not the way websites are supposed to be created. I have large fonts installed on my machine therefore 12px looks horrible. Maybe 12 pt? (Was Fixed) How about the horrible JS that is 110% not needed? (Was Fixed) How about the difference between vertical heights between the side cols and the middle col? (Was Fixed) How about the use of '&nbsp;' instead of padding? (Was Fixed) And I definatly like the use of '<br />' where there is no need to be breaking. (Was Fixed) The font is hard on the eyes. (Was Fixed) You had about 8% redundant CSS. (Was Fixed). '<center>' and '<font>' tags are deperciated and should not be used. (Was Fixed)

    I think that is all that I am going to post... o wait, yea you should change col1 and col3 to be lists. And the color scheme could use some updating. And the layout is nothing special. It is hard to read on my 61' HDTV even with FF scrolling the text to the largest size. (Due to color and font) Instead of writing all that up, I went ahead and coded it for you. Was that so wrong? Your right maybe it was.

    For your sake I hope you saved it. So that you could learn something...

    61" HDTV whats that got to do with anything?

    and am sure you could becuase i have a 42" HDTV plasma with VGA connection and it was only 2000 ($4500 around) and thats fine

    i also tried on my 19" moniter and 15" laptop moniter and it's ok to read did not test on psp will do tho?

    o and you must of paid 4500 ($9000 around) for that or more when i went looking for HD that size they had none at all and that was about a week ago in currys (uk)when i got mine.

    lets stick to topic tho. and not try to see if someone is lieing about seeing text.
    Last edited by Lyconpire; 09-22-2005 at 04:42 PM.

  • #13
    Senior Coder
    Join Date
    Dec 2004
    Location
    Essex, UK
    Posts
    2,636
    Thanks
    0
    Thanked 0 Times in 0 Posts
    I think it would look alot better if you made it fixed with a border round a container to hold it all in. Perhaps a white background on the container would look good also and break it up a bit.

    Bring the comments section up so its aligned with the menu and site stats.

    Add more padding so the text does not hug so close to the sides.

  • #14
    Registered User
    Join Date
    Jul 2005
    Posts
    153
    Thanks
    0
    Thanked 0 Times in 0 Posts
    Quote Originally Posted by mark87
    I think it would look alot better if you made it fixed with a border round a container to hold it all in. Perhaps a white background on the container would look good also and break it up a bit.

    Bring the comments section up so its aligned with the menu and site stats.

    Add more padding so the text does not hug so close to the sides.
    good review white background would look good

  • #15
    Regular Coder
    Join Date
    Sep 2002
    Location
    California
    Posts
    511
    Thanks
    1
    Thanked 1 Time in 1 Post
    Hey,

    Didnt think people were still putting comments up, ive made quite a few changes since the last batch, including a new style that I copied from DeviantArt.com, ive added BBcode (so far only to the admin section) and ive fixed a few errors. You guys really think that the news blocks shouold be moved up to be aligned with the menu and stats blocks huh?, I thought it was nice to have a break. Ya ive been wanting to add more padding too, for some reason when I add padding it makes the container wider so I have to compensate by making the actual width smaller, is that supposed to happen =S.

    Thanks for the reviews, I would like to apologize as I kinda came off as an *** at Spirit, but then again I took it rather offensively and that he bsaicly said 'your site sucks, here I fixed it for you so its better, use this code instead'
    When traveling down the road of life, Try not to drag your face in the gravel.


  •  
    Page 1 of 2 12 LastLast

    Posting Permissions

    • You may not post new threads
    • You may not post replies
    • You may not post attachments
    • You may not edit your posts
    •