[Oxygen] [Bug 425874] Shorter titlebar (without smaller font) option needed: less padding

[Duncan]

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

--- Comment #3 from Duncan <1i5t5.duncan at cox.net> ---
Created attachment 131226
  --> https://bugs.kde.org/attachment.cgi?id=131226&action=edit
oxygen titlebar negative vmargins demo hack-patch

Glad I took the time for a second look.  A lot more made sense this time thru
and I ended up dropping one patch entirely as unnecessary, and making another
basically test-time only.

This is the main idea here.

At the given negative 2 TitleBar_TopMargin and -1 TitleBar_BottomMargin
settings it squeezes things slightly on the Noto Sans Medium font, normally no
exotic (aka mostly ASCII) characters common in my titlebar here.  It's likely
to be worse for Asian charactersets and the like, and as such it's likely to be
the lower bound.

Zeroing out both values (along with TitleBar_OutlineMargin, already zeroed out
here) would yield a somewhat less vertically squeezed titlebar that if I'm
understanding the call to qt font-functions the metrics are based on correctly,
should just fit maximum font ascenders and descenders even for non-western
characters.  Zeroed values resulted in a 16px high titlebar at 8pt font I was
working with.

But the results for the negative values as posted are even better than I
expected, a 10px high titlebar at the 8pt font size I was working with.  That
allowed me to increase the font to 10pt for 14px high.  I'll have to play a bit
more to see whether I prefer the extra 4px I get at 8pt font or the somewhat
more comfortable to read 10pt.  (9pt was the same 14px high titlebar as 10pt,
while 11pt increase it to 17px.  So 8pt and 10pt are the practical choices.)

Would be cool if I could do the option UI too but that's beyond me ATM. 
Assuming it's implemented, maybe I can learn from the commit adding it...

There's another minor testing-only patch coming too, adding a tiny button size,
but it didn't reduce the titlebar size so I consider it only practical for
testing.

-- 
You are receiving this mail because:
You are the assignee for the bug.