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

Manual distance picker miles #458

Closed

Conversation

gerhardol
Copy link
Collaborator

Manual add distance picker mixed miles and meters

Also set defaults

Still presented as m without unit, but that is a general setup
TitleSpinner: mValue should be split in the presentation and value.

@jonasoreland
Copy link
Owner

the usage of 1000 has to be wrong ?

@gerhardol
Copy link
Collaborator Author

The change is to instead of entering meters for the minor unit is to enter thousands. So if you set 1.002 miles this would be 1612m.
An alternative would be to let the picker just set complete km/miles and parse the string.

@gerhardol
Copy link
Collaborator Author

Any comment on this?

@gerhardol
Copy link
Collaborator Author

Any suggestion how to not mix miles and meter?
One field with miles/km where the advance affects complete km/miles?

@gerhardol
Copy link
Collaborator Author

Suggestion: This should be reworked to have one field only but +/- steps complete km/miles

@gerhardol gerhardol added the bug label Aug 8, 2017
@ghost
Copy link

ghost commented Aug 15, 2017

Any update on this. Because it shows as m instead of mi when I upload to runalyze it throws off the distance.

@gerhardol
Copy link
Collaborator Author

My pull request was rejected, I still consider this as a bug though. How should it look like?

@gerhardol
Copy link
Collaborator Author

Superseded by #624

@gerhardol gerhardol closed this Aug 20, 2017
@gerhardol gerhardol deleted the manual-distance-picker-miles branch August 20, 2017 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants