fitzdevlin Posted January 5, 2007 Share Posted January 5, 2007 i'm new to this whole html/css world, and i'd like to clean this code up a little.anyone have suggestions?[code]<embed allowScriptAccess="never" allowNetworking="internal" enableJSURL="false" enableHREF="false" saveEmbedTags="true" src="http://www.filelodge.com/files/room18/475638/player.swf" enableJavascript="false" allowScriptAccess="never" allowNetworking="internal" allowScriptAccess="never" menu="false" quality="high" width="300" height="320" name="index" type="application/x-shockwave-flash" pluginspage="http://www.macromedia.com/go/getflashplayer" flashvars="playList=http://www.filelodge.com/files/room18/475638/playlist2.xml&ShowEQ=1&initVol=60" wmode="transparent"><Style type="text/css" source="http://www.strikefile.com/" author="Thomas Zwaagstra">table, tr, td{background-color:transparent;border-style:none;}table table table, table table table td{background-color:transparent;}body{background-color:rgb(57,51,51);background-image:url(http://img409.imageshack.us/img409/6186/skullofnudeste4.jpg);background-position:center center;background-repeat:no-repeat;background-attachment:fixed;}table table table table, table table table td{background-image:none;background-color:transparent;}body, div, p, strong, td, .text, .blacktext10, .blacktext12, a.searchlinkSmall, a.searchlinkSmall:link, a.searchlinkSmall:visited{color:rgb(255,204,0);font-size:11pt;font-weight:bold;font-style:normal;text-decoration:none;text-transform:lowercase;font-family:monospace,sans-serif;}.orangetext15, strong, .lightbluetext8, .whitetext12, .nametext, .btext, .redtext, .redbtext{color:rgb(255,204,0);font-size:12pt;font-weight:bold;font-style:normal;text-decoration:none;text-transform:lowercase;font-family:monospace,sans-serif;}a, a font, a:link, a:visited, a.navbar, a.navbar:link, a.navbar:visited, a.man, a.man:link, a.man:visited, a.man font, a.redlink, a.redlink:visited{color:rgb(255,204,0);font-size:11pt;font-weight:bold;font-style:normal;text-decoration:underline ;text-transform:lowercase;font-family:monospace,sans-serif;}a:hover, a:hover font, a:active, a.navbar:hover, a.navbar:active, a.man:hover, a.man:active, a.man:hover font, a.searchlinkSmall:hover, a.searchlinkSmall:active, a.redlink:hover, a.redlink:hover{color:rgb(255,51,0);font-size:12pt;font-weight:bold;font-style:normal;text-decoration:underline ;text-transform:uppercase;font-family:comic sans ms,sans-serif;}table table table, div table table{border-style:none;border-width:1px;background-color:rgb(51,51,51);opacity:0.8;..-opacity:0.8;filter:alpha(opacity=80);}table table table table, div table table table{border-style:none;}.contacttable td.text a img {width:auto; height:auto;}type="text/css">table table table table{border: 0px;}table table table {border: 1px ridge white; background-color:black;}</style></A>[/code] Quote Link to comment https://forums.phpfreaks.com/topic/32993-suggestions-on-cleaning-this-up/ Share on other sites More sharing options...
wildteen88 Posted January 5, 2007 Share Posted January 5, 2007 Redo the whole lot. You dont need to style every single tag on your page. Alot of the time you are repeating your self. Such as here:[code]table, tr, td{background-color:transparent;border-style:none;}table table table, table table table td{background-color:transparent;}[/code]Above you telling the browser to give table, tr and td tags a transparent background. Then on the next line you say give any td tag within table table table a transparent background. There is no need for that as you've already done it above!I ahve not seen your HTML source but looking at the CSS you are using them far too often. Cut down on the use of tables and try to go for a CSS layout instead. You will have far less html and have smaller file space (which can help to save bandwidth). Quote Link to comment https://forums.phpfreaks.com/topic/32993-suggestions-on-cleaning-this-up/#findComment-153607 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.