Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dokąd zmierza routes? #94

Open
4 of 6 tasks
Cosiek opened this issue Aug 6, 2014 · 6 comments
Open
4 of 6 tasks

Dokąd zmierza routes? #94

Cosiek opened this issue Aug 6, 2014 · 6 comments
Assignees

Comments

@Cosiek
Copy link
Collaborator

Cosiek commented Aug 6, 2014

WIęc routes potrafi już czytać gpx-a i dodawać trasy z palca. Pytanie co jeszcze powninien umieć?

Kilka pomysłów:

  • edycja zapisanej trasy (bardzo trudne dla tras z gpx-a)
  • wyświetlanie markerów w zależności od zbliżenia
  • ustalanie pozycji usera (fajnie by było zacząć od przeszukiwania miasta, zamiast półkuli)
  • wyświetlanie wielu tras na raz (nie wiem po co)
  • "zapisz" formularza zapisuje też trasę (usunąc guzik zapisz trasę)
  • ładniejsze przeciąganie trasy (klik+drag, zamiast klik+puść+drag marker)

Wystawiam na to issue, bo będę chciał zrefaktoryzować mapHandlera (po akceptacji #93), ale nie wiem jak. Jeżeli uznasz, że wyświetlanie wielu tras jednocześnie może nam się kiedyś przydać, to będę działał innaczej niż jeżeli stwierdzisz, że taka sytuacja nigdy nie zajdzie.

@v0y 🎱

@v0y
Copy link
Owner

v0y commented Nov 23, 2014

@Cosiek zaznaczyłem fajne pomysły.

Edycja trasy - chyba na tym etapie za bardzo zbędna
Wiele tras - też nie wiem, po co :)

I coś, czego mi brakuje przy wyznaczaniu trasy - przycisk "usuń ostatni punkt".

Powiedz mi jeszcze, czemu jeden trening może mieć wiele tras, nie na odwrót? Miałem jakiś zamysł tak robiąc, czy coś mi się pojebało?
Chciałem, żeby wiele treningów mogło dzielić jedną trasę, więc dlaczego tak zrobiłem?

@Cosiek
Copy link
Collaborator Author

Cosiek commented Nov 24, 2014

I coś, czego mi brakuje przy wyznaczaniu trasy - przycisk "usuń ostatni punkt".

Wychodzi problem z brakiem dokumentacji. Kliknięcie prawym na dowolny marker, powoduje jego usunięcie.

Chciałem, żeby wiele treningów mogło dzielić jedną trasę, więc dlaczego tak zrobiłem?

Nie pamiętam, ale też ficzer z dzieleniem trasy przez wiele treningów nie do końca mnie przekonywał. Swoją drogą, jeśli chcesz, żeby user mógł podłączyć istniejącą trasę do treningu, to będze trzeba mu zrobić widok w którym będzie mógł to zrobić i prawdopodobnie dobrze będzie wyświetlić tam mapę z większą liczbą tras.

Chyba jednak zrobię ten refactor tak, żeby umożliwić wyświetlanie wielu tras. Trochę to bardziej skomplikowane, ale coś mi mówi, że i tak będzie trzeba.

Więc pod tym issue robię refactor, a potem ficzery z listy.

@v0y
Copy link
Owner

v0y commented Nov 24, 2014

Kliknięcie prawym na dowolny marker, powoduje jego usunięcie.

👍

ficzer z dzieleniem trasy przez wiele treningów nie do końca mnie przekonywał.

Ale mnie chyba przekonuje :P Samo wyświetlanie wielu tras to raczej daleka przyszłość, początkowo widzę zastosowanie dzielenie trasy na zasadzie: biegły razem dwie osoby, jedna dodała trening, druga się podpięła.

Jak zrobisz refactor, to to wypchnij, wciągniemy :)

@v0y
Copy link
Owner

v0y commented Nov 29, 2014

ech, siedzę i próbuję zmienić relację tras i treningów z m2o na o2m, ale chujowo mi to idzie. boję się, że za głęboko to siedzi w tych javascriptach...

@Cosiek
Copy link
Collaborator Author

Cosiek commented Nov 29, 2014

Hmm...
Zmaiana na o2m (jedna trasa wiele treningów?) nie wydaje się słuszna. Podejrzewam że są sytuacje gdzie podpięcie wielu tras pod jeden trening może mieć sens (np. na trening składa się jazda na rowerze i bieg - choć do tego to i treningi jeszcze nie są gotowe).
Chyba raczej m2m jeśli chcesz wspóldzielić trasy - ale jak wolisz.

JS jest zupełnie nieświadomy jaka jest relacja między drogą a treningem (właściwie to nie jest świadomy istnienia treningu). Przy dodawaniu treningu zapisuje trasę AJAX-em i zwraca jej id, a przy podglądzie pobiera trasę AJAX-em na podstawie otrzymanego id. Dopóki dostaje właściwe id, nic więcej nie powinno go obchodzić.

@v0y
Copy link
Owner

v0y commented Nov 29, 2014

olałem temat, za dużo pieprzenia się...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants