site coding review!?

B

Bergamot

catalyst said:
Hey!!! Which one to use, which one do you have arguments that it's better to
use for a list?

A list is marked up with <ol> <ul> or <dl>. A series of <a> elements in
a div is not a list said:
i find <ul><li></li></ul> better because of
types which i can apply. I am aware that's amateurish

Not at all. Use the most appropriate mark up for the particular content.
I
suppose you could do the same with the class="nav", and bring on some .gif
pictures instead circles or discs?

Look up CSS list-style properties.

About the code below, I have 2 words: Icky poo! Start getting in some
better coding habits and drop all that excess junk.
<div id="kvartovi">
<span id="kvartoviposition"><a href="mapa_cetvrti.html"><font
color="blue" size="2"><strong> Drugi kvartovi</strong></font></a></span>

What is this text really - a heading for the list? If so, use heading
markup (h1-h6, as applicable) instead of span. And stop using <font>,
please!

<div id="kvartovi">
<h2><a href="mapa_cetvrti.html">Drugi kvartovi</a></h2>

and the CSS:
#kvartovi {font-size:smaller} /* applies to all elements in the div */
#kvartovi h2 a {color:blue} /* <a> inside <h2> inside the div */

It is likely, however, the font-size and color properties (plus
background colors) should be set globally at higher level, so they apply
to elements other than just those in this particular div.
<div style="margin-top: -10px;">

Why do you think you need this? Is it because there is a 1-line gap
between your <span> and the first list item? If so, it's caused by the
default margin-top on the <ul> element. So you don't need that div at all.

#kvartovi ul {margin-top:0}

BTW, inline styles will be a nightmare to maintain, so best to not use
them at all.
<ul type="disc" >

"disc" is the default for the first <ul> level so you don't need to set
it at all. If you did want to change it, use CSS instead

#kvartovi ul {margin-top:0; list-style-type:square}
<font size="2" color="#7B68EE">

No more <font>!
#kvartovi ul a {color:#7B68EE}

You should also set a background color to go with the foreground color.
<li><a href="http://www.Dubrava.hr">Dubrava</a></li>
<li><a href="http://www.Maksimir.hr">Maksimir</a></li>
<li><a href="http://www.novi-zagreb.info">Novi Zagreb</a></li>
<li><a href="http://www.sesvete-online.info/">Sesvete</a></li>
</font></ul>
</div></div>
</td>

After you get a little more practice, you can work on getting rid of the
layout tables. ;)
 

Ask a Question

Want to reply to this thread or ask your own question?

You'll need to choose a username for the site, which only take a couple of moments. After that, you can post your question and our members will help you out.

Ask a Question

Members online

No members online now.

Forum statistics

Threads
473,769
Messages
2,569,582
Members
45,066
Latest member
VytoKetoReviews

Latest Threads

Top