RFC: Set c.border_width to 0 when maximized about awesome HOT 22 CLOSED

-- No border for maximized clients

client.connect_signal("focus",
  function(c)
    if c.maximized_horizontal == true and c.maximized_vertical == true then
      c.border_color = beautiful.border_normal
    else
      c.border_color = beautiful.border_focus
    end
  end)

client.connect_signal("unfocus",
  function(c) c.border_color = beautiful.border_normal
  end)

-- Arrange signal handler
for s = 1, screen.count() do screen[s]:connect_signal("arrange", 
  function ()
    local clients = awful.client.visible(s)
    local layout  = awful.layout.getname(awful.layout.get(s))

    if #clients > 0 then -- Fine grained borders and floaters control
      for _, c in pairs(clients) do -- Floaters always have borders
        if awful.client.floating.get(c) or layout == "floating" then
          c.border_width = beautiful.border_width

        -- No borders with only one visible client
        elseif #clients == 1 or layout == "max" then
          c.border_width = 0
        else
          c.border_width = beautiful.border_width
        end
      end
    end
  end)
end

-- Handle border sizes of clients.
do
    local beautiful = require("beautiful")
    local layout = require("awful.layout")
    local screen = require("awful.screen")
    local timer = require("gears.timer")

    function client.automatic_border_width(c, s)
        s = s or c.screen
        local layout_name = layout.getname(layout.get(s))
        -- No borders with only one humanly visible client
        if c.marked then
            -- Handled separately.
            return
        elseif c.maximized then
            -- NOTE: also handled in focus, but that does not cover maximizing from
            -- a tiled state (when the client had focus).
            c.border_width = 0
        elseif c.floating or layout_name == "floating" then
            c.border_width = beautiful.border_width
        elseif layout_name == "max" or layout_name == "fullscreen" then
            c.border_width = 0
        else
            -- No border for a single tiled client.
            -- TODO: Needs to take into account "master_fill_policy" == "mwfact"?!
            local tiled = client.tiled(s)
            -- local c_tag = c:tags()[1]
            if #tiled == 1 then  -- and awful.tag.getproperty(c_tag) ~= 'master_fill_policy' then -- and c == tiled[1] then
                c.border_width = 0
                -- XXX: SLOW!  Not necessary anymore?!
                -- awful.client.moveresize(0, 0, 2, 0, tiled[1])
            else
                c.border_width = beautiful.border_width
            end
        end
    end

    local client_border_width_for_s = function(s)
        timer.delayed_call(function (c)
          local clients = client.visible(s)
          for _, c in pairs(clients) do
              client.automatic_border_width(c, s)
            print(c.name, c.border_width)
          end
        end)
    end
    capi.screen.connect_for_each_screen(function(s)
        screen[s]:connect_signal("arrange", client_border_width_for_s)
    end)

    local client_border_maximized = function(c)
        c.border_width = not c.maximized and beautiful.border_width or 0
    end
    capi.client.connect_signal('property::maximized', client_border_maximized)
    -- client.connect_signal('property::border_width', client_set_border_width)
end

-- Fine grained borders and floaters control {{{
local client_border_color = function (c)
    -- if awful.client.ismarked(c) then
    if c.marked then
      if c == capi.client.focus and beautiful.border_marked_focus then
        c.border_color = beautiful.border_marked_focus
      else
        c.border_color = beautiful.border_marked
      end
    elseif c == capi.client.focus then
      c.border_color = beautiful.border_focus
    else
      c.border_color = beautiful.border_normal
    end
end
client.connect_signal("focus",    client_border_color)
client.connect_signal("unfocus",  client_border_color)
client.connect_signal("marked",   client_border_color)
client.connect_signal("unmarked", client_border_color)

    local client_border_width_for_s = function(s)
        timer.delayed_call(function (c)
          local clients = client.visible(s)
          for _, c in pairs(clients) do
              client.automatic_border_width(c, s)
            print(c.name, c.border_width)
          end
        end)
    end

local function on_client_focus(c)
  local t = choose_tag(c)
  local layout = t.layout
  local num_tiled = get_num_tiled(t)

  if persistent.titlebar.get() and (
    num_tiled > 1 or (
      num_tiled > 0 and t.master_fill_policy ~= 'expand'
    )
  ) then
    clog("F: tile: titlebars enabled explicitly")
    titlebar.make_titlebar(c, beautiful.actionless_titlebar_bg_focus, beautiful.titlebar_shadow_focus)
  elseif c.maximized then
    clog("F: maximized")
    titlebar.remove_border(c)
  elseif c.floating then
    clog("F: floating client")
    titlebar.make_titlebar(c, beautiful.actionless_titlebar_bg_focus, beautiful.titlebar_shadow_focus)
  elseif layout == awful.layout.suit.floating then
    clog("F: floating layout")
    titlebar.make_titlebar(c, beautiful.actionless_titlebar_bg_focus, beautiful.titlebar_shadow_focus)
  elseif num_tiled > 1 then
    clog("F: multiple tiling clients")
    c.border_width = beautiful.border_width
    titlebar.make_border(c, beautiful.actionless_titlebar_bg_focus, beautiful.titlebar_shadow_focus)
  elseif num_tiled == 1 then
    if t.master_fill_policy == 'expand' and screen.count() == 1 then
      clog("F: one tiling client: expand")
      titlebar.remove_border(c)
    else
      clog("F: one tiling client")
      c.border_width = beautiful.border_width
      titlebar.make_border(c, beautiful.actionless_titlebar_bg_focus, beautiful.titlebar_shadow_focus)
    end
  else
    clog("F: zero tiling clients -- other tag?")
    return on_client_unfocus(c)
  end

  c.border_color = beautiful.border_focus
end

  awful.tag.object.get_gap = function(t)
    t = t or awful.screen.focused().selected_tag
    if not t then return end
    local num_tiled = get_num_tiled(t)
    if num_tiled == 1 and t.master_fill_policy == "expand" then
      return 0
    end
    return awful.tag.getproperty(t, "useless_gap") or beautiful.useless_gap or 0
  end