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.
Results 1 to 12 of 12
  1. #1
    New to the CF scene
    Join Date
    Oct 2004
    Posts
    8
    Thanks
    0
    Thanked 0 Times in 0 Posts

    New site Under Development, Looking for creative criticism!!

    Once I am finished school I hope to develope web pages part time to help pay for my student loans. I am in the process of designing my site and was hoping to get some feed back. Thanks!

    Here is the site address www.lwd.ca

    Labyrinth Web Designs

  • #2
    Senior Coder
    Join Date
    Jul 2004
    Location
    New Zealand
    Posts
    1,315
    Thanks
    0
    Thanked 2 Times in 2 Posts

    The term is "constructive" criticism

    Wow, I really like the design of the header, the body and footer don't really help the design, they seem dis-jointed, seperate even.

    Your code however is horrible. I sense Dreamweaver has been at it again. You need to learn HTML and move to clean, semantic code. For example, when you have a list, such as a list of links, you should mark them up a list. You also need to get rid of the layout tables.

    Hmm, just realised your Javascript doesn't work at all.

    My advice, you do the design, and contract out the coding.

  • #3
    New to the CF scene
    Join Date
    Oct 2004
    Posts
    8
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Thanks!

    Thanks I fixed the javascript(spelling mistake in the url of my script file)

    The reason that I have so many tables is to allow my page to expand as more content is added. The way it is set up, the background image tiles as more content is writen. It also makes my images fit together nice and tight. If there is a better way of doing it without the tables let me know, I don't enjoy having so many tables, they can get to be a pain in the *** to set up. But once they are set up you are done and you can add as much content as you like and the images still look good.

    I see what you mean about the footer being seperated, I'll see what I can do with it.

    Thanks for the input

    Labyrinth Web Designs

  • #4
    Senior Coder
    Join Date
    Jul 2004
    Location
    New Zealand
    Posts
    1,315
    Thanks
    0
    Thanked 2 Times in 2 Posts
    index.html
    Code:
    <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
    <html lang="en">
    	<head>
    		<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
    		<title>lwd.ca</title>
    		<link rel="stylesheet" href="css/index.css" type="text/css" media="screen, projector" title="default">
    	</head>
    	<body>
    		<div id="header">
    			<h1>Labyrinth Web Designs</h1>
    			<p>
    				Imagination is the beginning of creation.
    			</p>
    		</div>
    
    		<div id="introduction">
    			<p>
    				We are a fresh and energetic web development team dedicated to building creative and dynamic web pages.
    			</p>
    			<p>
    				With cutting edge technology and reliable service, we can build a web page to suit your needs.
    			</p>
    			<p>
    				In this age of competitive advertising on the internet, you need a web page that will not only get you clients but also compliments.
    			</p>
    			<p>
    				<a href="/about/contact">Contact us</a> for a free consultation and find out how a web page could benefit you.
    			</p>
    		</div>
    
    		<div id="news">
    			<h2>News</h2>
    			<dl>
    				<dt>24.10.04</dt>
    				<dd>New page design is up and running</dd>
    				<dt>22.10.04</dt>
    				<dd>New sample page added, <a href="/InternalSites/Aplac/web/template.html">Aplac Investigations</a></dd>
    				<dt>21.10.04</dt>
    				<dd>New sample page added, <a href="/InternalSites/GreenPine/Web/GreenPine_Template.htm">Green Pine Recyclers</a></dd>
    			</dl>
    		</div>
    
    		<ul id="menu">
    			<li><a href="/">Home</a></li>
    			<li><a href="/news/">News</a></li>
    			<li><a href="/portfolio/">Portfolio</a></li>
    			<li><a href="/services/">Services</a></li>
    			<li><a href="/about/">About Us</a></li>
    			<li><a href="/links/">Links</a></li>
    		</ul>
    	</body>
    	<script type="text/javascript">
    		var new_menu = document.getElementById("menu").cloneNode(true);
    		new_menu.setAttribute("id","footer_menu");
    		document.getElementsByTagName("body")[0].appendChild(new_menu);
    	</script>
    </html>
    index.css
    Code:
    html	{
    	background:#000 url("../images/footer.jpeg") center bottom no-repeat;
    	line-height:1.25em;
    	font-size:1em;
    	color:#9cb0bd;
    }
    body	{
    	width:728px;
    	margin:auto;
    	position:relative;
    	margin-bottom:51px;
    	padding-bottom:3em;
    }
    
    #header	{
    	background-image:url("../images/header.jpeg");
    	height:135px;
    	text-align:center;
    }
    #header h1,
    #header p	{
    	display:none;
    }
    
    #introduction	{
    	margin:auto 2em;
    	text-align:center;
    }
    
    #news h2	{
    	text-align:center;
    }
    #news h2:before	{
    	content:".: ";
    }
    #news h2:after	{
    	content:" :.";
    }
    #news dl	{
    	text-align:left;
    }
    #news dl dt	{
    	float:left;
    	width:5em;
    }
    
    #menu	{
    	position:absolute;
    	left:93px;
    	top:92px;
    
    	width:542px;
    	height:24px;
    
    	margin:0;
    	padding:0;
    
    	list-style-type:none;
    }
    #menu li	{
    	float:left;
    	margin:0 16px;
    }
    
    #menu a	{
    	background-image:url("../images/menu.png");
    	display:block;
    
    	overflow:hidden;
    	padding-top:24px;
    	height:0;
    }
    
    #menu a[href="/"]	{
    	background-position:0 0;
    	width:44px;
    }
    #menu a[href="/"]:hover	{
    	background-position:0 24px;
    }
    
    #menu a[href="/news/"]	{
    	background-position:-43px 0;
    	width:46px;
    }
    #menu a[href="/news/"]:hover	{
    	background-position:-43px 24px;
    }
    
    #menu a[href="/portfolio/"]	{
    	background-position:-89px 0;
    	width:78px;
    }
    #menu a[href="/portfolio/"]:hover	{
    	background-position:-89px 24px;
    }
    
    #menu a[href="/services/"]	{
    	background-position:-167px 0;
    	width:66px;
    }
    #menu a[href="/services/"]:hover	{
    	background-position:-167px 24px;
    }
    
    #menu a[href="/about/"]	{
    	background-position:-233px 0;
    	width:73px;
    }
    #menu a[href="/about/"]:hover	{
    	background-position:-233px 24px;
    }
    
    #menu a[href="/links/"]	{
    	background-position:-306px 0;
    	width:42px;
    }
    #menu a[href="/links/"]:hover	{
    	background-position:-306px 24px;
    }
    
    #footer_menu	{
    	position:absolute;
    	bottom:0;
    	list-style-type:none;
    	padding:0;
    	margin:0;
    	width:100%;
    	text-align:center;
    }
    #footer_menu li	{
    	display:inline;
    	padding:0 0.5em;
    }
    Your header and footer images were cropped to 728px.
    Attached Thumbnails Attached Thumbnails New site Under Development, Looking for creative criticism!!-menu.png  

  • #5
    Regular Coder
    Join Date
    Oct 2004
    Location
    England
    Posts
    282
    Thanks
    0
    Thanked 0 Times in 0 Posts
    I'm liking the shape of that, very nifty.

  • #6
    Senior Coder
    Join Date
    Jul 2004
    Location
    New Zealand
    Posts
    1,315
    Thanks
    0
    Thanked 2 Times in 2 Posts
    As a web-developer you should know this stuff. I suggest you try and get yourself a position as a web-designer or find a coder to work with you.

  • #7
    New to the CF scene
    Join Date
    Oct 2004
    Posts
    8
    Thanks
    0
    Thanked 0 Times in 0 Posts
    I do know this stuff, I understand the code you wrote I just don't see the advantage of it over the code I wrote. I also don't understand your statement "I sense dreamweaver at it again" since the code you wrote looks like the code genterated by using the design view in dreamweaver.
    If i understand your code correctly you use a css for the mouse-over events, is there a reason for that, is it better then using javascript? If it is then I might consider doing it that way. But if there is no advantage then it is easier for me to use javascript, since imageready generates the code for you.

  • #8
    Senior Coder
    Join Date
    Jul 2004
    Location
    New Zealand
    Posts
    1,315
    Thanks
    0
    Thanked 2 Times in 2 Posts
    Quote Originally Posted by lwd.ca
    I do know this stuff, I understand the code you wrote I just don't see the advantage of it over the code I wrote.
    You don't see the advantage of using clean semantic markup? Seperating structure from style? Using CSS for layout and style rather than tables?
    Quote Originally Posted by lwd.ca
    I also don't understand your statement "I sense dreamweaver at it again" since the code you wrote looks like the code genterated by using the design view in dreamweaver.
    It isn't. I always hand-code. What I meant was your code looks as though it was generated by some WYSIWYG editor. It's bloated and generic.
    Quote Originally Posted by lwd.ca
    If i understand your code correctly you use a css for the mouse-over events, is there a reason for that, is it better then using javascript? If it is then I might consider doing it that way. But if there is no advantage then it is easier for me to use javascript, since imageready generates the code for you.
    CSS should be used for all styling. That includes rollovers. My code requires no forced preloading, uses one image, is faster than Javascript, and if the user has Javascript disabled, my code still works. The only problem, of course, is probably Internet Explorer, and if the user has images disabled. But then they can always disable the stylesheet to see the text.

  • #9
    New to the CF scene
    Join Date
    Oct 2004
    Posts
    8
    Thanks
    0
    Thanked 0 Times in 0 Posts
    I can see how your way would be good, but if somebody has images disabled and javascript then i guess they wouldn't be coming to my site. If somebody has images disabled then my site will be useless to them because I develope my sites in photoshop and they are just one big image sliced up. If I am ever developing a site where I want it to reach the maximun amount of people with a varitey of systems, I will keep what you said in mind.

  • #10
    Senior Coder
    Join Date
    Jun 2002
    Location
    The Netherlands, Baarn, Ut.
    Posts
    4,253
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Hooo-boy...

    For a web design company, that's a very poor attitude...
    Instead of cutting off significant numbers of visitors - and potential customers, especially your site should aim at the largest possible visitor base. Since your site isn't about photography or art, the imagery is purely decorative and should never, ever be an obstruction for any visitor to make full use of your site.
    Accessibility of web sites to others then those with 20/20 vision, a graphical browser and a fast connection is not a luxury; it's something you really should take care of.
    Regards,
    Ronald.
    ronaldvanderwijden.com

  • #11
    New to the CF scene
    Join Date
    Oct 2004
    Posts
    8
    Thanks
    0
    Thanked 0 Times in 0 Posts
    I didn't mean to sound that snotty. I should have worded myself better.
    All of my images are optimized when i save then, some of the gifs are saved using only 32 colors and take 2 or 3 seconds to load with a 28.8 modem. So I do take into consideration that some people have dial-up. But I personal don't know a single person that still has dial-up, I know people that live on a farm 1 hour away from a city and they have high-speed internet. So I don't think that I am keeping to many people from my site by using lots of images, but I could be wrong, and if I am, well you learn something new everyday.
    Thanks for pointing out the attitude, I didn't realize how it sounded until you said something.

    thanks
    Labyrinth Web Designs

  • #12
    Senior Coder
    Join Date
    Jun 2002
    Location
    The Netherlands, Baarn, Ut.
    Posts
    4,253
    Thanks
    0
    Thanked 0 Times in 0 Posts

    Not by images alone...

    I am keeping to many people from my site by using lots of images
    That's not what I meant; people should be able to use your site to its full extend without being able to see images.
    Regards,
    Ronald.
    ronaldvanderwijden.com


  •  

    Posting Permissions

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