SITE REVIEW: Sorcerer's Awards

  • sirussblack
  • Newbie
  • Newbie
  • sirussblack
  • Posts: 6

Post 3+ Months Ago

I made a site a while ago and then closed it and I made it again. :) I'd love some feedback on it whether positive or negative. You can find it at http://sa.iwillspeak.us. :) Thanks in advance.
  • Anonymous
  • Bot
  • No Avatar
  • Posts: ?
  • Loc: Ozzuland
  • Status: Online

Post 3+ Months Ago

  • Rat
  • Guru
  • Guru
  • User avatar
  • Posts: 1190
  • Loc: desk

Post 3+ Months Ago

The layout isnt bad. Simple and easy. The /may/ be a little hard on some eyes. You might wanna make the header/banner blend in more/better. Loads fasts and all links work:

5/10
  • Black_Legon
  • Beginner
  • Beginner
  • Black_Legon
  • Posts: 35
  • Loc: Netherlands

Post 3+ Months Ago

it's clean :)

But...here's some comments on your code:

    * A close tag was found without a matching open. This is probably an extra tag that you should remove.

    * The close tag </BODY> was not found, though the open was. You should open and close the BODY tag in order to get consistent performance on all browsers.

    *An open tag was found without a matching close. This may be an extra tag you should remove, or there may be a missing close tag later in the page that you need to add.
    * The open tag <HEAD> was not found, though the close was. You should open and close the HEAD tag in order to get consistent performance on all browsers.

    * Did not find the required open and close HTML tags. You should open and close the HTML tag in order to get consistent performance on all browsers.


And i agree with Rat about your header...

Have Fun
  • Gints Vitols
  • Novice
  • Novice
  • Gints Vitols
  • Posts: 15
  • Loc: Latvia

Post 3+ Months Ago

I think it's a nice site with easy to use navigation and clear structure. The simplicity of layout and visual style allows the reader to concentrate on content rather than appearance.


Speaking of problems I could find only some insignificant ones:

Use of strikeout text for hovered links is a somewhat nonstandard practice.

In screen resolutions higher than 800x600, the page content is left aligned in Internet Explorer and centered in Mozilla browser (I think it's because in Internet Explorer page displays in quirks mode because of missing DOCTYPE - as far as I understand content must be centered since margin-left and margin-right are both set to auto).

I would expect that navigation box or content box (whichever of them is shorter) extends down towards the footer (so that their bottom edges are aligned - like in table-based layouts), but I guess it's impossible to do in CSS-based layout without the use of some ugly hacks.

You might want to change the page background from white to some light pastel color, as white, in my opinion, gives a "feel" of incompleteness.


P.S. The idea with "Monkey" question is fun. Now you can tell for sure who has read the rules :).
  • rtm223
  • Mastermind
  • Mastermind
  • User avatar
  • Posts: 1855
  • Loc: Uk

Post 3+ Months Ago

Gints Vitols wrote:
In screen resolutions higher than 800x600, the page content is left aligned in Internet Explorer and centered in Mozilla browser (I think it's because in Internet Explorer page displays in quirks mode because of missing DOCTYPE - as far as I understand content must be centered since margin-left and margin-right are both set to auto).


IE always displays in quirks mode lol. Nope, the problem is an IE glitch, to get centered layout using CSS and get IE to behave nice you have to add:
Code: [ Select ]
text-align:center;

to the body. This will then get inherited to other elements so you then add:
Code: [ Select ]
text-align:left;

to the #container.

That will give you cross browser conformance, there is no other way. dontcha just love CSS :D

I agree with Gints about the links - I really hate when people do stuff like this. My rule of thumb is that only background colour/image, text colour and underlines should be added to a link on hover. Thats just me though.

I would suggest using some "faux columns":
http://www.alistapart.com/articles/fauxcolumns/
to get the nav column as long as the content and vice-versa.

I'm alos not keen on the urine-colour background. Sorry but that is what it makes me think of :lol:

Your code is pretty clean but as Black legon was saying - you really should be adding doctypes and making sure everything is properly nested and valid.

The following is the minimum requirement for a valid xhtml 1 doc:

Code: [ Select ]
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html>
  <head>
    <title>simple document</title>
  </head>
  <body>
    <p>a simple paragraph</p>
  </body>
</html>
  1. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  2. <html>
  3.   <head>
  4.     <title>simple document</title>
  5.   </head>
  6.   <body>
  7.     <p>a simple paragraph</p>
  8.   </body>
  9. </html>


If you are looking to do css layouts (Well done on that BTW :) ), you could clean up and strip down your markup by removing your center and font tags, and adding classes or id to elements. A good idea is to use as few types of tag as possible - I use:

