remove some old sys_vgui commands

pd/nw/css/c64.css

@@ -195,6 +195,11 @@ text {
     display: inline;
 }
 
+/* text inside selected objects */
+:not(.gop).selected text {
+    fill: #cc9933;
+}
+
 /* for an object that didn't create */
 .obj .border.broken_border {
     fill: #3e32a2;

pd/nw/css/default.css

@@ -226,6 +226,11 @@ text {
     display: inline;
 }
 
+/* text inside selected objects */
+:not(.gop).selected text {
+    fill: blue;
+}
+
 /* for an object that didn't create */
 .obj .border.broken_border {
     fill: #f7f7f7;

pd/nw/css/extended.css

@@ -188,6 +188,11 @@ text {
     display: inline;
 }
 
+/* text inside selected objects */
+:not(.gop).selected text {
+    fill: blue;
+}
+
 /* for an object that didn't create */
 .obj .border.broken_border {
     fill: #f7f7f7;

pd/nw/css/inverted.css

@@ -198,6 +198,11 @@ text {
     display: inline;
 }
 
+/* text inside selected objects */
+:not(.gop).selected text {
+    fill: #ffff00;
+}
+
 /* for an object that didn't create */
 .obj .border.broken_border {
     fill: #002222;

pd/nw/css/strongbad.css

@@ -196,6 +196,11 @@ text {
     display: inline;
 }
 
+/* text inside selected objects */
+:not(.gop).selected text {
+    fill: green;
+}
+
 /* for an object that didn't create */
 .obj .border.broken_border {
     fill: transparent;

pd/nw/css/subdued.css

@@ -190,6 +190,11 @@ text {
     display: inline;
 }
 
+/* text inside selected objects */
+:not(.gop).selected text {
+    fill: blue;
+}
+
 /* for an object that didn't create */
 .obj .border.broken_border {
     fill: transparent;

pd/nw/css/vanilla.css

@@ -171,6 +171,11 @@ text {
     stroke: black;
 }
 
+/* text inside selected objects */
+:not(.gop).selected text {
+    fill: black;
+}
+
 .graph .border {
     stroke: black;
     fill: none;

pd/nw/css/vanilla_inverted.css

@@ -182,6 +182,11 @@ text {
     stroke: white;
 }
 
+/* text inside selected objects */
+:not(.gop).selected text {
+    fill: white;
+}
+
 .graph .border {
     stroke: white;
     fill: none;

pd/nw/index.js

@@ -79,7 +79,6 @@ function open_external_doc(target) {
 }
 
 function nw_window_focus_callback() {
-    pdgui.post("pd window was focused");
     if (process.platform === "darwin") {
         nw_create_pd_window_menus(gui, window);
     }

pd/src/g_editor.c

@@ -546,7 +546,8 @@ void glist_select(t_glist *x, t_gobj *y)
         gobj_select(y, x, 1);
         c_selection = x;
 
-        sys_vgui("pdtk_canvas_update_edit_menu .x%lx 1\n", x);
+        /* Not sure if this is still needed */
+        //sys_vgui("pdtk_canvas_update_edit_menu .x%lx 1\n", x);
         canvas_draw_gop_resize_hooks(x);
     }
     //fprintf(stderr,"select done\n");
@@ -646,8 +647,9 @@ void glist_deselect(t_glist *x, t_gobj *y)
         }
         if (fixdsp)
             canvas_resume_dsp(1);
-        if (!x->gl_editor->e_selection)
-            sys_vgui("pdtk_canvas_update_edit_menu .x%lx 0\n", x);
+        /* Not sure if this is still needed */
+        //if (!x->gl_editor->e_selection)
+        //    sys_vgui("pdtk_canvas_update_edit_menu .x%lx 0\n", x);
         canvas_draw_gop_resize_hooks(x);
     }
     //reenter = 0;

pd/src/g_rtext.c

@@ -544,24 +544,27 @@ void rtext_displace(t_rtext *x, int dx, int dy)
     //    x->x_tag, dx, dy);
 }
 
-/* Not sure if this is still used */
+/* This is no longer used-- we do this with CSS now. But keeping the code
+   here until we test a bit more. */
 void rtext_select(t_rtext *x, int state)
 {
-    t_glist *glist = x->x_glist;
-    t_canvas *canvas = glist_getcanvas(glist);
-    if (glist_istoplevel(glist))
-        sys_vgui(".x%lx.c itemconfigure %s -fill %s\n", canvas, 
-            x->x_tag, (state? "$pd_colors(selection)" : "$pd_colors(text)"));
-    if (x->x_text->te_pd->c_wb && x->x_text->te_pd->c_wb->w_displacefnwtag)
-    {
-        if (state)
-            sys_vgui(".x%lx.c addtag selected withtag %s\n",
-                   glist_getcanvas(glist), x->x_tag);
-        else
-            sys_vgui(".x%lx.c dtag %s selected\n",
-                   glist_getcanvas(glist), x->x_tag);
-    }
-    canvas_editing = canvas;
+//    t_glist *glist = x->x_glist;
+//    t_canvas *canvas = glist_getcanvas(glist);
+//    if (glist_istoplevel(glist))
+//        sys_vgui(".x%lx.c itemconfigure %s -fill %s\n", canvas, 
+//            x->x_tag, (state? "$pd_colors(selection)" : "$pd_colors(text)"));
+//    if (x->x_text->te_pd->c_wb && x->x_text->te_pd->c_wb->w_displacefnwtag)
+//    {
+//        if (state)
+//            sys_vgui(".x%lx.c addtag selected withtag %s\n",
+//                   glist_getcanvas(glist), x->x_tag);
+//        else
+//            sys_vgui(".x%lx.c dtag %s selected\n",
+//                   glist_getcanvas(glist), x->x_tag);
+//    }
+    /* Not sure the following is needed anymore either-- commenting it
+       out to test what (if any) side-effects there are */
+//    canvas_editing = canvas;
 }
 
 void rtext_activate(t_rtext *x, int state)

pd/src/g_text.c

@@ -1639,10 +1639,10 @@ static void text_select(t_gobj *z, t_glist *glist, int state)
                              glist_istoplevel(glist))
                 {
                     /* Not sure yet what this is doing... */
-                    sys_vgui(".x%lx.c itemconfigure %sR -strokewidth 1 "
-                             "-strokedasharray {} "
-                             "-fill $pd_colors(box)\n",
-                        glist_getcanvas(glist), rtext_gettag(y));
+                    //sys_vgui(".x%lx.c itemconfigure %sR -strokewidth 1 "
+                    //         "-strokedasharray {} "
+                    //         "-fill $pd_colors(box)\n",
+                    //    glist_getcanvas(glist), rtext_gettag(y));
                 }
 
                 t_object *ob = pd_checkobject(&x->te_pd);
@@ -1678,12 +1678,13 @@ static void text_select(t_gobj *z, t_glist *glist, int state)
                 if (pd_class(&x->te_pd) == text_class &&
                     x->te_type != T_TEXT)
                 {
-                    /* This might be for broken objects... */
-                    sys_vgui(".x%lx.c itemconfigure %sR -strokewidth 2 "
-                        "-strokelinecap projecting -strokedasharray {2 3} "
-                        "-fill %s\n",
-                        glist_getcanvas(glist), 
-                        rtext_gettag(y), invalid_fill);
+                    /* This is for broken objects, but we're doing
+                       it in CSS now... */
+                    //sys_vgui(".x%lx.c itemconfigure %sR -strokewidth 2 "
+                    //    "-strokelinecap projecting -strokedasharray {2 3} "
+                    //    "-fill %s\n",
+                    //    glist_getcanvas(glist),
+                    //    rtext_gettag(y), invalid_fill);
                 }
 
                 t_object *ob = pd_checkobject(&x->te_pd);