Vurdering af redesign

Der er da ikke så meget at komme med...
Af et design af den art er du da sluppet godt afsted med det...

Hvis du som sådan mener at man skal kigge css'en igennem osv, så tror jeg det ville være rart for folk hvis du linkede den direkte :)... hvis det altså er det du ønsker når du skriver: "Kan jeg gøre noget anderledes i css?"

Oki, det kommer her så...



/* CSS Document */
html, body {
    margin: 0;
    padding: 0;
    }
   
body {
    font-family: Verdana, Arial, Helvetica, Sans-serif;
    font-size: 11px;
    color: #000000;
    background: #393A3D;
    background-image: url(../images/billeder/baggrund2.jpg);
    background-repeat: repeat;
    background-attachment: fixed;
}   
   
   
   
.hidden {
    display: none;
}   
   

#page-container {
                width: 760px;
                margin: auto;
                background-color: #E0FCEB;
                }
               
#main-nav {
    background: #db0048;
    height: 50px;
   
}

#main-nav a {
    color: #E0FCEB;
    text-decoration: none;
    font-weight: bold;
}

#main-nav a:hover {
    color: #393A3D;
    font-weight: bold;
}


#main-nav dt { float: left;
                padding: 10px;
                color: #E0FCEB;
               
            }


       


#header {
    height: 150px;
    background: #5D9E9B
        url(../images/headers/banner1.jpg);
       
}

#sidebar-a {
    float: right;
    width: 280px;       
    background: #C8DBBF;
    line-height: 18px;
    clear: right;
}

#sidebar-a .padding {
    padding: 0px 25px 25px 25px;
}

#content {
    margin-right: 280px;       
    background: #E0FCEB;
    line-height: 18px;
}

#content .padding {
    padding: 5px 25px 25px 25px;
}


#footer {
    clear: both;   
    background: #DB0048;
    height: 35px;
    font-family: Tahoma, Arial, Helvetica, Sans-serif;
    font-size: 12px;
    color: #E0FCEB;
    border-top: 3px solid #393A3D;
    padding: 13px 25px;
    line-height: 18px;

}
#footer a {
    color: #E0FCEB;
    text-decoration: none;
}

#footer a:hover {
    color: #393A3D;
}

#footer #altnav {
    width: 350px;
    float: right;
    text-align: right;
}




h1 {
    margin: 0;
    padding: 0;
    float: right;
    margin-top: 55px;
    padding-right: 3px;
    }

#content h2 {
    margin: 0;
    padding: 0px;
}

#content p {
    margin: 0;
    padding: 0;
}

#billede1 {
    float:left;
    width: auto;
    padding-right: 25px;

}

h2  {
    font-size: 14px;
    color: #DB0048;
   
    }
   
a:link {
        color: #DB0048;
        text-decoration: none;
        font-weight: bold;
        }
       
a:visited    {   
            color: #db0048;
            text-decoration: none;
            font-weight: bold;
            }       
           
a:hover        {
            color: #393A3D;
            text-decoration: none;
            font-weight: bold;
            }
           
#billede2 {
    float:left;
    width: auto;
    padding-right: 25px;

}



* {
   
}
.highslide {
    cursor: url(galleri/highslide/graphics/zoomin.cur), pointer;
    outline: none;
}
.highslide-active-anchor img {
    visibility: hidden;
}
.highslide img {
    border: 2px solid gray;
}
.highslide:hover img {
    border: 2px solid white;
}
.highslide-wrapper {
    background: white;
}
.highslide-image {
    border: 10px solid white;
}
.highslide-image-blur {
}
.highslide-caption {
    display: none;
    border: 5px solid white;
    border-top: none;
    padding: 5px;
    background-color: white;
}
.highslide-loading {
    display: block;
    color: black;
    font-size: 8pt;
    font-family: sans-serif;
    font-weight: bold;
    text-decoration: none;
    padding: 2px;
    border: 1px solid black;
    background-color: white;
   
    padding-left: 22px;
    background-image: url(galleri/highslide/graphics/loader.white.gif);
    background-repeat: no-repeat;
    background-position: 3px 1px;
}
a.highslide-credits,
a.highslide-credits i {
    padding: 2px;
    color: silver;
    text-decoration: none;
    font-size: 10px;
}
a.highslide-credits:hover,
a.highslide-credits:hover i {
    color: white;
    background-color: gray;
}

a.highslide-full-expand {
    background: url(galleri/highslide/graphics/fullexpand.gif) no-repeat;
    display: block;
    margin: 0 10px 10px 0;
    width: 34px;
    height: 34px;
}

/* These must always be last */
.highslide-display-block {
    display: block;
}
.highslide-display-none {
    display: none;
}

Da jeg sagde linke mente jeg like: http://mopsviggo.dk/css/master.css

:P... længe siden jeg har arbejdet med skittet desuden så hvis det er valideret osv, tja... så er det kun folk der virkelig arbejder meget med det... :)

Og et hurtigt kig ned over revealer ikke noget jeg ville ændre...

Oki ha ha, nå men nu er koden herinde...

Ja det validerer det hele:)

Ingen andre der har lyst til at kommentere?

hmm ved ikke? :) det ser jo meget simpelt ud, og faktisk fin design og jeg kan godt lide teksten der står mopsviggo.dk i den grå ramme! :) men jeg synes body baggrunden er lidt kedelig, du har sikkert brugt #ffffff og #aaaaaa eller noget i den stil, prøv istedet med #ffffff og #111111 eller #222222 ! :) altså den farve som er baggrund for "mopsviggo.dk" :) så tror jeg det ser meget bedre ud! :) men det er selvfølgelig min mening! :b

Det ser da meget fornuftigt ud.

Jeg har dog et lille tip, som jeg synes, gør det mere overskueligt, når jeg arbejder med stier. F.eks. i linjen: background-image: url(../images/billeder/baggrund2.jpg);
Der har du måtte starte stien med ../ for at den skal gå en mappe ud og ind i mappen images, men hvis du bare starter stien med / dvs. /images/billeder/baggrund2.jpg, så vil den starte ude i roden af serveren, altså hovedmappen, og dernæst gå ind i images.

Det synes jeg er noget lettere at arbejde med, end en relativ sti, som du bruger.

En meget lille detalje, det ved jeg godt, men jeg har ikke rigtig andet at kommentere i din kode. =)

Takker Kurdo og w13 for jeres kommentarer.

Kurdo, er selv oki tilfreds med  farven på banneret.

w13, jeg vil se på det med de stier :)

Hej Viggosmor.
Generelt et nydeligt design.
Det der springer mig i øjnene ved første øjekast er baggrunden på siden, den er meget forvirrende og fremtrædende. Du kunne eventuelt lave den hvide farve i en grå nuance, der ligger op af den anden du bruger på baggrunden, eksempelvis #8F8F8F eller #7F7F7F. Så er du fri for at baggrunden stjæler opmærksomheden fra det primære ved hjemmesiden, som vel er indholdet ;)

Brug unordered/ordered-lists til menuen og list-information, istedet for definitions-lists.

Og tekststørrelser bør angives i EM og ikke PX : )

w13 læg et svar :)