What's in the eBook?
- Full list of reasons for poor performance
- Ultimate measures to speed things up
- Primary web monitoring types
- KPIs you should be monitoring in order to increase your ROI
Course Of The Month
Become a Premium Member and unlock a new, free course in leading technologies each month.
Solved
Please critique my work!
Posted on 2012-03-31
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! :)
thanks,
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>
thanks,
[X]
Welcome to Experts Exchange
Add your voice to the tech community where 5M+ people just like you are talking about what matters.
- Help others & share knowledge
- Earn cash & points
- Learn & ask questions
-
4
2
- +1
8 Comments
sorry, it seems I did not attach the original png for this.
boxTest.png
boxTest.png
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-Compatibl
<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>
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-Compatibl
<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>
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?
> 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.
% is better becase it displays much better/consistently on different monitor sizes, aspect ratios and resolutions.
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.
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.
Featured Post
We value your feedback.
Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!
Question has a verified solution.
If you are experiencing a similar issue, please ask a related question
Preface
This is the third article about the EE Collaborative Login Project. A Better Website Login System (http://www.experts-exchange.com/A_2902.html) introduces the Login System and shows how to implement a login page. The EE Collaborative Logi…
Use these top 10 tips to master the art of email signature design. Create an email signature design that will easily wow recipients, promote your brand and highlight your professionalism.
In this tutorial viewers will learn how to style a decorative dropcap for the first letter in a paragraph using CSS.
In CSS, create a new paragraph class by typing "p.fancy": Then, to style only the first letter of the first sentence, include the ps…
The viewer will learn the basics of jQuery including how to code hide show and toggles.
Reference your jQuery libraries:
(CODE)
Include your new external js/jQuery file:
(CODE)
Write your first lines of code to setup your site for jQuery…
Suggested Courses
Earn Certification
HTML5 Specialist - Certification
Free withPremium
Course of the Month3 days, 21 hours left to enroll
630 members asked questions and received personalized solutions in the past 7 days.
Join the community of 500,000 technology professionals and ask your questions.