The goal is to catch cases where the return value exists, but is
forgotten. There was a large enough number of them to turn this
into a real check. Initially, I just wanted to implement it to fix
the problems, then delete the code. But since this is so common, I
think it is worth the annoyance.
The constraint:set_width function listed ‘height’ as a required parameter; the constraint:set_height function listed ‘width’ as a required parameter. Swapped those around.
Also, fix the fitting logic so that the result width and height are fitted in the available space. Previously the result dimensions can be larger than the input, but cropped in e7a21947e6/lib/wibox/widget/base.lua (L547). But still it can cause problems e.g. when used inside wibox.container.place.
It looked horrible/buggy when combined. Now it looks like people would
expect it to work. Another problem was the `bar_border_width` look
when `ticks` was `true.
This exposes the `librsvg` DPI and Stylesheet properties. This
is a groundwork commit to port the xresources theme to use a SVG
wallpaper instead of hardcoded Cairo code to generate the colorfar
logo.
Previously, it was really hard to get the image to do one *or* the
other. The `resize` property set both at the same time.
While it could, this does not deprecate the old `resize` property
since 99% of the user wan't to lock both.
The list of possible values was missing one. Additionally, this adds an
example to visualize the available options.
Signed-off-by: Lucas Schwiderski <lucas@lschwiderski.de>
The newly changed code doesn't handle this well:
local w = wibox.widget {
{
--add anything here
widget = wibox.layout.fixed.horizontal
},
widget = wibox.layout.fixed.horizontal,
}
This will cause the "inner" fixed layout to have the minimum size
it supports. In that case, if the last widget has "no size" because
it supports up to 0x0, then it isn't added to the layout.
This was done "on purpose" because if there is a spacing, then `:fit`
would have returned a size "too small" because the last spacing area
would be (correctly) missing.
But if the zero sized widget isn't added to the layout, then it's size
isn't tracker. So if it emits a layout_changed signal, nothing catches
it.
The "fix" is rather hacky and probably a little incorrect. It rely
on the behavior of `:fit()` to avoid adding the "wrong" widgets to
the layout, which is fragile.
However, I don't have a better idea.
feat(menubar): allow setting beautiful.menubar_font
fix(menubar: init: get_current_page): label return value
Update lib/wibox/widget/textbox.lua
Co-authored-by: Lucas Schwiderski <4508454+sclu1034@users.noreply.github.com>
Update lib/wibox/widget/textbox.lua
Co-authored-by: Lucas Schwiderski <4508454+sclu1034@users.noreply.github.com>
revert textbox changes
fix(menubar: init: show): fix font height detection
Update lib/menubar/init.lua
fix(menubar: init: get_current_page): list_spacing depends on presence of icon; also take cursor block width into consideration
fix(menubar: init: label): force icons to be square
fix(menubar: init: get_current_page): also take border_width into consideration when computing available space
merge conflict
Emmanuel Lepage Vallee
Pepijn Krijnsen
Actionless Loveless
Xinhao Yuan
Lucas Schwiderski
Anton Bulakh
mergify[bot]
mattplm
aflorea-2k
Aire-One