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

Disable or remove GPX export until active route #19

Open
jbelien opened this issue Apr 2, 2020 · 4 comments
Open

Disable or remove GPX export until active route #19

jbelien opened this issue Apr 2, 2020 · 4 comments
Labels
brussels For brussels. ux A UI/UX issue

Comments

@jbelien
Copy link
Contributor

jbelien commented Apr 2, 2020

Nothing happens when you click on "GPX" button.

Error in console:

TypeError: e is undefined
@xivk
Copy link
Contributor

xivk commented Apr 2, 2020

Just tried, works fine for me... do you have an active route?

@xivk
Copy link
Contributor

xivk commented Apr 2, 2020

Perhaps changes this into:
'Disable or remove GPX export until active route?'

@jbelien
Copy link
Contributor Author

jbelien commented Apr 2, 2020

You're right !
It works when there is an active route but returns an error when there is no active route.

Don't know what I was expecting to export without any active route to be honest 😂
Shouldn't return an error in the console though.

@xivk xivk added the brussels For brussels. label Apr 2, 2020
@xivk xivk changed the title Issue with GPX Export Disable or remove GPX export until active route Apr 2, 2020
@pietervdvn
Copy link
Contributor

I originally placed the download-GPX button in the sidebar, in the instruction pane that only appears when a route has been planned for exactly this reason.

@xivk xivk added the ux A UI/UX issue label Jun 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
brussels For brussels. ux A UI/UX issue
Projects
None yet
Development

No branches or pull requests

3 participants