Avatar billede netspinner Nybegynder
24. maj 2005 - 11:33 Der er 4 kommentarer

Siden og CSS i orden? Hjælp!

Hej,
Jeg er ved at lægge sidste hånd på en side jeg skal bruge til et mindre eksamensprojekt. Imidlertid er jeg blevet forvirret over min egen kode og kunne godt tænke mig at få andre øjne til at give mig feedback - på både HTML- og CSS-koden. Designet (farver mv.) efterlyser jeg ikke kommentarer til, det er på plads.

Desuden kunne jeg godt tænke mig at have en footer i form af en tabel nederst på siden, hvor der bare lige bliver opgivet en e-mail-adresse og et navn etc. Det er dog vigtigt at det hele forbliver i 800x600.

Derudover ser det ud til at der er en frilinje nederst (hvis man marker alt i browseren (med CTRL+A) kan det ses), hvilket der ikke skal være... kan bare ikke finde det i HTML-koden :-(

Der udloves gode point, som kan deles, hvis der er flere der svarer fornuftigt på det hele :-)

Filerne skal gemmes som index.htm og style.css
Det er vigtigt at hente sidens pt. eneste billede ned, som findes på:
http://www.upitfree.dk/upload/files/logo.jpg[2]

Håber ikke det var for meget - ellers spørg...


------------------ index.htm ------------------


<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
  "http://www.w3.org/TR/html4/loose.dtd">

<html>
<head>
<title>&Oslash;koG&aring;rden</title>

<link href="style.css" rel="stylesheet" type="text/css">


<script language="JavaScript" type="text/javascript">
<!--
window.onload = function() {
  var divs=document.getElementsByTagName("div");
    for (i=0;i<divs.length;i++){
      if (divs[i].className=="top") {
          divs[i].onmouseover=function() {
            this.firstChild.nextSibling.className="on";
            }
          divs[i].onmouseout=function() {
            this.firstChild.nextSibling.className="off";
            }
        }
    }   
}
//-->
</script>
</head>
<body>
<center>
<div id="title_tbl">
        <img src="grafik/logo.jpg" alt="&Oslash;koG&aring;rden - p&aring

nettet" class="billede">
<h1 id=t1>&Oslash;koG&aring;rden</h1>
<p id=t2>... din &oslash;kologiske partner!</p>
</div>
<div id="all">
<table> <!-- Menu indeholdt i tabel -->
<td>
<center>
<div id="menu">
<div class="top">
  Om g&aringrden
  <div class="skjult">
    <div class="menup" onmouseover="style.backgroundColor='lightgreen';"

onmouseout="style.backgroundColor='rgb(240, 240, 240)';"><a

href="historie.htm">Historie</a></div>
        <div class="menup"

onmouseover="style.backgroundColor='lightgreen';"

onmouseout="style.backgroundColor='rgb(240, 240, 240)';"><a

href="medarbejdere.htm" >Medarbejdere</a></div>
        <div class="menup"

onmouseover="style.backgroundColor='lightgreen';"

onmouseout="style.backgroundColor='rgb(240, 240, 240)';"><a href="job.htm">Job hos

os?</a></div>
  </div>
</div>

<div class="top">
  Produkter
  <div class="skjult">
    <div class="menup" onmouseover="style.backgroundColor='lightgreen';"

onmouseout="style.backgroundColor='rgb(240, 240, 240)';"><a

href="produkter.htm">Vores produkter</a></div>
</div>
</div>

<div class="top">
  V&aelig;rd at vide
  <div class="skjult">
    <div class="menup" onmouseover="style.backgroundColor='lightgreen';"

onmouseout="style.backgroundColor='rgb(240, 240, 240)';"><a href="oekologi.htm"

>L&aelig;s om &oslash;kologi</a></div>
        <div class="menup"

onmouseover="style.backgroundColor='lightgreen';"

onmouseout="style.backgroundColor='rgb(240, 240, 240)';"><a href="kontakt.htm"

>Kontakt os</a></div>
  </div>
</div>
</td>
</table> <!-- Slut tabel for menu -->

<!-- Indholdet -->
<table width="430px">

<div id="txt">
  <b>Velkommen hos &Oslash;koG&aring;den!</b><br><p>
  Vi er et selvst&aelig;ndigt &oslash;kologisk landbrug, der dyrker vores egne

produkter uafh&aelig;ngigt af andre. Hos <i>&Oslash;koG&aringrden</i> den er du

altid sikker p&aring at f&aring de bedste &oslash;kologiske r&aringvarer mellem

h&aelig;nderne.
  <i>&Oslash;koG&aringrden</i> befinder sig i det smukke midtfynske landskab, hvor

jorden er god til dyrkning.<p>
    For at l&aelig;se mere om <i>&Oslash;koG&aringrden</i> og dens mange

facetter, da anvend menunavigationen foroven.</p>
</div>
</table>
</center>
</body>
</html>



----------------- style.css -------------------

#t1 {
    FONT: bold italic 4em "Times New Roman", serif; COLOR: #99cccc; position: relative; TOP: -0.15em;
}

#t2 {
    FONT: 1 em arial, sans-serif; COLOR: #336666; LETTER-SPACING: 3px; POSITION: relative; TOP: -2.8em; left: 8em;
}

a {
color: black;
}

a:visited {
color: black;
}

body {
}

#menu {
position: relative;
width: 500px; height: 85px;
/*height: 100px;*/
left: 125px;
}

.menup {
padding-top: 3px;
padding-bottom: 3px;
border-top: solid green 1px; /* solid også en mulighed */
}

#title_tbl {
width: 800px;
height: 10px;

/*background-color: lightgreen;*/
}

#dato {
width: 800px;
height: 10px;
background-color: lightred;
}

#all {
position: relative;
width: 800px;
height: 510px;
background-color: rgb(240, 240, 240);
/*border: solid #000 1px;*/
border: dotted lightgreen 1px;
top: -19px;
}

#txt {
font-family:Verdana;font-size:11px;
}

.top {font-family:Arial;font-size:12px;float:left;border:1px dotted green; width: 25%;}
.skjult {display:none;}
.on {display:block;border-top:2px solid green;}
.off {display:none}

.billede {position:relative;float:left;clear:right;}
Avatar billede netspinner Nybegynder
24. maj 2005 - 11:35 #1
Linket hedder: http://www.upitfree.dk/upload/files/logo.jpg[2] (med end bracket)
Avatar billede olebole Juniormester
24. maj 2005 - 16:11 #2
<ole>

For det første er det en sjusket kodeopstilling, du har kørende. Hvorfor 'knække' linierne på tilfældige steder - og hvorfor dobbelt linieafstand nogen steder midt i koden? Udover at gøre den dårligt læselig, giver det sig bl.a. udslag i uheldige linieskift i alt-teksten på dit logo (i IE).

Dernæst er det en gang skidt kode. Hvis det er et eksamens projekt, er der vel en eller anden, der har lært dig at checke dine sider i forskellige browsere. Er det virkelig meningen, siden skal se ud på én måde i IE - og helt anderledes i alle andre?

Derudover er script ikke beregnet til at rette op på dårlige elementvalg i HTML. Det er helt misforstået at forsøge at efterligne et element med andre, der er komplet uegnede - og så forsøge at lappe og sminke fejlen med JavaScript.

Der er kun ét eneste element i HTML, der er beregnet til at oprette synkrone søjler med - og det er tabellen. Hvorfor i alverden brygger du dog en masse skidt sammen for at undgå at bruge det korrekte element? Det giver ingen mening ... kun dårlig kode.

Brug en tabel til din overordnede opdeling af siden - og fyld så den med andre elementer ... så får du skrevet en langt bedre kode  :)

/mvh
</bole>
Avatar billede netspinner Nybegynder
24. maj 2005 - 16:22 #3
Sikke en masse kommentarer. Men det er kun godt.
Faktisk er det et kursus i CSS jeg har fulgt. Kravet er at opbygge sitet i - så vidt muligt - 800x600, dvs. et fast layout. Derudover er der krav om bygge en simpel menu. Vi har kun brugt IE i undervisningen, og det ser fint ud på de to maskiner jeg har testet med. Altså heller ingen krav der. Kravet er at lave en menu, der aktiverer sig selv når musen bevæges henover - det er jo løst til fulde, så forstår ikke helt din kritik der?
Avatar billede netspinner Nybegynder
24. maj 2005 - 16:25 #4
olebole > hvis du kan give nogle konkrete eksempler på forbedringer med den listede kode, så vil jeg være taknemmelig.

(Kan lige tilføje at grunden til at det ser sjusket ud her, er fordi jeg har kopieret direkte fra notepad!)
Avatar billede Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Loading billede Opret Preview
Kategori
Vi tilbyder markedets bedste kurser inden for webudvikling

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester