Gå til innhold

Konstruktiv kritikk ønskes


Anbefalte innlegg

Hepp.

 

Jeg vil gjerne ha litt konstruktiv kritikk på den nye siden jeg har fått opp, etter at det gamle domenet mitt igjen ble tilgjengelig. Det jeg på forhånd kan si er at menyen nok ikke er den letteste å finne, uten at man svirrer musen over bildet. Der kan det nok helt sikkert forbedres. I IE benytter jeg vanlige GIF-filer, pga at IE ikke støtter delvis transparente PNG'er. Så lenge den ser grei ut med tanke på bredder og padding'er i IE, får det være godt nok.

 

Koden skal validere XHTML 1.1 Strict, men finner dere noen ting som har stort forbedringspotensiale, rop ut.

 

www.auduns.no :thumbup:

Lenke til kommentar
Videoannonse
Annonse

Likte ikke menyen

Hadde vært greit å kunne se hva som var hvor uten å måtte lete seg frem.

Likte ikke bakgrunden heller, den ble litt kornete.

Siden ble av en eller annen grund litt ubehagelig å se på

Samt at du må endre de punktene domo' kom med.

 

Viste ikke at xhtml 1.1 strict fantes :\

Er det noe utvidelse ifra 1.1?

Lenke til kommentar

Flott design og fine koder. Rent og pent... Et av de bedre designene jeg har sett.

 

Et lite plukk:

alt=""

. Enkelt å lure en validator, men det er fortsatt ikke riktig. alt skal være spesifisert...

 

Edit: "http://www.auduns.no/index.php?p=skriveri" er ikke i samme posisjon som resten av siden.

 

En ting til: Bildegalleriet burde ha vært i samme layout som de andre sidene.

Endret av anth
Lenke til kommentar
Hvorfor har du ikke menyen som en <ul>?

Og h1 skal vel brukes bare én gang, du kan jo bruke h2 på den andre

overskriftene?.

 

Ellers ser det jo greit ut, menyen var ikke så brukbar, men men.

Hvordan kan jeg ha menyen som en ul ? Forklar. Menyen består av en

tabell med ett bakgrunnsbilde, og celler med a:hover-bakgrunnsbilder som

er transparente PNG'er.

 

Angående h1 er jeg forsåvidt enig, men siden bloggen genereres av

CuteNews vet jeg ikke helt hvordan jeg kan få den nyeste til å bruke h1,

mens resten bruker h2.

Viste ikke at xhtml 1.1 strict fantes :\

Er det noe utvidelse ifra 1.1?

Er vel heller noe strengere validering vil jeg tro, uten at jeg vet 100%

sikkert.

Enkelt å lure en validator, men det er fortsatt ikke riktig. alt

skal være spesifisert...

Det har du helt rett i, det fikses omgående :thumbup:

Mine synspunkter:

- Bakgrunnen passer ikke inn i det hele tatt

- Brukte litt tid på å finne menyen, den bør vises hele tiden etter min

mening. Ved :hover ser den ganske bra ut

- Menyen burde vært satt opp av en liste og ikke av tables

Ser flere som reagerer på bakgrunnen. Jeg får se om jeg ikke finner noe

annet. Vet ikke helt hva jeg skal gjøre med menyen, men det er mulig

link-navnene kan vises svakt ved a.

 

Og igjen, forklar meg hvordan jeg kan sette opp dette med lister, og

ikke i en tabell.

 

Grunnen til at galleriet ikke er i samme stil som resten av designet, er

vel rett og slett at jeg syns det er mer funksjonabelt slik det er nå.

 

Takk for all tilbakemelding!

Lenke til kommentar

Hvordan kan jeg ha menyen som en ul ? Forklar. Menyen består av en

tabell med ett bakgrunnsbilde, og celler med a:hover-bakgrunnsbilder som

er transparente PNG'er.

Bytt ut

<table cellpadding="0" cellspacing="0">
<tr>
<td class="meny1"><a href="index.php"></a></td>
<td class="meny2"><a href="index.php?p=audun"></a></td>
<td class="meny3"><a href="gallery"></a></td>
<td class="meny4"><a href="index.php?p=linux"></a></td>
<td class="meny5"><a href="index.php?p=skriveri"></a></td>
<td class="meny6"><a href="index.php?p=linker"></a></td>
</tr>
</table>

med

<ul>
<li><a href="index.php" class="meny1">Blogg</a></li>
<li><a href="index.php?p=audun" class="meny2">Audun</a></li>
<li><a href="gallery" class="meny3">Blogg</a>Galleri</li>
<li><a href="index.php?p=linux" class="meny4">Linux</a></li>
<li><a href="index.php?p=skriveri" class="meny5">Skriveri</a></li>
<li><a href="index.php?p=linker" class="meny6">Linker</a></li>
</ul>

Angående h1 er jeg forsåvidt enig, men siden bloggen genereres av

CuteNews vet jeg ikke helt hvordan jeg kan få den nyeste til å bruke h1,

mens resten bruker h2.

Så lenge du ikke bruker H1 til å beskrive sidenavnet, er det egentlig helt OK å bruke H*-elementene som du gjør. :)

Viste ikke at xhtml 1.1 strict fantes :\

Er det noe utvidelse ifra 1.1?

Er vel heller noe strengere validering vil jeg tro, uten at jeg vet 100%

sikkert.

Tingen er vel helst at det ikke eksisterer noe som heter XHTML 1.1 Strict :)

Ressursnavnet er som du sier XHTML 1.1 Strict. Problemet er bare at den ikke eksisterer

Hvis du videre ser på ressursID-en til DTDen står det http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd, og den peker til XHTML 1.0's DTD. :)

 

DTD-en du vil ha er

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" 
  "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">

 

Jeg ser at du prøver å sende MIME-typen application/xhtml+xml, men det fungerer dårlig på windows-platformen når META-elementet brukes.

 

Legg heller til en referanse til dette scriptet HELT i toppen av dokumentet ditt (over DOCTYPE).

Lagre filen som xml.php, og inkluder via <?php include_once('xml.php'); ?>.

Da kan du bare fjerne META-elementet som deklarerer MIME-typen. :)

Endret av SirIce
Lenke til kommentar

- likte ikke menyen helt, det er fordel å se hva man trykker på før man har musa over knappen.

- jeg føler banneren ikke passet helt. Altså, er det bilde av en massegrav?

- Synes det er kult når bakgrunnen "står fast" og siden "sklir" over den, dette så bra ut, men du kunne vel kanskje valgt en annen bakgrunn.

 

:)

Lenke til kommentar

Jeg vil si at menyen var original, og krever litt tankeprosess før man finner den. Dette er en av de sidene som ikke er mainstream. Behold menyen, audun. Det er alltid artig med nytenking. Agående bakgrunnen så passer den inn, men bruk mindre komprisjon.

Endret av anth
Lenke til kommentar

Opprett en konto eller logg inn for å kommentere

Du må være et medlem for å kunne skrive en kommentar

Opprett konto

Det er enkelt å melde seg inn for å starte en ny konto!

Start en konto

Logg inn

Har du allerede en konto? Logg inn her.

Logg inn nå
×
×
  • Opprett ny...