Feature - Add routing options screen

Sygic / sygic-maps-kit-android

Project containing Android libraries separated into modules which are built over Sygic Maps SDK and provide rapid development of apps with Sygic maps solution.

MIT License

22 stars 3 forks source link

Feature - Add routing options screen #99

Closed vbieleny closed 4 years ago

Darxie commented 4 years ago

v Transport mode prosim nechaj zatial iba Car, Transport truck a Pedestrian. a kedze je to az takto perzistentne, pridal by som Reset/Reset to defaults tlacidlo, ktore by vytvorilo nove routing options a resetlo by aj tie UI prvky.

MartinStyk commented 4 years ago

Vsimol som si, ze ked vyberiem Routing service offline, nikdy sa mi envypocita routa. Je to ok mat tam tuto moznost ked appka funguje len online (ak spravne rozumiem)

vbieleny commented 4 years ago

@MartinStyk k tomu manazeru, ano, myslim si ze by to bolo takto ovela lepsie, zjednodusilo by to vela veci. Povodne som sa nechcel spoliehat na to ze ta classa sa bude dat takto injectovat, ale zbytocne by to pridalo komplexitu. Z pohladu daggeru si myslim ze by to nemuselo byt komplikovane.

vbieleny commented 4 years ago

Vsimol som si, ze ked vyberiem Routing service offline, nikdy sa mi envypocita routa. Je to ok mat tam tuto moznost ked appka funguje len online (ak spravne rozumiem)

Zatial funguje len online, ale pravdepodobne sa v buducnosti dorabat aj offline rezim, tak si myslim ze by to tam mohlo zatial ostat.