https://bugs.kde.org/show_bug.cgi?id=453111

--- Comment #7 from loca...@tutanota.com ---
> s = m_secondsIncluded ? QLocale().toString(time, QLocale::LongFormat) : 
> QLocale().toString(time, QLocale::ShortFormat);

If m_secondsIncluded refers to the "Show seconds" config setting then I don't
think it's still quite right, I'd replace it with

> s = QLocale().toString(time, QLocale::LongFormat) 

Also, please see the https://bugs.kde.org/show_bug.cgi?id=452888 bug, they are
related.

I know it may look somewhat inconsistent that on one hand I want the 24-hour
setting to be propagated to the "Copy to clipboard" submenu but don't want the
"Copy to clipboard" submenu to be affected by the "Show seconds" setting. The
issue here is that if I enable the "Show seconds" setting then seconds will be
shown in the Digital Clock on the taskbar and I don't want things changing on
my taskbar every second, it's just distracting. Also I don't see a lot of use
cases where someone copying date and time would care about seconds, it mostly
happens when someone is copying the exact time, which is addressed by the
change above.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to