Closed GoogleCodeExporter closed 9 years ago
Taking ownership.
Original comment by kevin.a....@gmail.com
on 21 Jan 2012 at 8:43
Solution in revision f92c44bd7853, please review.
Original comment by kevin.a....@gmail.com
on 23 Jan 2012 at 7:27
Kevin, it seems that your implementation of Issue 196 broke the functionality
Andrew implemented in Issue 160 -- shift click selects multiple nodes. Shift
click works in revision e57f7536de7e, and stopped working in revision
ebe6694573cc. You are both working on the same JS logic, so you both have to
understand each other's code to integrate with it cleanly. Kevin, please
implement a fix for 160 as part of this issue.
Original comment by bestchai
on 23 Jan 2012 at 5:29
Fixed this bug in revision a5e9d5ad66d9, please review.
Original comment by kevin.a....@gmail.com
on 23 Jan 2012 at 8:28
Two new bugs:
1. The "View Paths" button in the ModelTab is never activated. The function to
activate it is ModelTab.toggleViewPathsButton. It's never activated because
ModelTab.addSelectedNode and ModelTab.removeSelectedNode never seem to fire.
Relevant code in ModelGraph.createGraph -- $wnd.addSelectedNode is set there,
and this function should be called from JS logic.
2. When doing a shift-click, or just a single click, all nodes of the node type
being clicked are highlighted and remain highlighted. Instead, just the node of
that type should remain highlighted after the click.
Original comment by bestchai
on 23 Jan 2012 at 8:50
Fixed these two bugs in revision 6654f736e8bb, please review.
Original comment by kevin.a....@gmail.com
on 24 Jan 2012 at 7:35
Ok, the visual behavior looks good, but there is another logic bug related to
Issue 160 -- the "View Paths" button never returns any paths.
Here is a test:
1. Switch to revision e57f7536de7e (i.e., "hg up e57f7536de7e")
2. Run the 2PC example
3. Open a JavaScript console window (in Chrom this is under View -> Developer
-> JavaScript Console)
4. Select the tx_prepare node in the model.
5. Click on "View Paths"
6. Observe that the console outputs 5 traces -- TRACE ID 0 through TRACE ID 4.
If you try the same test at the head of the Issue201 branch, then you'll notice
an error at top of page that says that no paths were found through selected
nodes.
Original comment by bestchai
on 24 Jan 2012 at 8:19
I wasn't able to reproduce this bug every single time. But I believe I have the
correct fix though. Solution in revision a5061fc8e4c8, please review.
Original comment by kevin.a....@gmail.com
on 25 Jan 2012 at 12:29
That seems to have fixed it for me. Code looks good. Closing issue, merged
branch into default with revision e41bb930c923.
Original comment by bestchai
on 25 Jan 2012 at 2:01
Original issue reported on code.google.com by
bestchai
on 20 Jan 2012 at 6:26