From 3c848b65fd3c481b655a3e461e62e88f8137fec5 Mon Sep 17 00:00:00 2001
From: Jonathan Wilkes <jon.w.wilkes@gmail.com>
Date: Sun, 31 Jan 2016 13:28:42 -0500
Subject: [PATCH] update todo
---
pd/nw/todo.txt | 15 ++++++++++-----
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/pd/nw/todo.txt b/pd/nw/todo.txt
index b498c6169..6d6ba98db 100644
--- a/pd/nw/todo.txt
+++ b/pd/nw/todo.txt
@@ -70,13 +70,18 @@ Everything else: (A [x] means we've fixed it)
[ ] in draw_vis, remove all the coords logic and let svg_togui do that work.
In svg_new, leverage the "d", "points", etc. methods to do the same.
[ ] remove the draw_vis warning/code about needing 2 points to draw
-[ ] make gui_menu_close message less hacky (currently includes patch args in
+[x] make gui_menu_close message less hacky (currently includes patch args in
the middle)
[x] make dialogs with [yes/no] instead of [cancel/ok]
-[ ] draw graph inside gobj group
-[ ] merge gui_text_select and gui_text_deselect
-[ ] in function gui_cord_inspector_update, use parameter "moved" to flash text
-[ ] cord_inspector_flash doesn't seem to be deterministic-- if you move among
+[x] draw graph inside gobj group
+ Note: good idea, but this will be extremely difficult to get right given
+ the C API for specifying coords, glists, etc.
+[x] merge gui_text_select and gui_text_deselect
+ Note: doesn't seem necessary.
+[x] in function gui_cord_inspector_update, use parameter "moved" to flash text
+ Note: not sure I understand this one. But if this is a bug we'll run into
+ later...
+[x] cord_inspector_flash doesn't seem to be deterministic-- if you move among
several cords very fast it will try to flash a non-existent svg item.
(Probably want to either make a permanent item in the pd_canvas.html for it,
or hard-code a separate div.)
--
GitLab