Make latitude, longitude, altitude and time settable via the web UI #124
Conversation
python/views/header.tpl
Outdated
| @@ -25,6 +26,7 @@ | |||
| <li><a href="/">Home</a></li> | |||
| <li><a href="/remote">Remote</a></li> | |||
| <li><a href="/network">Network Setup</a></li> | |||
| <li><a href="/gps">GPS Setup</a></li> | |||
There was a problem hiding this comment.
I'd rather have this on the 'Tools' page as it should (ideally) be rarely used. I know it's come up a lot in the discord (people not using GPS, issues with locks), but I hope these are not a representative sample of users. If people are using this often I have failed in some significant way as it's absolutely NOT the intended (or a good) user experience.
There was a problem hiding this comment.
then I'd need to integrate it with the Tools page, and it's a lot of code, tools might become a garbage bin then. Maybe not adding it to the menu is enough? On save it returns to home
There was a problem hiding this comment.
Yes... tools is probably not the right place. How about not on the menu, but accessible from the main page via the edit icon?
There was a problem hiding this comment.
that's what I did in the last commit
| @@ -18,6 +18,7 @@ | |||
| <i class="material-icons medium">{{gps_icon}}</i> | |||
| </td> | |||
| <td class="grey-text text-lighten-1">{{gps_text}}<br>lat: {{lat_text}} / lon: {{lon_text}}</td> | |||
| <td><a href="/gps" class="grey-text"><i class="material-icons">edit</i></a></td> | |||
There was a problem hiding this comment.
Sorry @mrosseel I missed this edit to the index page! Merging now!
This helps in case of either forgetting your GPS dongle, or having a malfunctioning internal GPS.
Possible future work: