xresources.apply_dpi: use "round" instead of "ceil"

Given an outcome of e.g. 1.01 its more sane to use 1 than 2, especially
for `border_width`.

This uses the method from `wibox.layout.flex`.

Closes https://github.com/awesomeWM/awesome/pull/389.
This commit is contained in:
Daniel Hahler 2015-08-02 21:51:06 +02:00
parent 88ba798ae3
commit b028a0ba1d

View file

@ -10,6 +10,7 @@
-- Grab environment
local print = print
local awesome = awesome
local floor = math.floor
local xresources = {}
@ -91,9 +92,9 @@ end
--- Compute resulting size applying current DPI value (optionally per screen).
-- @tparam number size Size
-- @tparam[opt] integer s The screen.
-- @treturn integer Resulting size (using `math.ceil`).
-- @treturn integer Resulting size (rounded to integer).
function xresources.apply_dpi(size, s)
return math.ceil(size/96*xresources.get_dpi(s))
return floor(size/96*xresources.get_dpi(s) + 0.5)
end
return xresources