search

ini20 / berliner-schulen

Finde die passende Schule in Berlin
http://berlinerschulen.ini20.de
BSD 3-Clause "New" or "Revised" License
7 stars 3 forks source link

'Tech' in Schlagwort voreingetragen #68

Closed Janekdererste closed 9 years ago

Janekdererste commented 9 years ago

Im Moment wird das Schlagwortfeld mit 'Tech' vorbelegt. Schöner wäre es, wenn es bei Aufruf der Seite leer wäre.

Es wird eine Vorauswahl getroffen und beim erstmaligen Einstellen eines Filters gelöscht.

obstschale commented 9 years ago

Leer ist doof. Es ist ja gerade ein Keyword da damit es beim ersten Lade nicht so lange dauert.

Janekdererste commented 9 years ago

In meinem Rechner ist die Ladezeit der Karte noch in ordnung würde ich sagen. Der Rest der Seite ist dann auch schon zu sehen.

Allen, denen ich die Seite gezeigt habe, haben nicht so richtig verstanden, dass bereits eine Vorauswahl getroffen wurde. Es erscheint mir besser etwas länger zu laden, als die Nutzer zu verwirren.

@obstschale Fällt uns noch was besseres ein, als eine Vorauswahl zu treffen?

Janekdererste commented 9 years ago

@obstschale Habe es mal entfernt und ein temporäres Array für die Marker in der 'onUpdateSchools' Methode hinzugefügt. Müsste man nochmal messen, ob es einen Unterschied macht, aber ich könnte mir vorstellen, dass die leaflet direvtive jedesmal, wenn sich das Markers array ändert ein rendering ansößt?

Janekdererste commented 9 years ago

sorry @obstschale zwar habe ich den Branch bereits gemerged, aber nun ist mir aufgefallen, dass wenn man die Seite läd und anschließend, ohne den Filter zu verändern auf die Detailseite einer Schule geht und wieder zurück kommt, im Schlagwortfeld 'tech' zu sehen ist. Das sollte so eher nicht sein. Vor allendingen bleibt es dann auch dort, wenn man einen Filter einstellt.

obstschale commented 9 years ago

Hmm Komisch. Macht nichts. Ich werde mich ab frühestens Mittwoch drum kümmern können.

Am 19. April 2015 bei 18:01:27, Janekdererste (notifications@github.com) schrieb:

sorry @obstschale zwar habe ich den Branch bereits gemerged, aber nun ist mir aufgefallen, dass wenn man die Seite läd und anschließend, ohne den Filter zu verändern auf die Detailseite einer Schule geht und wieder zurück kommt, im Schlagwortfeld 'tech' zu sehen ist. Das sollte so eher nicht sein. Vor allendingen bleibt es dann auch dort, wenn man einen Filter einstellt.

— Reply to this email directly or view it on GitHub.