naughty: Fix replaces_id + changing colors (#2041)

This fixes a regression introduced in be29ee6768. This commit changed
naughty to reuse an already-existing wibox when replaces_id is used,
instead of creating a new wibox. However, some of the properties that
are set only when creating a wibox were ignored due to this.

Fixes: https://github.com/awesomeWM/awesome/issues/2040
Signed-off-by: Uli Schlachter <psychon@znc.in>

lib/naughty/core.lua

@@ -775,18 +775,18 @@ function naughty.notify(args)
     notification.iconbox = iconbox
 
     -- create container wibox
-    notification.box = wibox({ fg = fg,
+    if not reuse_box then
-                               bg = bg,
+        notification.box = wibox({ type = "notification" })
-                               border_color = border_color,
+    else
-                               border_width = border_width,
-                               shape_border_color = shape and border_color,
-                               shape_border_width = shape and border_width,
-                               shape = shape,
-                               type = "notification" })
-
-    if reuse_box then
         notification.box = reuse_box
     end
+    notification.box.fg = fg
+    notification.box.bg = bg
+    notification.box.border_color = border_color
+    notification.box.border_width = border_width
+    notification.box.shape_border_color = shape and border_color
+    notification.box.shape_border_width = shape and border_width
+    notification.box.shape = shape
 
     if hover_timeout then notification.box:connect_signal("mouse::enter", hover_destroy) end