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

Use all available width for leg message #291

Merged
merged 1 commit into from
Jan 9, 2017

Conversation

full-duplex
Copy link
Contributor

@full-duplex full-duplex commented Jan 9, 2017

I don't know if this is the best way to fix this, but it seems to work without breaking anything.

Before:

After:

@grote
Copy link
Owner

grote commented Jan 9, 2017

Hey thanks for this PR! You are on to something here, but I am worried that there is no leg duration in your last (right) screenshot.

@full-duplex
Copy link
Contributor Author

The leg duration is hidden by default (like the leg above it) and only shown when the intermediate stops are shown. Since this leg has no intermediate stops, it only shows/hides the duration of the whole leg on press.

To illustrate:

@grote
Copy link
Owner

grote commented Jan 9, 2017

Awesome, thanks! :)

Would be nice to know what you think about this idea: #231 (comment)

@grote grote merged commit 9986212 into grote:master Jan 9, 2017
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

Successfully merging this pull request may close these issues.

2 participants