Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

  • Congratulations strongm on being selected by the Tek-Tips community for having the most helpful posts in the forums last week. Way to Go!

Feedback and advice needed 2

Status
Not open for further replies.

tysu

Programmer
Mar 30, 2002
14
CA
I offer free (linkware) graphics on my humble site and have just made a new Web Set that viewers will be able to upload. It's not exactly working right because I think my coding/table-ing is wrong, or that I should be using CSS or something else. I made it in Photoshop and then cut the pieces separately, which is how people will download it, piece by piece. Is there a fairly basic way to get the Title ("it's my site" - which I have to re-do anyway due to colour bleed) layered over the picture? And I can't get rid of the gap between the column on left and the photo.
Feedback on how it looks now and any kind suggestions appreciated.
It's at
(no pop-ups)

Thanks.

tysu
 
Hi first things first. unfortunatly there are a lot of mistakes in the code!
here is a cleaner version:

<!DOCTYPE html PUBLIC &quot;-//W3C//DTD HTML 4.01 Transitional//EN&quot;>
<html>
<head>
<title>
:: studiozuzu :: It's My Grey web set ::
</title>
<style type=&quot;text/css&quot;>
<!--
body (background: #ffffff; color: #000000;}
a:link { color: navy }
a:visited { color: maroon }
a:active { color: white }
a:hover { color: maroon }

.intro {font-size:9pt}
.copyright {font-size:8pt}
-->
</style>
</head>
<body bgcolor=&quot;white&quot; link=&quot;navy&quot; vlink=&quot;maroon&quot; alink=&quot;white&quot;>
<center>
<!-- main table -->
<table summary=&quot;&quot;>
<tr>
<td>
<!-- inner table -->
<table width=&quot;600&quot; cellspacing=&quot;0&quot; cellpadding=&quot;0&quot;
border=&quot;0&quot; summary=&quot;&quot;>
<tr>
<td bgcolor=&quot;#CCCCCC&quot;>
<br>
<br>
<br>
<img alt=&quot;&quot; src=
&quot; <br>
<br>
<a href=&quot; src=
&quot; border=&quot;0&quot; alt=&quot;Pictures from a Life&quot;></a><br>
<br>
<a href=&quot; src=
&quot; border=&quot;0&quot; alt=&quot;What's doing...&quot;></a><br>
<br>
<a href=&quot; src=
&quot; border=&quot;0&quot; alt=&quot;Connections&quot;></a><br>
<br>
<a href=&quot; src=
&quot; border=&quot;0&quot; alt=&quot;Mail me&quot;></a>
</td>
<td>
<p class=&quot;intro&quot;>
Ramblings of a teenaged mind...<br>
as if you can't get enough of that.
</p>
<p class=&quot;intro&quot;>
This site is here because I'm not near<br>
my friends right now and we want to know
</p>
<img alt=&quot;&quot; src=
&quot; </td>
</tr>
</table>
<!-- end inner table -->
<p class=&quot;copyright&quot;>
photo: <a href=
&quot; </p>
<p class=&quot;copyright&quot;>
all &copy; 2002<br>
<a href=
&quot; </p>
<p class=&quot;copyright&quot;>
Nothing here may be resold in any form<br>
Credit for this set must be given on same page
</p>
</td>
</tr>
</table>
<!-- end main table -->
</center>
</body>
</html>



I removed the font tags and changes to CSS. its easier and cleaner. I also corrected mistakes in table syntax etc etc

The gap was because you had set a table padding of 5, it is now 0.

To over lap the image there are a number of ways. You could set the photo as a back ground image and then put the image you want to overlap into the <td></td> tags.

It can also be done with CSS. I'll try it and let you know.


É
:: ::
 
Oh my, gosh, cian, I never expected this!
Thank you. I'll go try it now and tell you how it works tomorrow.
Again - bless you!!

tysu
 
Hi,

Ok, this is not perfect but the idea works:

I have taken the title image and placed it over the main picture:

<div id=&quot;overlap&quot;>
<img alt=&quot;alternate text&quot; src=&quot;gsTitle.jpg&quot;>
</div>

Then defined the styles for that:
#overlap {position:relative;left:-20;top:100px;z-index:5;}
you can change thw -20 and 100 to change the position of the image.

full code::

<!DOCTYPE html PUBLIC &quot;-//W3C//DTD HTML 4.01 Transitional//EN&quot;>
<html>
<head>
<title>
:: studiozuzu :: It's My Grey web set ::
</title>
<style type=&quot;text/css&quot;>
<!--
body (background: #ffffff; color: #000000;}
a:link { color: navy }
a:visited { color: maroon }
a:active { color: white }
a:hover { color: maroon }

.intro {font-size:9pt}
.copyright {font-size:8pt}
#overlap {position:relative;left:-20;top:100px;z-index:5;}
-->
</style>
</head>
<body bgcolor=&quot;white&quot; link=&quot;navy&quot; vlink=&quot;maroon&quot; alink=&quot;white&quot;>
<center>
<!-- main table -->
<table summary=&quot;&quot;>
<tr>
<td>
<!-- inner table -->
<table width=&quot;600&quot; cellspacing=&quot;0&quot; cellpadding=&quot;0&quot;
border=&quot;0&quot; summary=&quot;&quot;>
<tr>
<td bgcolor=&quot;#CCCCCC&quot;>
<br>
<br>
<a href=&quot; src=
&quot; border=&quot;0&quot; alt=&quot;Pictures from a Life&quot;></a><br>
<br>
<a href=&quot; src=
&quot; border=&quot;0&quot; alt=&quot;What's doing...&quot;></a><br>
<br>
<a href=&quot; src=
&quot; border=&quot;0&quot; alt=&quot;Connections&quot;></a><br>
<br>
<a href=&quot; src=
&quot; border=&quot;0&quot; alt=&quot;Mail me&quot;></a>
</td>
<td>
<p class=&quot;intro&quot;>
Ramblings of a teenaged mind...<br>
as if you can't get enough of that.
</p>
<p class=&quot;intro&quot;>
This site is here because I'm not near<br>
my friends right now and we want to know
</p>
<div id=&quot;overlap&quot;>
<img alt=&quot;&quot; src=
&quot; </div>
<div>
<img alt=&quot;&quot; src=
&quot; </div>
</td>
</tr>
</table>
<!-- end inner table -->
<p class=&quot;copyright&quot;>
photo: <a href=
&quot; </p>
<p class=&quot;copyright&quot;>
all &copy; 2002<br>
<a href=
&quot; </p>
<p class=&quot;copyright&quot;>
Nothing here may be resold in any form<br>
Credit for this set must be given on same page
</p>
</td>
</tr>
</table>
<!-- end main table -->
</center>
</body>
</html>




É
:: ::
 
Anytime,

one thing I forgot to do:
change
<body bgcolor=&quot;white&quot; link=&quot;navy&quot; vlink=&quot;maroon&quot; alink=&quot;white&quot;>

to simply <body> because these are already defined in the styles

a:link { color: navy }
a:visited { color: maroon }
a:active { color: white }
a:hover { color: maroon }



É
:: ::
 
Thanks, again. Going to try the new one now.
I kept the original page-in-question up in case anyone wants to see it, with a link to the coding you did at the bottom of that original page. OK with you? This page is not active yet.
(I changed some tiny things - dropping some extraneous &quot;;&quot;, changing the font size and adding a little space in the &quot;Ramblings...&quot; portion.)

Here's a new dilemma : If I do use your modifications (corrections!)can I still claim this as &quot;my&quot; work? I dunno...?


tysu
 
Well I guess I'll have to e-mail you,then.
Ha-ha. (don't even know where to get the smileys here!)

I tried the over-lay version and it changed the set-up considerably, so I'm sticking with the first. I'll re-touch the photo to get rid of what shouldn't be there on the left and make another title graphic.
Tomorrow.

You're great, cian.
Erin go braless, and all that...

tysu



 
Hi mate,

If you want to use absolute positioning for all the images then you could simply slice the page up in ImageReady and then export it using CSS.

I was actually suprised just how well that worked.

Hope this helps Wullie

sales@freshlookdesign.co.uk

The pessimist complains about the wind. The optimist expects it to change. The leader adjusts the sails. - John Maxwell
 
I've never seen Erin braless, wouldn't mind though! :)

The layout changes cos your widest image was the title one, you could specify the width of that column or put in a transparent image to keep the shape.

For your information, under the textbox where you make type your message you'll find a link Emoticons/smileys!


slán
:: ::
 
When I said I cut the pieces I meant that I made this in Photoshop, then selected and extracted each image to its own new file for reassembling on the Web Set page.
I'm a bleeping coward around real slicing, Image Ready and even CSS!!
My skills are so basic it's not funny sometimes. Know I've got a lot of catching up to do.

tysu
 
Hi tysu,

It looks nice and neat but unfortunatly you still have a number of errors. It would be important to correct this if you want the page to look the same in all browsers.

1. missing doctype
<!DOCTYPE html PUBLIC &quot;-//W3C//DTD HTML 4.01 Transitional//EN&quot;>

2. two body tags.

3. incorrect table. your first table is simply <table> you need to give this table rows and cells! the minimun table should be
<table summary=&quot;&quot; title=&quot;&quot;>
<tr>
<td>
</td>
</tr>
</table>

4. The <center> and <font> tags are basically obselete. Although still used by some it would be best to get rid of these and upgrade to CSS.

5. missing alt=&quot;&quot; from some images. It is very important to have alternate text for every image.

6. attribute quotes. width=&quot;&quot; alt=&quot;&quot; cellspacing=&quot;&quot; cellpadding=&quot;&quot; border=&quot;&quot;

7. there is no valign=&quot;&quot; in <table> put it in the cells.

CSS is easy to learn and will, in the end, save you a truck load of work on your sites.

An editor such as will help you write clean code fast and also spot mistakes.

Hope that helps! :)


:: ::
 
Status
Not open for further replies.

Part and Inventory Search

Sponsor

Back
Top