|
|||||||||
|
|||||||||
| |||||||||
|
|||||||||
|
|||||||||
| |||||||||
|
|
|
| |||||||||
 |
|
|
«
Previous Thread
|
Next Thread
»
|
Thread Tools | Search this Thread | Rate Thread | Display Modes |
Dev Shed Forums Sponsor:
|
|
#1
March 8th, 2004, 06:47 AM
|
|||
|
|||
|
New site for a software provider
Hello everybody
I have been working on a new site for work. I have to admit to being more of a programmer than a designer so I need some style criticsim. Bring it on. http://www.pellcomp.co.uk/mark2/ thanks in advance
|
|
#2
March 9th, 2004, 04:37 PM
|
|||
|
|||
|
my 2cents
Ok well the layout after the preload is pretty decent. There's a good amount of content right at the user's fingertips which is good.
Here's some things I personally would change:
I hope that helps ya a little. Keep up the good work!
|
|
#3
March 10th, 2004, 12:26 AM
|
|||
|
|||
|
review by www.webmorpheus.com
that could easily be made into a "liquid" design.
right before the "home" link on the nav bar, make a td expand to 100% width. or you could expand it after the "help" button on the far right. yes, like mentioned earlier, expand the width of both side bars, make each at least 175 pixels or so. Add some nice stock photos of techy looking people next to or behind the title of the company. the drop down is nice, but make the boxes bigger...give some padding or something. should also get into some CSS, make either the scrollbar not showup if page isnt long enough, or set minheight and height to 100%. and, im afraid to say, that is just the start....
|
|
#4
March 10th, 2004, 04:32 AM
|
||||||
|
||||||
|
Thanks for taking the time to run through my site
Quote:
I have been thinking about ditching the blue grad but have left it purley because i don't know what to replace it with. I will think of something, any suggestions? Quote:
I hear you, i'll see what jiggery pokery i can come up with. One problem is that i don't have total free reign on the design and I was told to make that bar smaller so more content could be displayed. Quote:
Quote:
|
|
#5
March 10th, 2004, 04:37 AM
|
|||
|
|||
|
the copyright info should read: (changes in bold)
Copyright © 2004 Pellcomp Software Ltd
|
|
#6
March 10th, 2004, 04:38 AM
|
|||
|
|||
|
http://www.pellcomp.co.uk/mark2/sit...11b30842c1cbf88
the header looks a bit our of place. might want to try reworking that graphic.
|
|
#7
March 10th, 2004, 05:09 AM
|
|||
|
|||
|
Thanks for the copyright tip, thats done
Quote:
Which graphic, can you be more specific about what you think is wrong? One other problem is that the company logo is the wee palm tree in the top right and there isn't really a company font or any money to get a pro designer in to redo the logo.
|
|
#8
March 10th, 2004, 09:47 AM
|
||||
|
||||
|
URL: http://www.pellcomp.co.uk/mark2/
As per you request:  First page: The initial page does not load properly and diverts to the main page. Code:
Date: Wed, 10 Mar 2004 14:29:14 GMT Server: Apache/1.3.27 (Unix) mod_perl/1.27 PHP/4.3.1 Transfer-Encoding: chunked Content-Type: text/html; charset=iso-8859-1 11c <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN"> <HTML><HEAD> <TITLE>404 Not Found</TITLE> </HEAD><BODY> <H1>Not Found</H1> The requested URL /mark2/style.css was not found on this server.<P> <HR> <ADDRESS>Apache/1.3.27 Server at www.pellcomp.co.uk Port 80</ADDRESS> </BODY></HTML> There should not be an in-between page. Your main page [ http://www.pellcomp.co.uk/mark2/sit.../t128741/s.html ] should load directly. As you can see there are errors in above HTML. PHP programming: I am not sure if you have done the programming yourself or not but above URL is what I call UGLY URL. Passing data as GET is not only ugly, but unsafe. I would advise any experienced programmer to avoid using GET but that is something for the PHP board. Your site relies heavily on JavaScript. I usually have JavaScript disabled and i am not alone. As a rule, I do not use JavaScript in my web design thus ensuring cross browser, cross platform compatibility. You make up your own mind. DOCTYPE: You are missing the DOCTYPE. A simple <HTML> might be enough for newbies but not for advanced designer. Check: List of valid DTDs For future compatibility with XHTML [why isn't everyone using XHTML?], use lowercase for all attributes and tags. CSS & Style: Since you are using CSS, as everyone should, there should not be any formatting in you HTML with the exception of <b> and <i>. BTW, <u> should be avoided at all times in order to avoid misrepresenting text as link. Therefore attributes such as: bgcolor="#CCF4FF", leftmargin="1" topmargin="0", <font size="1"> </font> etc should not apear in the HTML. As an example: Code:
<td valign="top" bgcolor="#EBEBEB" width="100" height="350" class="right_line"> Should have been: Code:
<td class="right_line"> And all above should been specified in the class for that td. I am personally not very keen on sessions in PHP. But passing them as part of url is not safe ie: Code:
<a href="site.php?PHPSESSID=84878672e179e13a24878512affd6b78"> Positioning: Use of 'white space' for positioning is extremely questionable: Code:
<font color="#696969"> leading supplier of software solutions since 1991</font> Whatever happened to align, margin or padding? Empty tags: Line 123: Beside the fact that 'strong' should not be part of HTML and part of CSS, it is a waste to have: Code:
</table> <strong></strong> <p align="left"> IMG: Since I noticed you are using the XHTML <br /> tag then you should also consider closing img tags too: Code:
<img src="images/buttons/spacer.gif" width="1" height="18" alt="" /> Instead of assigning border="0" here, set it in the CSS. INPUT: Close the input. Code:
<input type="text" name="mysearchterms" size="10" /> TABLES: Use of tables should be limited to tabular data. Using tables for design is frowned upon although I must admit, thee are limitations on what could be achieved without tables. In any case, nested tables should be avoided. There should not be anything after </html>. It is also unsightly and useless. You also have some text in between that </table> and </body> Code:
1078930715</body> </html><font size="1">0.025949954986572</font> hit 84878672e179e13a24878512affd6b78 no 0 AESTHETICS: Since your page is a small one and can fit into most browsers, I would consider centre justification [personal choice] and slight margins, to let it breath. On the Left menu, Latest News sticks to the left edge and the right border and looks suffocated. I would decrease it size to the same as size as icon_dl_pics.gif or increase tits container in order to have some space around it. Width 100% does not really work here. All the text seems too close to the edges. A padding of 3-5 px would make it stand out better and breath. The SEARCH and GO buttons are not pretty. Use the standard browser buttons or design something different. They look TOO flat and lifeless and frankly not like a button. On both left and right side, the dotted border is way too close to the elements. Border should not stick to text or button. Padding is your friend. As mentioned in another thread, use of image in navigation should only be attempted for special effects with can not be achieved via text. Although the text of your navigation looks nice and rich, you can create a similar effect without using images and rollover image. That would mean smaller page, faster load time, less JavaScript thus again, smaller page and faster load time. arrows.gif could be implemented beautifully via CSS in LIST or background image. Look into it. It would eliminate the need of entering it for each line. The dotted border under the nav bar, looked more jugged and thorn. I would look into it. That should be enough to get you going. Good luck 
__________________
© AZ AZbb :: AZ Bulletin Board - Secure BBS script, Front page CMS, Chat, No database Simple Guide to Apache and PHP installation on Windows
|
|
#9
March 10th, 2004, 10:56 AM
|
|||
|
|||
|
Clan your the man!
Thank you for the very through critique. Theres a lot there so i would like to take some time and go through your points. Is it ok to get back to you with any questions?
|
|
#10
March 10th, 2004, 12:41 PM
|
||||
|
||||
|
Quote:
No prob. Any time. Good luck
|
|
#11
March 10th, 2004, 11:00 PM
|
||||
|
||||
|
Quote:
The initial loader logo is very nice. April
|
|
#12
March 31st, 2004, 05:01 AM
|
|||
|
|||
|
do you know how i can set valign="top" in CSS?
|
|
#13
March 31st, 2004, 06:42 AM
|
||||
|
||||
|
Quote:
vertical-align: top Good luck
|
|
#14
March 31st, 2004, 06:53 AM
|
|||
|
|||
|
i have this code in my style sheet:
Code:
body, span, p, pre, table, tr, th, td, ul, li, .form, form, input, select, IFRAME, textarea {
font-family: bookman antiqua, verdana;
font-size: x-small;
color: #000000;
SCROLLBAR-FACE-color: #99BCDF;
SCROLLBAR-HIGHLIGHT-COLOR: #6699CC;
SCROLLBAR-SHADOW-COLOR: #FFFFFF;
SCROLLBAR-3DLIGHT-COLOR: #EEEEEE;
SCROLLBAR-ARROW-COLOR: #FFFFFF;
SCROLLBAR-TRACK-COLOR: #FFFFFF;
SCROLLBAR-DARKSHADOW-COLOR: #336699
vertical-align: top;
}
And it doesn't work. Have I made a mistake? BTW, I have only tested it on IE 6 Last edited by tobycatlin : March 31st, 2004 at 07:00 AM. |
Del.icio.us
Digg
Google
Spurl
Blink
Furl
Simpy
Y! MyWeb
