render, pager: fix canvas background painting order

We were painting the background box in render for dump mode, but this
conflicted with the standard requirement that the canvas be painted
before other elements.  So now we handle this directly in the pager.

Conveniently enough, this also fixes the issue of canvas color adding
pointless spacing to pages (which often made the selection feature less
useful.)

1 files changed, 8 insertions, 7 deletions

diff --git a/src/css/render.nim b/src/css/render.nim
index 565b7c9e..d613fb9c 100644
--- a/src/css/render.nim
+++ b/src/css/render.nim
@@ -450,13 +450,14 @@ proc renderBlock(grid: var FlexibleGrid; state: var RenderState;
         #TODO bgimage
         # note: this eats the alpha
         state.bgcolor = bgcolor
-      let ix = toInt(offset.x)
-      let iy = toInt(offset.y)
-      let e = offset + box.state.size
-      let iex = toInt(e.x)
-      let iey = toInt(e.y)
-      grid.paintBackground(state, bgcolor, ix, iy, iex, iey, box.element,
-        bgcolor0.a, box.render.clipBox)
+      else:
+        let ix = toInt(offset.x)
+        let iy = toInt(offset.y)
+        let e = offset + box.state.size
+        let iex = toInt(e.x)
+        let iey = toInt(e.y)
+        grid.paintBackground(state, bgcolor, ix, iy, iex, iey, box.element,
+          bgcolor0.a, box.render.clipBox)
     if box.computed{"background-image"} != nil:
       # ugly hack for background-image display... TODO actually display images
       const s = "[img]"