Jump to content

your opinions on my website


MDanz

Recommended Posts

The links have a childish font

I would lose their bullets

align them left maybe(user search and user stacks on the right)

make the links look like buttons (everything else looks like buttons).

 

Instead of making the site content dividers extend across the screen, you could cut them off and and a drop shadow to gray (seems to be a more modern trend.)

http://www.westlakehardware.com/ has the drop shadow on the sides, instead of extending the background images across the whole browser, and it looks much nicer.

It also ads a much lighter feeling to the site, right now your site makes me feel kinda sad inside cause of all the dark colors.

 

I remember your old thread and i like this version much more (although i preferred the grays to the green), just needs a little help, almost there!

Link to comment
Share on other sites

The site looks to heavy to me. Invest in image compression and use GZ (like gzip) for faster serving of the files. Can be as simple as adding this one line at the top of your code:

<?php
ob_start('gz_handler');

 

Your forms "reggie -> Uzer" and "search" are suseptable to XSS, I'd recommend nullifying the following characters of which are at risk:

;	\	/	<	>	=

Link to comment
Share on other sites

links look better. It might just be me, but im still not a fan of the rows going across the screen

 

got some html errors though

 

http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Fwww.stackway.com%2Findex.php

 

Did you learn HMTL a long time ago?

 

I noticed you are using tables for your layout, divs are whats generally used now.

I reccomend http://www.blueprintcss.org/ for formatting them.

Link to comment
Share on other sites

thanks for feed back.  a majority of the html errors are because div id's have to start with a letter, but because of javascript i started the id's with a number.  For some reason i couldn't get it  the javascript code to work starting with a letter. So for every result(block) it will come with invalid html markup.  I can live with that, most other site's i checked are not perfect either with html markup.

 

what's the benefit of divs over tables, is it a user advantage or a developer advantage?

 

yeah i'm working on the site to make it look "lite" on the eyes.

 

I started html since october last year, same wth php, so i'm learning on the go.

Link to comment
Share on other sites

Divs are more versatile then tables and (this is hearsay) they are easier for browsers to render.

 

Also you can take advantage of css grid frameworks with divs. Bringing me back to blueprintcss.

 

Makes life so easy.

<html>
<div class='container'><!--Creates a centered page thats 950px good to work on almost all resolutions-->
   <div class='span-24 last' id='banner'><!--'span-24 last' means it will be 24 columns wide and last means the next row will start below it. -->
   </div>
   <div class='span-24 last' id='nav'><!--'span-24 last' means it will be 24 columns wide and last means the next row will start below it. -->
      <a href=''>Link 1</a> | <a href=''>Link 2</a> | <a href=''>Link 3</a> | 
   </div>
   <div class='span-4' id='ad1'><!--'span-4' will take up 4 columns and leave a 10px buffer between itself and content -->
   </div>
   <div class='span-18' id='content'><!--'span-18' will take up 18 columns and leave a 10px buffer between itself and ad2-->
      <p>Notice that all of the rows add up to 24. Its a 24 column system. Refer to the wiki for more help</p>
   </div>
   <div class='span-4 last' id='ad2'><!--'span-4 last' will take up 4 columns and finish of this row-->
   </div>
</div>

 

is all you need to create a banner div that floats perfectly across

a nav div right below it

an ad div to the left content in the middle and an ad2 div to the right.

 

set the heights in css and your set.

 

it also takes care of most css issues with ie.

 

http://urlgreyhot.com/personal/resources/photoshop_template_blueprint_css_comps . use this while your creating your site layout in photoshop and most of the layout work is done for you. just create background images that are the right size.

Link to comment
Share on other sites

This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.