Door
Sander N
op 21-05-2012 23:35
gewijzigd op 21-05-2012 23:42
3.133 views
Hallo,
Bij deze wil ik jullie http://www.nijhof.biz of http://www.nijhof.nu presenteren. Deze site zal gaan functioneren als persoonlijk portfolio. Momenteel staat het nog vol met spellingsfouten, en ook de CSS kan beter en gestructureerder opgebouwd worden, maar goed, het begin is er in middels al.
Achter de site draait een klein CMS dat ongeveer compleet op AJAX gebaseerd is, vond ik wel een leuke uitdaging, en aangezien de achterkant toch niet door Google gevonden hoeft te worden dacht ik van 'ja waarom niet?'.
De rest is gewoon allemaal PHP/HTML, ik ben hier nog relatief gezien een beginner in, aangezien ik pas sinds 1 februari begonnen ben met PHP (ik heb wel ervaring met andere talen).
Misschien wat leuke technische details:
- SQL injecties zijn niet mogelijk.
- MySQL
- PDO
- PHP gedeelte is goed object georiënteerd
Ik hoor graag van jullie,
Sander
Items die nog op de planning staan:
- Simpel blog.
- Contactformulier.
- Alles gestructureerd opbouwen.
Not bad, maar een herkenning bovenaan de site, dat het een portfolio / 'About me' zou het wel verder afmaken.
Verder top dat je zo actief bezig bent met PHP, en in ruim 3 maanden tijd al op een vrij hoog niveau zit.
Ik zie in je HTML-source dit:
<a href ="/pages/AboutMe"><div class = "menuButton"><p>About me</p></div></a>
<div class = "menuButton" id = "ProjectButton"><p>Projects</p></div>
<a href ="/pages/Contact"><div class = "menuButton" id = "contact"><p>Contact</p></div></a>
Nu ben ik niet een voorstander om overal divjes voor te gebruiken. Zou je met een html-list (li) niet beter af zijn?
Verder kan je een imageset van je gallery maken, zodat je kan doorklikken, zonder steeds op het kruisje te drukken en de volgende foto aan te klikken. Dit ondersteunt de Fancybox wel. http://www.nijhof.biz/pages/project/169
Bedankt voor je reactie, ik moet wel eerlijk toegeven dat ik op me stage ook een project in PHP/C# moet doen, dus op zich gaat je kennis dan vrij hard vooruit.
Wat betreft het menu, dit heb ik inderdaad met divjes gedaan, misschien is li een betere optie, zou ik even naar moeten kijken dan.
Wat betreft de afbeeldingen, je hebt gelijk. Ik moet alleen de plugin wat aanpassen dan denk, omdat als er 5 of meer afbeeldingen zijn, de laatste automatisch worden verborgen, en er verschijnt een knop zodat je naar rechts kan 'scrollen'. Ik weet niet of het netjes is als Fancybox foto's laat zien die eigenlijk nog niet getoond zijn.
Ik weet niet of je rekening houdt met IE7 gebruikers, zoja hier een aantal puntjes.
1) Als ik op projects klik komt hij helemaal rechts in beeld achter je foto omlaag.
2) De menuknoppen zijn niet volledig klikbaar (dus ik kan niet overal op de knop klikken voor de link).
3) Als ik met de muis over je menu ga, krijg ik geen handje, maar een select cursor.
4) Je facebook knop heeft er een hyperlink border omheen zitten, zet in a {} de border: none; in. Dan is dat blauwe (na bezocht paarse) randje ook weg.
Een extra advies: Zet op je contact page een simpel contact formulier, mensen zijn dan sneller geneigd contact met je op te nemen. Veel mensen moeten eerst inloggen in een webmail voordat ze dan een mailtje kunnen sturen.
?
Onbekende gebruiker
22-05-2012 08:09
gewijzigd op 22-05-2012 08:10
Bij contact zou ik een formulier maken.
Zoals Chris al zei
Bedankt voor de reacties, ik zie inderdaad dat het er in IE7 niet uitziet, vanaf IE8 dan weer wel. Ik denk er nog even over vanaf welke IE versie ik het ga ondersteunen, IE8 leek me hoog genoeg, maar daar kan ik wel eens naast zitten.
Contactformulier staat inderdaad op de planning, en heeft momenteel ook de hoogste prioriteit. Borders moet ik inderdaad even weghalen!
Wat je kunt doen is als mensen je site via IE7 benaderen je een kleine regel ergens laat zien dat ze op een verouderde versie van IE draaien, en voor betere resultaten moeten updaten.
Dus zoiets als dit om de mensen op de hoogte te stellen. Scheelt je een hele hoop stylen! IE7 is echt een ellende haha, echter zijn er veel bedrijven die er nog op draaien vanwege bepaalde webapps.
<!--[if IE 7]><p id="melding">U maakt gebruik van Internet Explorer 7, deze is verouderd en zal de site niet in optimale contditie weergeven. Voor beter resultaat kunt u deze upgraden naar <a href="http://linknaarIE9.com" target="_blank">Internet Explorer 9</a>.</p><[endif]--!>
Die optie is leuk en aardig, maar hoe wil je een IE9 op een XP bak zetten dan?
Dit is namelijk mij nog nooit en never gelukt en ik knutsel toch heel wat met pc's.
Als je het voorbeeld alleen als voorbeeld wou laten dienen. Dan lijkt het me voor TS handige dat hij dan 2 linken toont aan gebruikers met IE7. Dit dus voor Win 7 IE9 en XP IE8.
Het was maar een voorbeeld, hij kan ook gewoon verwijzen naar MS update site, dan update hij automatisch naar de laatste versie of het nu IE 7 8 of 9 is.
Dan maakt hij ervan "Update naar de laatste nieuwe versie" of iets in die richting. Ik vind het niet de verantwoordelijkheid van de site om mensen op hun omgeving te wijzen. Een simpele melding dat ze een verouderde browser gebruiken zou al moeten voldoen.
Mensen (buiten bedrijven om) die nu nog op oude browsers draaien, hebben geen idee hoe gevoel hun pc is voor aanvallen.
Hmm, dat zijn wel goede tips denk ik, ik ga het vanavond proberen te verwerken in mijn website. Inmiddels heb ik wel al de border weggehaald om het Facebook logo.
Ik meende het al dat het als voorbeeld was bedoelt.
Alleen vond ik even persoonlijk dat de informatie er bij moest ivm de IE versie op een Win 7 of XP bak. Wat je aangeeft om de bezoekers naar de MS site door te sturen is ook een betere oplossing.
