Jump to content
Killersites Forums

Link dont work


Recommended Posts

This is the code for that section.

<div class="box">

<div class="padding1">

<h2 class="p3">Important Links</h2>

<ul class="tabs">

<li><a href="http://www.canadapost.ca/cpotools/apps/track/business/findByTrackNumber?execution=e2s1">Canada Post</a></li>

<li><a href="https://www.usps.com/">USPS</a></li>

<li><a href="http://www.metric-conversions.org/weight/kilograms-to-pounds.htm">Weight Conversion</a></li>






It appears to be ok?

Link to comment
Share on other sites

Check all those divisions you have surrounding this list - I randomly deleted several of them, and the links started to work. I did not check into this in detail, I'd have to look through the CSS for each of them.


There really should not be any reason you need so many division here - I'd start over and keep it simple. Use only a division if one is needed. You can apply ID's or classes to a ul tag and define it that way, and may not need a single division around it at all, but I don't see any single reason where you actually need all the divisions you are using on the entire site. And when you do use divisions, give them meaningful class or ID names - grid 4 alpha will likely not mean a whole lot to you 3 months later.

Link to comment
Share on other sites

You'd learn a lot more recreating your own - and it's always a bit PITA to try and figre out what someone else has done, especially when it's as cluttered at this template.


Nothing wrong with templates - if there's nothing wrong with the one you're using rolleyes.gif

Link to comment
Share on other sites

I cannot tell you which ones to delete without seeing what they all do, I was just deleting blindly using the 'Edit HTML' in my FX Webdeveloper's bar.


I would seriously consider recreating this page from scratch - the layout is not that difficult, and you'll learn a lot more. The way this is coded right now, you almost have more divisions than you have content.


Example - a division class for padding - makes no practical sense. Just add the padding to the 'mother'division's CSS.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.


  • Create New...