]> git.mdlowis.com Git - proto/labwc.git/commit
theme: add window.titlebar.padding.{width,height}
authorJohan Malm <jgm323@gmail.com>
Tue, 8 Oct 2024 16:52:57 +0000 (17:52 +0100)
committerJohan Malm <jgm323@gmail.com>
Tue, 8 Oct 2024 19:04:05 +0000 (20:04 +0100)
commite16e78e7a4a6e40c8184edd7bb483b0f395482d0
tree06c45a4b859522a98206b5d1891923974ca8a77c
parent383fb20bf4f75b28de825c1226131d2535946f84
theme: add window.titlebar.padding.{width,height}

...to replace padding.{width,height} to minimize breaking changes with the
visual appearance of the titlebar.

With the diverging labwc specification for the titlebar (listed below)
we have to choose between (a) not supporting the padding.{width,height}
option which exist in many extant Openbox themes to keep titlebar height
(almost) the same; or (b) making the allocated button areas much smaller
and not keeping the default hover going all the way to the edges. All in
all it just seems a lot simpler and cleaner to break this link to the
openbox spec.

Examples of previous change driving the requirement for this change:

- SVG and PNG support which often results in large icons with hover
  effects.
- Theme option window.button.{height,width}
- Larger default areas for icons (26x26)

In way of an example, Numix theme sets a padding.height of 6 which would
have resulted in a titlebar 12px taller without this change.
docs/labwc-theme.5.scd
docs/themerc
include/theme.h
src/ssd/ssd-titlebar.c
src/theme.c
src/view.c