koopreynders/frontendvoordesigners

frontendvoordesigners's Issues

feature detect op XHR

frontendvoordesigners/opdracht3/PEmetXHR/script.js

Line 22 in 56a1f17

function setupXHR(){

setupXHR(); -> hier zou ik de feature detect omheen zetten: if (xhr
&& classList && docqueryselector) { setupXHR(); }, a la cutting the
mustard

Trailer links werken niet

In de trailer links van IMDB zit een timestamp verwerkt, die maar voor ongeveer 1 dag geldig is. Daarna is het denk ik niet meer geldig. Links zijn ook per browser verschillend dus wellicht werken ze überhaupt niet voor anderen?
De geupdate links werken voor mij in ieder geval ook niet..

Json truuk

Omdat de JSON files niet op https staat kan het niet in gh-pages worden gebruikt.
Als je dit probleem hebt kun je het JSON file zelf downloaden, in een mapje zetten, en de online link gebruiken in je project. Problem solved !

Zo ziet dat er voor mijn project uit:

frontendvoordesigners/opdracht3/XMLHttpRequest/script.js

Line 9 in 9d45163

var uri = 'https://koopreynders.github.io/frontendvoordesigners/opdracht3/json/movies.json'; //json file op github

De JSON file staat in een mapje in mijn project:
https://github.com/KoopReynders/frontendvoordesigners/tree/master/opdracht3/json

URLs uit de HTML halen natuurlijk

frontendvoordesigners/opdracht3/PEmetXHR/script.js

Lines 9 to 14 in 56a1f17

	var pages = [
	"https://koopreynders.github.io/frontendvoordesigners/opdracht3/PEmetXHR/index.html",
	"https://koopreynders.github.io/frontendvoordesigners/opdracht3/PEmetXHR/article2.html",
	"https://koopreynders.github.io/frontendvoordesigners/opdracht3/PEmetXHR/article3.html",
	"https://koopreynders.github.io/frontendvoordesigners/opdracht3/PEmetXHR/article4.html"
	];

var pages = [ -> URLs uit de HTML halen natuurlijk, daar staan ze
als het goed is ook al (in de <a href="...">), dan is je script ook
herbruikbaar als er opeens 5 pagina's zijn

Feature detects op querySelector

frontendvoordesigners/opdracht3/PEmetXHR/script.js

Lines 3 to 4 in 56a1f17

	var next = document.querySelector("a[rel='next']");
	var prev = document.querySelector("a[rel='prev']");

var next = document.querySelector("a[rel='next']") en
next.classList.add('disabled'); -> feature detects ontbreken! :)

Feature detect op responseType = 'document'

frontendvoordesigners/opdracht3/PEmetXHR/script.js

Line 71 in 56a1f17

request.responseType = 'document';

request.responseType = 'document'; -> cool, nooit geweten dat dit
bestond, thanks!

Vervolgvraag: wat gebeurt er in IE9? Die wel XHR en querySelector ondersteunt.
request.response bestaat daar volgens mij niet, dus geeft daar een error, maar dan heb je event.preventDefault() al gedaan, waardoor de link nooit gevolgd wordt, maar de XHR oplossing ook niet werkt. Gaat niet om IE9 natuurlijk, want die is brak en dood, maar wel om het principe. event.preventDefault() aan het eind had dit niet opgelost, omdat het in een andere functie kapot gaat, dus eigenlijk betere feature detect nodig.

event.preventDefault()

frontendvoordesigners/opdracht3/PEmetXHR/script.js

Lines 30 to 41 in 56a1f17

	next.onclick = function(){
	event.preventDefault();
	//counter bijhouden om te weten welke pagina ingeladen moet worden
	current+=1;
	if(document.querySelector("article[data-nr='"+current+"']")){
	//als de pagina al is ingeladen, dan hoeft allen de pagina getoond te worden
	shownext();
	}else{
	//pagina laden
	loadnext();
	}
	}

event.preventDefault() -> altijd aan het eind van je functie, zodat als er later iets kapotgaat, in ieder geval de default actie van de browser nog uitgevoerd wordt (het volgen van de klik).

Relatief pad

Relatief pad in een article van een button naar een element.
Handig, want dan wordt je code heel flexibel, en je hoef geen verwijzing naar het element dat je wil manipuleren mee te geven aan de functie

frontendvoordesigners/opdracht3/XMLHttpRequest/script.js

Line 67 in b0f319c

var reviews = this.parentNode.querySelector('ul');