search

dsprenkels / kn-next

Edge for a student's club homepage
0 stars 2 forks source link

Login zonder jQuery UI #30

Closed aykevl closed 10 years ago

aykevl commented 10 years ago

Ik heb de 'loginWindow' herimplementeerd zonder jQuery UI. Ik vind een venster met sluitknopje persoonlijk niet mooi. Bovendien stond die soms op de verkeerde plaats.

Dit betekent denk ik ook dat heel jQuery UI verwijdert kan worden. Een dependency minder, dus ook veel minder JavaScript en CSS dat geladen moet worden.

Getest in Chrome, Firefox en IE8.

Wat vinden jullie er van?

screenshot

aykevl commented 10 years ago

Ik zag net dat er een issue is (met csrftoken), dus nu niet mergen!

aykevl commented 10 years ago

En dit is het resultaat. Het CSRF probleem is opgelost. screenshot from 2014-01-19 11 21 00

thundur commented 10 years ago

Getest en het ziet er goed uit!

CSRF werkt (uiteraard) alleen als kninfra en kn-next op hetzelfde domein draaien.