div, p, a, img, span and ul/li for all of my formatting
I also use b, i, h* but I only use these for the purposes of seo.

This is all the advice I can think of. There is nothing "wrong" with the site and everything looks fine really. I just thought I would pass on some advice in terms of css useage :)
  • SharkShark
  • Guru
  • Guru
  • User avatar
  • Posts: 1013
  • Loc: Living In Today

Post 3+ Months Ago

not a bad site, some suggestions though, I am using a 1024x768 res and the site is sitting up in the upperleft corner of my screen. It looks kinda odd up there. Also, Damn but is that yellow on white necessary. geez Im still blinking. LOL i suggest you try and tone it down. Maybe you can change the rollover while your at it? I don't know, I dont like lines through my words when Im reading them. Maybe underline them on rollover or bold or change color. anything but the strikethrough. very basic site. but good content. keep working!
  • rtm223
  • Mastermind
  • Mastermind
  • User avatar
  • Posts: 1855
  • Loc: Uk

Post 3+ Months Ago

SharkShark wrote:
I am using a 1024x768 res and the site is sitting up in the upperleft corner of my screen. It looks kinda odd up there.


suggestion: click <a href="http://www.mozilla.org/products/firefox" target="_blank">here</a> :wink: that should solve the problem for you shark shark.
  • Rat
  • Guru
  • Guru
  • User avatar
  • Posts: 1190
  • Loc: desk

Post 3+ Months Ago

rtm: is that what you use?
  • rtm223
  • Mastermind
  • Mastermind
  • User avatar
  • Posts: 1855
  • Loc: Uk

Post 3+ Months Ago

rat: up until the point where I have to start fixing internet explorer bugs (like this one), yes. Although I'm getting quite good at internet explorer CSS hacks now and and can rattle off most of the common ones without having to open IE at all, which is nice :)

PS: do you think you could write a review in the review thread lol :wink:

//edit sorry rat I didn't see that post up there lol. Don't I look stupid now :roll: (editing this post so I don't clog up the review....)
  • sirussblack
  • Newbie
  • Newbie
  • sirussblack
  • Posts: 6

Post 3+ Months Ago

Heh, thanks for all your reviews. I do know about some of these errors but I think it looks okay with them until the next layout comes (that'll be much different). Er, I'll change the rollover links in a moment to an underline. The color is pretty bad but it's not horrible temporarily so I think I'll keep it for a while. And the banner was made by my friend for another layout similar to this and it looked okay with this one so I put it there. :) Other than that, thanks for the reviews.
  • Rat
  • Guru
  • Guru
  • User avatar
  • Posts: 1190
  • Loc: desk

Post 3+ Months Ago

rtm223 wrote:
PS: do you think you could write a review in the review thread lol :wink:


What do you mean? :roll: :?:
  • Gints Vitols
  • Novice
  • Novice
  • Gints Vitols
  • Posts: 15
  • Loc: Latvia

Post 3+ Months Ago

rtm223 wrote:
suggestion: click <a href="http://www.mozilla.org/products/firefox" target="_blank">here</a> :wink: that should solve the problem for you shark shark.


If there was a DOCTYPE declared for the site we are reviewing, it would show centered in Internet Explorer 6.0, too.
  • rtm223
  • Mastermind
  • Mastermind
  • User avatar
  • Posts: 1855
  • Loc: Uk

Post 3+ Months Ago

nope, it IS NOT a doctype problem. It is an IE rendering glitch. As you haven't bothered to check before "correcting me", look at this:

check out this example:
http://www.service6.org.uk/richard/notcenter.htm

it validates and has a valid doctype, no hint of "quirks mode"

Then check out this:

http://www.service6.org.uk/richard/propercenter.htm

