1. #1

    Registered
    09/10/11
    Location
    kortrijk
    Posts
    8
    iTrader
    0
    Mentioned
    0 Post(s)
    Reputation
    0/0

    Question Portfolio online

    Hey,

    Heb gisteren mijn nieuw (klassieke) portfolio online geplaatst. Heb hem relatief snel gemaakt om te kunnen gebruiken bij stage sollicitatie.

    Hier is de link:
    Benoît Boucart - Webdesigner & Webdeveloper portfolio

    Alvast bedankt voor de feedback
    no votes   Reply With Quote Reply With Quote

  2. #2

    Registered
    14/08/10
    Location
    Diest
    Posts
    2,419
    iTrader
    1 (100%)
    Mentioned
    0 Post(s)
    Reputation
    8/16
    Mooie site!
    Opmerkingen:
    - ik dacht altijd dat het punt van nieuwe HTML5-elementen was dat je minder met IDs moet werken. Wat jij doet, houdt dus weinig steek: je werkt met aside en geeft het nog een ID? Dat is hier toch niet nodig, want je werkt maar met één aside. Hetzelfde met de header.
    - Het gebruik van lege divs met class=clear begrijp ik niet. Zet die clear: both op de daaronder liggende div en verander de margin-top in padding-top?
    - Waarom staat uw logo binnen een h1?
    - Uw toTop is vreemd opgebouwd? Waarom niet gewoon een link?
    - Ge laadt twee keer uw conditionals (in head en voor sluiten van body)

    - Kleuren zijn OK, maar de kleur van .blue vind ik net ietsje te fel.
    - Fancybox -> customize het aub. De laatste tijd zie je veel webdevs die een hele mooie website maken en daar dan een redelijk standaard fancybox/lightbox opgooien. Pas dat aan aan je website! (En niet gewoon de kleuren, maar alles!)
    - Heel uw site is in het Engels, maar als je iets van het contactform niet invult krijg ge 'vul dit veld in', en dat dan nog in een hele lelijke tekstballon die totaal niet past bij de website.
    - ik mis hoverstates op: uw logo, de toTop en uw contactform 'send' button
    - ik mis een favicon
    - de knoppen van de slider zijn lelijk (die zwart-wit). Pas die kleuren best aan aan die van je website
    - de schaduw onder uw afbeeldingen is iets té, vooral omdat het bij de slider wat minder is. Bij de slider is de schaduw ook net niet breed genoeg. De schaduw op de hoverstate van uw hoofdmenu mag ook wat meer zijn.
    - Bij uw slider, die groenachtige achtergrondkleur van 'featured' past niet, beter dat blauw nemen.
    - Ik mis de 'next' functie bij fancybox. Nu moet ik telkens fancybox sluiten als ik een nieuw project wil zien.

    Zeker geen lelijke site! (zeker niet tegenover uw allereerste HTML portfolio! Want die vind ik echt niet mooi )
    no votes   Reply With Quote Reply With Quote

  3. #3

    Registered
    09/10/11
    Location
    kortrijk
    Posts
    8
    iTrader
    0
    Mentioned
    0 Post(s)
    Reputation
    0/0
    Thanks voor de zeer uitgebreide en interessante feedback!

    - ik dacht altijd dat het punt van nieuwe HTML5-elementen was dat je minder met IDs moet werken. Wat jij doet, houdt dus weinig steek: je werkt met aside en geeft het nog een ID? Dat is hier toch niet nodig, want je werkt maar met één aside. Hetzelfde met de header.
    > Je hebt gelijk. Achteraf (nu dus ) kan ik dit wijzigen, maar die IDs heb ik gegevens omdat ik toen nog niet wist of ik andere asides ging gebruiken. Dit is dus niet het geval en dus wijzig ik dit inderdaad best. Ook zo met header tag als ik me niet vergis

    - Het gebruik van lege divs met class=clear begrijp ik niet. Zet die clear: both op de daaronder liggende div en verander de margin-top in padding-top?
    Good point. Ik dacht dat de padding-top toen niet werkte. Zal nog eens testen. :P

    - Waarom staat uw logo binnen een h1?
    Ik las dit ergens, vond het ook raar. Maar indien ik het doet dan verander ik de titels best naar H2's. Maar die H1 van de logo is idd niet echt relevant.

    - Ge laadt twee keer uw conditionals (in head en voor sluiten van body)
    Goed gezien. Ik weet niet hoe ze daarin terecht zijn gekomen. :P

    - Kleuren zijn OK, maar de kleur van .blue vind ik net ietsje te fel.
    Is idd waar

    Fancybox wordt aangepast.

    - Heel uw site is in het Engels, maar als je iets van het contactform niet invult krijg ge 'vul dit veld in', en dat dan nog in een hele lelijke tekstballon die totaal niet past bij de website.
    Dit heeft te maken met de HTML5 required attribuut op de input tag. Kan er weinig aan doen. Dat het Nederlands is, heeft te maken met de taal van u browser. Ook het design van die tekstballon bepaalt dit. Kan er weinig aan doen, maar in Chrome is het inderdaad ERG lelijk.


    Erg bedankt voor die nuttige analyse! Great job.
    no votes   Reply With Quote Reply With Quote

  4. #4
    Shaddix's Avatar
    Registered
    08/09/09
    Posts
    6,121
    iTrader
    23 (100%)
    Mentioned
    9 Post(s)
    Reputation
    3/121
    nog niet naar de code gekeken, maar de looks vind ik best wel nice, strak met een speelse toets, mooi!
    PSN: Shaddix-be
    no votes   Reply With Quote Reply With Quote

  5. #5
    Zero Grav's Avatar
    Registered
    06/06/04
    Location
    Bilzen
    Posts
    8,805
    iTrader
    18 (100%)
    Mentioned
    0 Post(s)
    Reputation
    42/128
    Quote Originally Posted by BramVroy View Post
    This quote is hidden because you are ignoring this member. Show
    - ik dacht altijd dat het punt van nieuwe HTML5-elementen was dat je minder met IDs moet werken. Wat jij doet, houdt dus weinig steek: je werkt met aside en geeft het nog een ID? Dat is hier toch niet nodig, want je werkt maar met één aside. Hetzelfde met de header.
    Dat is niet de bedoeling van de nieuwe HTML5 elementen, de bedoeling is om semantischere code te kunnen schrijven. Niet overal dezelde div tag voor gebruiken, maar tags die aangeven wat het doel ervan is.

    Zo'n aside-tag is dus niet noodzakelijk bedoeld om een sidebar te maken, en zelfs dan is dat niet enkel en alleen de bedoeling ervan.
    Duidelijker is het misschien bij footer. Dat slaat terug op de footer van de pagina, maar evenzeer op de footer van elke post op een blog.

    In dit geval, als de aside maar één keer gebruikt wordt kan een id misschien achterwege gelaten worden. Maar doorgaans zijn id's performanter dan gewone tags of classes, dus ik zou het zeker niet afraden.

    Sorry voor de offtopic, maar ik wilde dit toch even vermelden.

    http://html5doctor.com/aside-revisited/
    no votes   Reply With Quote Reply With Quote

  6. #6

    Registered
    14/08/10
    Location
    Diest
    Posts
    2,419
    iTrader
    1 (100%)
    Mentioned
    0 Post(s)
    Reputation
    8/16
    Quote Originally Posted by Zero Grav View Post
    This quote is hidden because you are ignoring this member. Show
    Dat is niet de bedoeling van de nieuwe HTML5 elementen, de bedoeling is om semantischere code te kunnen schrijven. Niet overal dezelde div tag voor gebruiken, maar tags die aangeven wat het doel ervan is.

    Zo'n aside-tag is dus niet noodzakelijk bedoeld om een sidebar te maken, en zelfs dan is dat niet enkel en alleen de bedoeling ervan.
    Duidelijker is het misschien bij footer. Dat slaat terug op de footer van de pagina, maar evenzeer op de footer van elke post op een blog.

    In dit geval, als de aside maar één keer gebruikt wordt kan een id misschien achterwege gelaten worden. Maar doorgaans zijn id's performanter dan gewone tags of classes, dus ik zou het zeker niet afraden.

    Sorry voor de offtopic, maar ik wilde dit toch even vermelden.

    http://html5doctor.com/aside-revisited/
    Ik had voor ik postte ook al eens opgezocht hoe de aside tag juist gebruikt moet worden, en daar las ik dat inderdaad. Toch lijkt het mij vrij nutteloos dat als je maar één element van een soort hebt op je webpagina, je daar dan een id gaat aangeven. Het kan nodig zijn en relevant (bij de aside kan ik het nog snappen, het is semantisch inderdaad wat duidelijker) maar soms vind ik het echt overbodig (zoals bij de header-tag in dit geval).
    no votes   Reply With Quote Reply With Quote

  7. #7
    Shaddix's Avatar
    Registered
    08/09/09
    Posts
    6,121
    iTrader
    23 (100%)
    Mentioned
    9 Post(s)
    Reputation
    3/121
    euhm, in HTML 5 kan een headertag meermaals in één pagina terugkomen

    voorbeeld
    PSN: Shaddix-be
    no votes   Reply With Quote Reply With Quote

  8. #8

    Registered
    14/08/10
    Location
    Diest
    Posts
    2,419
    iTrader
    1 (100%)
    Mentioned
    0 Post(s)
    Reputation
    8/16
    Quote Originally Posted by Shaddix View Post
    This quote is hidden because you are ignoring this member. Show
    euhm, in HTML 5 kan een headertag meermaals in één pagina terugkomen

    voorbeeld
    Ja, maar dat bedoel ik niet. Ik bedoel dat in dit voorbeeld het duidelijk is waar die header voor dient (afgeleid van de naam zelf). Dat heb je bij die 'aside' niet. Ik zou dus die header geen id hebben gegeven en die aside wel. Het gaat dus wel over DEZE website dan hé.
    no votes   Reply With Quote Reply With Quote

  9. #9
    Dieterg's Avatar
    Registered
    08/01/05
    Location
    Turnhout
    Posts
    1,182
    iTrader
    0
    Mentioned
    0 Post(s)
    Reputation
    9/9
    Quote Originally Posted by bangbang View Post
    This quote is hidden because you are ignoring this member. Show
    Ik las dit ergens, vond het ook raar. Maar indien ik het doet dan verander ik de titels best naar H2's. Maar die H1 van de logo is idd niet echt relevant.
    Even hier op terug komen. Wat je best in uw geval doet is het logo in tekst typen in de <h1> tag. Dan met behulp van CSS uw text wegdoen met text-indent en overflow op hidden zetten.
    -
    no votes   Reply With Quote Reply With Quote

  10. #10

    Registered
    14/08/10
    Location
    Diest
    Posts
    2,419
    iTrader
    1 (100%)
    Mentioned
    0 Post(s)
    Reputation
    8/16
    Quote Originally Posted by Mam View Post
    This quote is hidden because you are ignoring this member. Show
    Even hier op terug komen. Wat je best in uw geval doet is het logo in tekst typen in de <h1> tag. Dan met behulp van CSS uw text wegdoen met text-indent en overflow op hidden zetten.
    Inderdaad, dat is wel goed voor SEO dan, maar als je der enkel een afbeelding in gooit niet.
    no votes   Reply With Quote Reply With Quote

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  

Log in

Log in