Konstruktiv Kritik

Tråden skapades och har fått 3 svar. Det senaste inlägget skrevs .
1

Hej!
Jag har som skoluppgift att göra en personlig portfolio på internet och har kommit typ halvvägs nu. Det jag behöver nu är synpunkter på min kod och min sida, så här följer båda.

Sidan finns på jokke.

Såhär ser CSS:en ut;

@charset "utf-8";
/* CSS Document */

#importantlink {
color:#333333;
text-decoration:underline;
}

#rosa {
    position:absolute;
    left: 587px;
    top: 257px;
}

#youtube-lank{
font:Arial, Helvetica, sans-serif medium;
color:#333333;
text-decoration:underline;
}

a {
text-decoration:none;
color:#000000;
}

body {
background-image:url(bilder/layout/SpoonGraphics_Pattern.gif);
}

hr {
width:640px;
color:#666666;
background-color:#666666;
border: 1px dashed #666666;
border-style:dashed;
}

#main {
position:absolute;
border:5px #000000 solid;
margin-top:80px;
margin-left:100px;
width:650px;
height:400px;
background:#FFFFFF;
z-index:1;
}

#menu {
width:310px;
height:70px;
float:left;
padding-right:10px;
z-index:1;
}

#low {
padding-left:10px;
z-index:1;
}

#high {
z-index:1;
float:right;
padding-top:10px;
padding-right:10px;
width:320px;
height:60px;
text-align:right;
font: "Times New Roman", Times, serif;
font-style: italic;
}

#rubrik {
font: Verdana, Arial, Helvetica, sans-serif large bolder;
}

.style1 {
color: #333333;
font-size: 16px;
}

.style2 {
color:#000000;
font-weight:bold;
font-size: 24px;
}

.pd_menu_01 {
float:left;
padding:0;
margin:0;
color: #FFFFFF;
background: #333333;
width:226px;
border:solid 1px #000000;
clear:both;
} /*Color navigation bar normal mode*/

.pd_menu_01  a, .pd_menu_01 a:visited {
font-family:Arial, Helvetica, sans-serif;
font-style:normal;
font-weight:bold;
font-size:12px;
color: #FFFFFF;
background-color: #333333;
text-decoration: none;
}

.pd_menu_01 ul {
list-style-type:none;
padding:0;
margin:0;
}

.pd_menu_01 ul li {
float:left;
position:relative;
z-index:auto !important /*Non-IE6*/;
z-index:1000 /*IE6*/;
border-right:solid 1px #000000;
}

.pd_menu_01 ul li a {
color: #FFFFFF;
background: #333333;
float:none !important /*Non-IE6*/;
float:left /*IE-6*/;
display:block;
height:30px;
line-height:30px;
padding:0 10px 0 10px;
text-decoration:none;
}

.pd_menu_01 ul li ul {
display:none;
border:none;
color:#FFFFFF;
background: #333333;
}

.pd_menu_01 ul li:hover a {
background-color:#FFFFFF;
text-decoration:none;
color:#000000;
} /*Color main cells hovering mode*/

.pd_menu_01 ul li:hover ul {
display:block;
position:absolute;
z-index:999;
top:29px;
margin-top:1px;
left:0;
}

.pd_menu_01 ul li:hover ul li a {
display:block;
width:12em;
height:auto;
line-height:1.3em;
margin-left:-1px;
padding:5px 10px 5px 10px;
border-left:solid 1px #000000;
border-bottom: solid 1px #000000;
background-color:#333333;
color:#FFFFFF;
} /*Color subcells normal mode*/

.pd_menu_01 ul li:hover ul li a:hover {
background-color:#FFFFFF;
text-decoration:none;
color:#000000;
} /*Color subcells hovering mode*/

.pd_menu_01 table {
position:absolute;
top:0;
left:0;
border-collapse:collapse;
color: #FFFFFF;
background: #333333;
}

.pd_menu_01 ul li a:hover {
background-color:#FFFFFF;
text-decoration:none;
color:#000000;
} /*Color main cells hovering mode*/

.pd_menu_01 ul li a:hover ul {
display:block;
width:12em;
position:absolute;
z-index:999;
top:29px;
left:0;
}

.pd_menu_01 ul li a:hover ul li a {
display:block;
width:12em;
height:1px;
line-height:1.3em;
padding:4px 16px 4px 16px;
border-left:solid 1px #000000;
border-bottom: solid 1px #000000;
background-color:#333333;
color:#FFFFFF;
} /*Color subcells normal mode*/

.pd_menu_01 ul li a:hover ul li a:hover {
background-color:#FFFFFF;
text-decoration:none;
color:#000000;
} /*Color subcells hovering mode*/

Tack på förhand!

  • Medlem
  • Göteborg
  • 2008-05-25 17:42

Ser ju fint ut! Jag skulle dock och förminska bakgrundsbilden, annars så tar det stora mönstret bort fokusen ifrån själva innehållet. Annars tycker jag att det ser bra och smart uppbyggt. Tummen upp

Tack för tipset! Det blir nog till att göra något sådant.

  • Medlem
  • Stockholm
  • 2008-06-05 23:06

En synpunkt jag har är att serif-typsnitt är svårlästa på skärm, särskilt i små grader. På en gammal "tjockskärm" kan det till och med vara oläsligt.

Skulle rekommendera Verdana, Helvetica, Arial varianter i stället.

Annars är det bra!

Janne

1
Bevaka tråden