View Single Post
Old 06-30-2008, 01:20 PM  
phil-flash
Confirmed User
 
phil-flash's Avatar
 
Industry Role:
Join Date: Apr 2003
Location: Florida
Posts: 649
Quote:
Originally Posted by harvey View Post
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
That is a good idea... I will take a look at doing that. THANKS!
phil-flash is offline   Share thread on Digg Share thread on Twitter Share thread on Reddit Share thread on Facebook Reply With Quote