Summary: | [patch] more psycho-visually-scannable "time to" column in kalarm | ||
---|---|---|---|
Product: | [Applications] kalarm | Reporter: | Robert <kdebz> |
Component: | general | Assignee: | David Jarvie <djarvie> |
Status: | CLOSED FIXED | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | RedHat Enterprise Linux | ||
OS: | Linux | ||
Latest Commit: | 56f3d3e4042a19dd6263edfb75613ecf2ece2515 | Version Fixed In: | 18.08.1 |
Sentry Crash Report: | |||
Attachments: |
patch to right-align & zero-pad the hour for "time to" column
screenshot showing the patch effect |
Created attachment 114290 [details]
screenshot showing the patch effect
Thank you for this patch. I think it does improve how "time to" values are displayed. Apart from some reformatting (adding spaces and removing unnecessary brackets, to make it similar to the existing code), I'm happy with your patch. Do you have KDE commit access, or do you want me to commit it? If the latter, do you want me to attribute the patch to you, with your name and email? I'm glad you approve! I do not have commit access, and don't require or object-to attribution... whatever is most convenient. This has now been committed. The fix will be in version 2.12.1, in KDE Applications 18.08.1 (git commit 56f3d3e4042a19dd6263edfb75613ecf2ece2515). Thank you. |
Created attachment 114289 [details] patch to right-align & zero-pad the hour for "time to" column Please consider the attached patch (and/or screenshot) intended to make the "time to" column more readily understandable "en masse" & "at a glance". Two simple changes: * right-align the "time to" column * zero-pad the hour segment (if it will have a number of days before it) Thanks.