From ff6983f4ddc11fe9710b0cbc7474aef7399b1dd8 Mon Sep 17 00:00:00 2001
From: user <user@user-ThinkPad-X60.(none)>
Date: Tue, 16 Jun 2015 01:29:15 -0400
Subject: [PATCH] fixed strange regression with gop scalar (or garray) having a
 crazy bbox because of scaling in the parent.

---
 pd/nw/pdgui.js | 5 +++--
 pd/nw/todo.txt | 2 ++
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/pd/nw/pdgui.js b/pd/nw/pdgui.js
index 0df09bcd8..3db15e1f2 100644
--- a/pd/nw/pdgui.js
+++ b/pd/nw/pdgui.js
@@ -2871,16 +2871,17 @@ function gui_scalar_draw_select_rect(cid, tag, state, x1, y1, x2, y2, basex, bas
     // means all the coordinate parameters are in the screen position. But we need
     // the coords relative to the scalar's x/y-- hence we subtract the scalar's basex/basey
     // from the coords below
+    // Additionally, we're not actually drawing the rect here.  It's drawn
+    // as part of the scalar_vis function.  We're merely changing its coords
+    // and size
     //gui_post("drawselectrect: " + x1 + " " + y1 + " " + x2 + " " + y2 + " " + basex + " " + basey);
     var g = get_gobj(cid, tag);
     var b = g.querySelector('.border');
     configure_item(b, {
-        display: (state ? 'inline' : 'none'),
         x: (x1 - basex),
         y: (y1 - basey),
         width: x2 - x1,
         height: y2 - y1,
-//        stroke: 'blue',
     });
 }
 
diff --git a/pd/nw/todo.txt b/pd/nw/todo.txt
index cb3c90e14..5e10a2828 100644
--- a/pd/nw/todo.txt
+++ b/pd/nw/todo.txt
@@ -190,6 +190,8 @@ Everything else: (A [x] means we've fixed it)
     yet.  Perhaps there's a better way to do it, without triggering a bunch
     of layouts.
 [ ] implement pdtk_canvas_editval and magicglassval
+[ ] move crispEdges value to CSS
+
 Crashers
 --------
 
-- 
GitLab