-
Notifications
You must be signed in to change notification settings - Fork 3
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
Added 12-clock (AM/PM) support #5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for such a late review. Just small changes. Thank you for your contribution!
* Why is this? | ||
* ``` | ||
* Collections.sort(linesData, Comparator.comparing(String::length)) | ||
* linesData = linesData.reversed() | ||
* ``` | ||
* If left that part: | ||
* 1. When switch between 24 and 12 clock modes it moves lines because line length change. | ||
* 2. When someone add localization, it's also going to move lines because words is diff length in different locale |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intended behaviour. It sorts lines so the longest one is at the top. Wastes less screen space
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Intended behaviour. It sorts lines so the longest one is at the top. Wastes less screen space
Maybe that can be also configurable? I'm ready to contribute it as another PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, call it something like "Sort lines by length"
No description provided.