The text was smaller than than minimum configuration, and
also smaller than the width allowed for.
Fix the logic to what was probably originally meant.
Elided is still disabled as it triggers too aggresively on narrow fits.
Does this work well with vertical panels?
It only affects vertical panels, it is in the vertical panel configuration.
Thanks for the patch! Looks like this doesn't fully fix the issue though. With "Short Date" I see the following:
(The bug I'm thinking of is https://bugs.kde.org/show_bug.cgi?id=417728)
In my case the text was too small.
I wonder how exactly Text.Fit works, if it just increases font-size for short format to fill width that could make it too big.
Well your patch doesn't regress anything for me, so if it fixes it for you, that's good.
Still it might be good to have some clarity here so we understand what's going on! :p
Fixing also for short data and very long dates
Now using the "Long Date" setting in a double-width panel results in the following:
And "Short Date" looks like this:
font.pixelSize: timeLabel.height * 0.4 doesn't look right to me.