View Single Post
Old 06-27-2008, 07:42 PM  
harvey
Confirmed User
 
harvey's Avatar
 
Industry Role:
Join Date: Jul 2001
Location: 127.0.0.1
Posts: 9,266
looks nice, although I'd use an ul for the yearbook, that way you can just add a linked thumb and it will automatically adjust to your css. in example, where you have

Code:
		<div class="yearbook-tn">
			<div class="yearbook-tn-placeholder">
				<a href="/student-nikki.php"><img src="http://facedownassupuniversity.com/pix/yb_nikki.jpg" alt="Next Door Nikki" /><br />Next Door Nikki</a>
			</div>
		</div>
I'd use

Code:
		<li><a href="/student-nikki.php"><img src="http://facedownassupuniversity.com/pix/yb_nikki.jpg" alt="Next Door Nikki" /><br />Next Door Nikki</a></li>

so, instead of
Code:
#yearbook					{
							position:				relative;
							height:					1360px;
							width:					980px;
							background:				#759aea	url('../pix/dani-legspread.png') no-repeat scroll;
							background-position: 	345px 82px;
							border-top:				1px solid black;
							border-bottom:			1px solid black;
							}	
#yearbook-padding			{
							margin-top:				3px;
							padding:				0 0px 0px 29px;
							}
.yearbook-tn,
.yearbook-tn-special		{
							float: 					left;
							height: 				200px;
							width: 					133px;
							text-align: 			center;
							margin: 				10px 25px 0 0;
							padding:				0;
							}	

.yearbook-tn-placeholder	{
							height: 				195px;
							width: 					133px;
							line-height: 			15pt;
							background: 			#435c9e url('../pix/yb_bg.gif') no-repeat
							}
.yearbook-tn img			{
							height:					175px;
							width:					131px;
							border:					0;
							position:				relative;
							top:					1px;
							}
.yearbook-tn-special img	{
							height:					176px;
							width:					133px;
							border:					0;
							}
.yearbook-tn a:link					{text-decoration:		none; color: #000;}
.yearbook-tn a:visited				{text-decoration:		none; color: #000;}
.yearbook-tn a:hover				{text-decoration: 		none; color: #fff992; background: none;}
.yearbook-tn a:active				{text-decoration: 		none; color: #000;}
.yearbook-tn-special a:link			{text-decoration:		none; color: #000;}
.yearbook-tn-special a:visited		{text-decoration:		none; color: #000;}
.yearbook-tn-special a:hover		{text-decoration: 		none; color: #fff992; background: none;}
.yearbook-tn-special a:active		{text-decoration: 		none; color: #000;}
you'd need to define just the ul, li, a: and container div if needed. Less code and easier to manage. ah, I'd make that black font white or at least some contrasting color over a blue bg. Anyway, well done, nicely organized CSS
__________________
This post is endorsed by CIA, KGB, MI6, the Mafia, Illuminati, Kim Jong Il, Worldwide Ninjas Association, Klingon Empire and lolcats. Don't mess around with it, just accept it and embrace the truth
harvey is offline   Share thread on Digg Share thread on Twitter Share thread on Reddit Share thread on Facebook Reply With Quote