Go Premium for a chance to win a PS4. Enter to Win

x
?
Solved

Please critique my work!

Posted on 2012-03-31
8
Medium Priority
?
197 Views
Last Modified: 2012-04-01
Greetings,

I've included the code for a "box test" I was given by someone to test my abilities and how I do things with regards to structure and style.  They sent me a .png (attached) and asked me to create it using HTML and CSS.  

Incidentally, while the png shows each box with a letter designation, I did not include that figuring it was beside the point of the exercise i.e. they wanted to see how I deal with creating a table-less layout.  Perhaps that was a mistake in judgement?

I would ask any experts to inspect my work and critique what I've done.  Please be specific about where I could have done something perhaps better and I also want to know if I did something that was a no-no or even egregiously wrong by today's standards.

Tell me the truth but, be nice! :)

<!DOCTYPE html>
<html lang="en">
	<head>
		<meta charset="utf-8" />
		<meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1" />
		<title>Box Test</title>
		<meta name="description" content="Box Test" />
		<style>
			#wrapper {
				width: 900px;
				height: auto;
			}
			#boxA, #two_box {
				border: 1px solid #000;
			}
			
			#boxB, #boxC {
				border-bottom: 1px solid #000;
			}
			
			#boxA, #two_box {
				width: 450px;
			}
			
			#boxA, #boxD, #two_box {
				height: 300px;
			}
			
			#boxA {
				background: #f4cccc;
			}
			#boxB {
				background: #b6d7a8;
				height: 150px;
			}
			#boxC {
				background: #cfe2f3;
				height: 149px;
			}
			#boxD {
				background: #ffe599;
				border: solid #000;
				border-width: 0 1px 1px 1px;
			}
			#two_box {
				float: right;
				margin-top: -302px;
			}
		</style>
	</head>
	<body>
		<div>
			<header>
				<h1>Box Test</h1>
			</header>
			<div id="wrapper">
				<div id="boxA"></div>
				<div id="two_box">
					<div id="boxB"></div>
					<div id="boxC"></div>
				</div>
				<div id="boxD"></div>
			</div>
			<footer></footer>
		</div>
	</body>
</html>

Open in new window


thanks,
0
Comment
Question by:max7
8 Comments
 
LVL 1

Author Comment

by:max7
ID: 37791581
sorry, it seems I did not attach the original png for this.
boxTest.png
0
 
LVL 20

Expert Comment

by:n2fc
ID: 37791632
Good job!  Neatly organized and very readable code...
0
 
LVL 20

Accepted Solution

by:
Daniel Van Der Werken earned 1200 total points
ID: 37791887
I would get rid of the unused <footer> tag. Unless it's necessary, which I doubt.
It looks like you friend wants the box and the text centered.
Also, I changed the width of the two smaller boxes to 50% instead of 450px.

Here's the modified code I came up with:

<!DOCTYPE html>
<html lang="en">
      <head>
            <meta charset="utf-8" />
            <meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1" />
            <title>Box Test</title>
            <meta name="description" content="Box Test" />
            <style>
                  #outer {
                        width: 100%;
                  }
                  #wrapper {
                        width: 90%;
                        height: auto;
                        margin: 0px auto;
                  }
                  #boxA, #two_box {
                        border: 1px solid #000;
                  }
                  
                  #boxB, #boxC {
                        border-bottom: 1px solid #000;
                  }
                  
                  #boxA, #two_box {
                        width: 50%;
                  }
                  
                  #boxA, #boxD, #two_box {
                        height: 300px;
                  }
                  
                  #boxA {
                        background: #f4cccc;
                  }
                  #boxB {
                        background: #b6d7a8;
                        height: 150px;
                  }
                  #boxC {
                        background: #cfe2f3;
                        height: 149px;
                  }
                  #boxD {
                        background: #ffe599;
                        border: solid #000;
                        border-width: 0 1px 1px 1px;
                  }
                  #two_box {
                        float: right;
                        margin-top: -302px;
                  }
                  #boxtest {
                        text-align: center;
                  }
            </style>
      </head>
      <body>
            <div>
                  <header>
                        <h1 id="boxtest">HTML/CSS Box Test</h1>
                  </header>
                  <div id="outer">
                        <div id="wrapper">
                              <div id="boxA"></div>
                              <div id="two_box">
                                    <div id="boxB"></div>
                                    <div id="boxC"></div>
                              </div>
                              <div id="boxD"></div>
                        </div>
                  </div>
            </div>
      </body>
</html>
0
VIDEO: THE CONCERTO CLOUD FOR HEALTHCARE

Modern healthcare requires a modern cloud. View this brief video to understand how the Concerto Cloud for Healthcare can help your organization.

 
LVL 1

Author Comment

by:max7
ID: 37791951
I would get rid of the unused <footer> tag. Unless it's necessary, which I doubt.

Yes, I noticed that as well.  Does it look terribly bad to have that hanging around unused?

It looks like you friend wants the box and the text centered.

Not sure what you mean by wanting the box centered.  Could you expand on that?

Regarding the text, are you indicating you think it was a mistake NOT to include it?

Also, I changed the width of the two smaller boxes to 50% instead of 450px.

Is there a particular advantage to using % for width instead of px?  Is it a poor choice to use pixel instead of % here?

Keeping the issues you mentioned in mind, overall, do you think the work here is decent?  Can you give it a grade e.g. A through F?
0
 
LVL 24

Assisted Solution

by:Eirman
Eirman earned 800 total points
ID: 37792027
> Is there a particular advantage to using % for width instead of px?

% is better becase it displays much better/consistently on different monitor sizes, aspect ratios and resolutions.
0
 
LVL 1

Author Comment

by:max7
ID: 37792067
Not sure what you mean by wanting the box centered.  Could you expand on that?

ok, I feel foolish.  I wasn't carefully reviewing your code changes but I now see the outer div and how you centered the wrapper div within it. Doh!

% is better becase it displays much better/consistently on different monitor sizes, aspect ratios and resolutions.

Excellent, thanks for that.

Still hoping for an over-all assessment of this work.
0
 
LVL 20

Expert Comment

by:Daniel Van Der Werken
ID: 37792196
Grade A- because of the centering and sizing. Plus I think you should put in the letters A B C D like the image has.
0
 
LVL 1

Author Closing Comment

by:max7
ID: 37793151
Thanks very much for the feedback, greatly appreciate it!
0

Featured Post

What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Is your Office 365 signature not working the way you want it to? Are signature updates taking up too much of your time? Let's run through the most common problems that an IT administrator can encounter when dealing with Office 365 email signatures.
Originally, this post was published on Monitis Blog, you can check it here . Websites are getting bigger and more complicated by the day. Video, images and custom fonts are all great for showcasing your product or service. But the price to pay in…
HTML5 has deprecated a few of the older ways of showing media as well as offering up a new way to create games and animations. Audio, video, and canvas are just a few of the adjustments made between XHTML and HTML5. As we learned in our last micr…
The viewer will learn how to create a basic form using some HTML5 and PHP for later processing. Set up your basic HTML file. Open your form tag and set the method and action attributes.: (CODE) Set up your first few inputs one for the name and …
Suggested Courses

885 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question