Tijden veranderen
[Review] I-cey.net
Mijn jaarlijkse 'tijd voor een update!' zorgde er ook deze keer weer voor dat mijn website weer eens een remake kreeg. Op zich ben ik er tevreden over maar ik zou graag jullie meningen horen danwel wat extratjes voor wat extra kleur...
Klik.
Contactformulier is nu naar mijn mening lelijk sorry.
De rest is echt super mooi simpel en overzichtelijk
edit: Nog wel even favicon aanpassen he;-)
Niek, Het is logisch dat we weer naar boven gaan. Het enige dat veranderd is de bovenste afbeelding... als we dan niet terug naar boven zouden gaan zou niemand het merken ;). Dat tweede heb je gelijk in en zou nu gefixed moeten zijn.
@wes
Het is de bedoeling dat een website mooi, simpel en functioneel is, wanneer het het type site betreft wat die van Arjan ook is. (of Y-cey)
En ik kan niet anders zeggen dan:
Mooi
Simpel
Functioneel
Doeltreffent
Toch nog een paar opmerkingen
Valideer je css en html code eens, er zijn maar een paar kleine dingetjes die niet goed zijn. (voor je css geldt dat overal, mbt je container div)
Voor de html geld dat niet voor de home, (heb ze niet allemaal getest) maar bijvoorbeeld wel voor de tab "Kenia".
Het zijn maar zogenaamde "schoonheidsfoutjes" niets ernstigs.
Mooi dat je een stukje over Kenia erbij hebt gezet :) en mooie foto's!
Robert, normaal ben ik niet zo'n validator persoon maar je hebt gelijk, ik heb even een paar dingetjes aangepast en alles is nu als het goed is goed :). Thx :).
niek90, de vorige leek meer op wat arjaninkenia nu is. Natuurlijk is het geen dramatische 'make-over' maar dat komt omdat mijn website geen doel heeft. :).
Chef, ken je het KISS princiepe? 'Keep it simple stupid!' Ik hou daar wel van! Overzichterlijk, duidelijk en makkelijk te begrijpen voor iedereen. Die achtergrond moet ik ook nog even over nadenken maar ik vind de standaard schuine streep die ik overal gebruik een beetje afgezaagd :P. Thx :).
En ik weet ook dat je niet zo'n validator type bent, maar vond het toch zonde omdat het maar zo weinig was ;)
Voor de rest: Mooi! :-)
Quote:
Niek, Het is logisch dat we weer naar boven gaan. Het enige dat veranderd is de bovenste afbeelding... als we dan niet terug naar boven zouden gaan zou niemand het merken ;). Dat tweede heb je gelijk in en zou nu gefixed moeten zijn.
Ok, daar heb je gelijk in ;-)
Gewijzigd op 01/01/1970 01:00:00 door niek s
Maar vanuit dezelfde reden dat het niet meer dan logisch is om weer boven terecht te komen waneer je een link hebt geklikt, is het ook best logisch om 'volgende' en 'vorige' (ook) onderaan de pagina in het fotoalbum te hebben. Ik neem aan dat je niet onderaan begint en naar boven de pagina afwerkt.
En je achtergrond-link kan wel een beetje groot worden ;)
Code (php)
1
http://www.i-cey.net/?pagina=fotoalbum&paginanr=2&map=fotoalbum/kenia&achtergrond=bergen&achtergrond=bergen&achtergrond=bergen&achtergrond=bergen&achtergrond=bergen&achtergrond=bergen&achtergrond=strand&achtergrond=bergen
Maar dat is op zich geen probleem, want zodra je een andere pagina opzoekt is de link weer lekker kort. Wel jammer dat je geen mod_rewrite hebt toegepast, staat net even wat netter en simplistischer.
Gewijzigd op 01/01/1970 01:00:00 door Jelmer -
Quote:
edit: je contact formulier word ook niet goed uitgelijnd in IE
Gewijzigd op 01/01/1970 01:00:00 door Marien xD
- Als je op de meest rechtse van de drie achtergronden klikt, komt er een andere dan op de thumb staat.
- Die achtergrond is echt héél lelijk. (mijn mening hoor)
- De links in de tekst zou ik een andere kleur geven, die iets beter past.
- Als ik de tab van de pagina waar ik nu zit hover, wordt de tekst een beetje lastig leesbaar. Klein detail...
Verder is het echt een prachtsite!
Ook zou het helemaal tof zijn als je de menu items transparant doet... zodat je nog een beetje van de foto daaronder ziet...(opacity)
Jelmer, heb je gelijk in. Ik heb de volgende/vorige ook onderin toegevoegd. Dat dat linkje een beetje lang word heb je gelijk in, maar ik wou het voor mijzelf niet te ingewikkeld maken...
Code (php)
1
2
3
4
5
6
7
8
9
10
11
2
3
4
5
6
7
8
9
10
11
<?php
foreach($achtergronden as $achtergrond)
{
if(!empty($_SERVER['argv']))
{
echo '<a href="?'.$_SERVER['argv'][0].'&achtergrond='.$achtergrond.'"><img src="afbeeldingen/'.$achtergrond.'_klein.jpg" alt="'.$achtergrond.'"></a> ';
}else{
echo '<a href="?achtergrond='.$achtergrond.'"><img src="afbeeldingen/'.$achtergrond.'_klein.jpg" alt="'.$achtergrond.'"></a> ';
}
}
?>
foreach($achtergronden as $achtergrond)
{
if(!empty($_SERVER['argv']))
{
echo '<a href="?'.$_SERVER['argv'][0].'&achtergrond='.$achtergrond.'"><img src="afbeeldingen/'.$achtergrond.'_klein.jpg" alt="'.$achtergrond.'"></a> ';
}else{
echo '<a href="?achtergrond='.$achtergrond.'"><img src="afbeeldingen/'.$achtergrond.'_klein.jpg" alt="'.$achtergrond.'"></a> ';
}
}
?>
In die modrewrite heb je ook wel gelijk maar dat heb ik niet gedaan omdat ik in de problemen kwam met mijn fotoalbum. Onderstaande link is moeilijk te rewriten door de slash in de $map var. Tenzij ik alles om ga gooien...
Code (php)
1
http://www.i-cey.net/?pagina=fotoalbum&map=fotoalbum/kenia&foto=11#bovenkant
Niek, Ik zie geen bug :P.
Marien, heb je groot lijk in! Ik heb een A name toegevoegd. In IE werkt het prima maar in Firefox werkt het helaas NIET dus als iemand het probleem kan vinden...
Herjan,
- Niet waar, het is een uitsnede. Kijk maar zo'n beetje boven de I van Icey.
- Ieder zijn mening.
- Moet ik nog over nadenken, om GUI-practische redenen is die nu standaard.
- Kijk ik later nog wel eens naar :).
Arian, opacity heb ik eerder leuke dingen mee gedaan (klik maar trekt mij op dit moment niet zo. Misschien kijk ik er later nog eens naar. Met betrekking tot die anchors heb je gelijk maar die krijg ik dus niet goed werkend.