On 31 Jul 2008 in macromedia.dreamweaver, Sw Jiten wrote:
> Dear Murray :
>
> 1) I can not enter the link you sent me.. I do not know why... I
> even tried :
> www.murraytestsite.com and nothing...
Works fine here.
> 2) You said you neither approve parts of my code... I will
> appreciate to know
> what parts...? This ia s great opportunity for me to learn from a
> big one like you, so please do not dissapoint me and show me how to
> optimize some parts that you might consider... thanks !!!
..style15 {font-size: 10px}
</style>
<!--Fireworks 8 Dreamweaver 8 target. Created Sat Apr 12 00:55:50 GMT-0500 2008-->
</head>
<body bgcolor="#74acc7">
<table width="779" border="0" align="center" cellpadding="0" cellspacing="0">
<!-- fwtable fwsrc="websitetemplate.png" fwbase="websitetemplate.gif" fwstyle="Dreamweaver" fwdocid = "1112975483" fwnested="0" -->
<tr>
<td><img src="img/spacer.gif" width="20" height="1" border="0" alt="" /></td>
<td><img src="img/spacer.gif" width="9" height="1" border="0" alt="" /></td>
<snip>
<div id="flashheader">
</div>
<script type="text/javascript">
var so = new SWFObject("header.swf", "movie", "557", "87", "8", "#74ACC7");
so.write("flashheader");
</script>
<snip>
<td rowspan="5" colspan="7">
Murray's being way too nice. Where to begin?
See the style above? Note that it says 'style15' - it means that
you're using DW's properties bar to create a whole bunch of separate
little CSS styles. They're tough to keep track of, and you wind up
with a bunch of them scattered through your code. Think through your
CSS, create your classes in your (external) stylesheet, and apply them
as necessary.
You used Fireworks to export HTML code. Fireworks creates notoriously
fragile HTML. That's the reason for the line of spacer .gifs.
They're totally unnecessary and bloat your code with no need. That's
also the reason for having a rowspan and a colspan on the td. I
wouldn't even attempt to try to sort something like that out.
You used Flash for navigation. That doesn't work for people using
assistive devices to visit your site, it doesn't work for people who
don't have Flash, and probably more important, it doesn't work for
search engines.
You use an iframe on the page. That creates an internal scrollbar.
This one is arguable, but many of us consider internal scrollbars to
be a detriment to usability.
In my opinion - your best bet? Start over. Take your fireworks comp,
slice it up, and use the slices as parts to a page you create using
one of DW's built-in sample pages.
--
Joe Makowiec
http://makowiec.net/
Email:
http://makowiec.net/contact.php