search

malmostad / sitevision_assets

Sitevision specific assets used in addition to Global Assets
http://malmostad.github.io/wag-external-v4/
GNU Affero General Public License v3.0
0 stars 1 forks source link

JS-bugg i Kontaktruta #261

Closed martent closed 9 years ago

martent commented 9 years ago

Såg precis att vi har en bugg i Javascriptet för Kontaktrutan som visar sig när något annat formulärfält ändras på en sida.

På raden: https://github.com/malmostad/sitevision_assets/blob/master/src/javascripts/contact_us.coffee#L27 sätts en change-lyssnare på $(document.body) som sedan tillämpas på $chooseDistrict.find("select"). Det buggar ur när man ändrar värdet i ett annat formulärfält på en sida, val() på $chooseDistrict.find("select") hämtas men det senare är undefined.

Går att se på te.x. login-sidan på malmo.se prod när login-fälten ändras (vid blur).

Funkar det att ändra till $(document.body) till något mer avgränsat som ändå finns i DOM:en vid sidladdning? Eller att lägga på en check om objekten finns innan man kollar properties på dem (kan vara på fler ställen i samma skript)

cc: @legander

legander commented 9 years ago

Har deployat en fix på test 553d93eb80868ab42682c908ae472286b62d6390. Jag har testat och vad jag kan se så fungerar det ok, testa ni också.

martent commented 9 years ago

Funkar bra på www-test. Propagera ut i miljöerna.

martent commented 9 years ago

Funkar fint i prod!