text-align:center on the body is how to make pages center in IE. It is the only sensible way of doing it, and with a valid doctype it <b>makes no difference</b>



here are the validation results:

http://validator.w3.org/check?uri=http% ... &verbose=0
http://validator.w3.org/check?uri=http% ... &verbose=0

If that is not enough <a href="http://www.glish.com/css/3.asp" target="_blank">here</a> is the explanation from glish.com, and this is done with text-align:center; on the body.....


//siruss: if you stick the centering method I described you will get the same result in all browsers. It is best to have a valid doctype and a valid document though, and I would still advise you to add the neccesary code to make your file validate, as it <i>can</i> prevent browser incompatibilities. It just won't in this situation :wink:
  • Liquescence
  • Proficient
  • Proficient
  • User avatar
  • Posts: 353
  • Loc: Queens, NY

Post 3+ Months Ago

Honestly, the yellow is kind of painful for me to look at for extended periods of time. The top graphic looks a bit lo-quality; I'm not sure whether that was intended. Good content. I would recommend adding colors; everything is either black or yellow. Another is, I think the yellow background the right side should stretch out at least as long as the menu (http://sa.iwillspeak.us/dates.php). It seems kind of odd with the little white area in the middle there.
  • rtm223
  • Mastermind
  • Mastermind
  • User avatar
  • Posts: 1855
  • Loc: Uk

Post 3+ Months Ago

Liquescence wrote:
I think the yellow background the right side should stretch out at least as long as the menu (http://sa.iwillspeak.us/dates.php). It seems kind of odd with the little white area in the middle there.


To be fair, this is a little tricky with css, the divs aren't "locked" together as with tables, but the tutorial I posted above should help get this one sorted.
  • Gints Vitols
  • Novice
  • Novice
  • Gints Vitols
  • Posts: 15
  • Loc: Latvia

Post 3+ Months Ago

rtm223 wrote:
nope, it IS NOT a doctype problem. It is an IE rendering glitch. As you haven't bothered to check before "correcting me", look at this:

check out this example:
http://www.service6.org.uk/richard/notcenter.htm


I am starting to get increasingly confused.

I am viewing that page in Internet Explorer 6.0 and it is horizontally centered. That is, the distance from the left border of the browser window to the gray box with text is equal to the distance from that gray box to right border of browser window.

Also, there's http://msdn.microsoft.com/library/default.asp?url=/library/en-us/dnie60/html/cssenhancements.asp page on Microsoft Developer Network stating that Internet Explorer 6 in standards mode conforms to CSS when calculating margin widths (under "More Value for Your Property" section).

Are we talking about two different things or are we using two different versions of Internet Explorer?
  • rtm223
  • Mastermind
  • Mastermind
  • User avatar
  • Posts: 1855
  • Loc: Uk

Post 3+ Months Ago

using IE6 here, all updates blah blah blah, and I get the left aligned as I described....
It works the same on my comp at home and at work.

Do you know what I really hate - I'm starting to come to the conclusion that different copies of the <b>same version</b> of IE behave differently even though they shouldn't. This is a really weird phenomenon that is damn nigh impossible to tell if its real or not.

As for your link to msdn, I wouldn't trust that for one second - Microsoft also claims to have full support for CSS level one, but they don't....

If you want to discuss this further can we do it on a different thread or by pm. I'm not being funny but I just don't wanna be hijacking a reviews thread.
  • sirussblack
  • Newbie
  • Newbie
  • sirussblack
  • Posts: 6

Post 3+ Months Ago

I'll take everything under consideration. I think I'm going to keep it aligned to the left though, it looks unique which makes it a bit unique. And...maybe I'll put a few of the other ideas to use in someway or another. Or I'll just work on the next layout. :P Which would probably be up for a long time.

Post Information

  • Total Posts in this topic: 18 posts
  • Moderator: Website Reviewers
  • Users browsing this forum: No registered users and 6 guests
  • You cannot post new topics in this forum
  • You cannot reply to topics in this forum
  • You cannot edit your posts in this forum
  • You cannot delete your posts in this forum
  • You cannot post attachments in this forum
 
 

© 1998-2014. Ozzu® is a registered trademark of Unmelted, LLC.