Thread: Evadegols.be
-
12-08-2013, 20:53 #1
Evadegols.be
Nieuwe website voor een fotografe, zeeeeeeer basic naar ontwerp. Eerste poging tot integratie Flickr Api, dus vooral naar code toe zou'k graag wat feedback hebben. Tips qua design altijd welkom natuurlijk. Favicon is nog niet optimaal en caching moet'k nog enablen.
www.evadegols.beno votes
-
-
12-08-2013, 21:18 #2Member
- Registered
- 27/06/10
- Location
- Oost Eeklo
- Posts
- 15,139
- iTrader
- 53 (100%)
- Mentioned
- 12 Post(s)
- Reputation
- 3/1155
misschien een domme opmerking, maar veel van haar foto's zijn aan de donkere kant. Dit geplaatst tegenover een groot wit vlak zorgt ervoor dat de foto's nog donkerder overkomen en grote delen gewoon zwart worden (bij mijn ogen toch). Heb net zo'n foto eens op een zwarte achtergrond geplaatst en zo was er meer detail te zien.
no votes
-
12-08-2013, 21:25 #3Approved 9-lifer
- Registered
- 06/11/09
- Location
- Turnhout
- Posts
- 841
- iTrader
- 8 (100%)
- Mentioned
- 0 Post(s)
Nog een andere opmerking die niet direct gelinkt is met de site... als contact e-mail heb je staan info@evagols.be en het domein van je website is evaDEgols.be. Ik neem aan dat je dat domein ook bezit of anders staat er een fout

En als je geen contact formulier voorziet zou ik er een mailto:info@evagols.be van maken
is altijd klantvriendelijker.
no votes
-
12-08-2013, 21:40 #4Approved 9-lifer
- Registered
- 08/01/05
- Location
- Turnhout
- Posts
- 1,182
- iTrader
- 0
- Mentioned
- 0 Post(s)
- Reputation
- 9/9
Ik zou volgende aanpassingen doornemen:
- Geef uw link een HREF attribuut mee met de link, zodat je uiteindelijk bv http://www.evadegols.be/#Landschap krijgt als je op de link klikt
- Geef de omringende List-item een ID attribuut (identiek aan de HREF, de titel dus)
- in uw ul.submenu li click functie maak je volgende aanpassing
Code:var hashID = window.location.hash; var photoset_id = $(hashID).attr('data-id');
Dit is uit de losse pols en kan nog wat schoonheidsfoutjes bevatten. Door deze aanpassing kan je uw subcategorieën linken.-no votes
-
12-08-2013, 23:40 #5
Heyheyhey,
bedankt voor de snelle comment. Ivm de donkere foto's begrijp'k wel wat je bedoelt maar volg ik de wensen van de klant.
Mailto aangepast en moest inderdaad degols zijn
. Ook de opmerkingen van Dieterg zijn aangepast.
Hmm. Merk nu wel dat'k zo'n irritant ding heb waarbij het logo soms even verdwijnt
. Morgen eens uitpluizen.
Last edited by Scissor; 13-08-2013 at 01:00.
no votes
-
13-08-2013, 14:25 #6Approved 9-lifer
- Registered
- 08/01/05
- Location
- Turnhout
- Posts
- 1,182
- iTrader
- 0
- Mentioned
- 0 Post(s)
- Reputation
- 9/9
Je hebt nog één foutje, of je moet het ID toevoegen aan de list of pas het volgende aan:
naarCode:var photoset_id = $(hashID).attr('data-id');
Vervolgens zou ik nog een controle uitvoeren als er een # voorkomt in de URL, dan zou ik het submenu zowiezo tonen.Code:var photoset_id = $(hashID).parent().attr('data-id');-no votes
-
13-08-2013, 19:19 #7
Hmmm, mag'k btw ff vragen wat de bedoeling is van die aanpassingen? Sinds 'k die heb toegepast bugged mijn site (soms twee albums zichtbaar etc.). Controle van die # in de url is beetje nutteloos aangezien het single-page is en zelf de "contact" link een # add aan de url. Contactpagina is gewoon een hidden div.
Edit: Blijkbaar als je snel twee verschillende links aanklikt wordt de eerste niet gecleared. Had dat eerst ook als'k dubbelklikte op een link maar daar heb'k nu controle op gezet of die al active is of niet. Iemand ideetjes? Link inactive maken voor paar seconden?no votes
-
13-08-2013, 19:34 #8Member
- Registered
- 25/05/09
- Location
- OVL
- Posts
- 1,031
- iTrader
- 17 (100%)
- Mentioned
- 0 Post(s)
Tekst en plaatje staan niet mooi naast elkaar op de about pagina bij mij.
Of is dat de bedoeling?
http://i.imgur.com/WiLe5Fh.pngno votes
-
13-08-2013, 20:18 #9
Is de bedoeling (vraag v klant) maar vind't idd ook niet echt super. Mss 's aanpassen.
no votes
-
13-08-2013, 21:38 #10Approved 9-lifer
- Registered
- 08/01/05
- Location
- Turnhout
- Posts
- 1,182
- iTrader
- 0
- Mentioned
- 0 Post(s)
- Reputation
- 9/9
De bedoeling die ik had (werkt precies nog niet, ligt aan uw active klasse denk ik..) is dat uw klant albums kan linken naar klanten. bijvoorbeeld Eva Gols - Fotografie komt dan uit bij het juiste album. Nu werkt dit nog niet omdat het de klasse active niet heeft, vermoed ik. Nog niet helemaal kunnen nakijken, heb even niet genoeg tijd
. Maar je hebt het idee en ik denk dat dit een serieuze verbetering is, ookal is het "maar" een single page webapp!
-no votes
-
14-08-2013, 10:39 #11
Kan'k nog 's op zoeken. Op dit ogenblik zit'k met nog 2 dringendere "probleempjes":
- Als ik About openklik en vervolgens terug een album aanklik, laad hij de pagina volledig opnieuw, terwijl dat normaal gewoon een div is met een Z-index 999 die boven het open album zit. Je zou denken als die about div fadeout dat gewoon de onderliggende div zichtbaar wordt.
- Als ik twee categorieën snel na elkaar openklik, laden ze dubbel. Best links uitschakelen tot m'n Json request volledig binnen is? Of betere practice?
Edit: hashlinking zou nu moeten werken. Code is wel allesbehalve optimaal heb ik het gevoel en 'k merk ook dat hij soms de navigatie vergeet te appenden.
Eva De Gols - LandschapLast edited by Scissor; 15-08-2013 at 23:20.
no votes

