search

farahshun / scxmlgui

Automatically exported from code.google.com/p/scxmlgui
0 stars 0 forks source link

Cannot create History Nodes #51

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Open Editor
2. File->New SCXML
3. Right click in editor Window
4. Add node
5. Right Click on new node
6. Context menu does not show "Toggle deep history" or "Toggle shallow history" 
options. (see attached HistoryNodeBug.png)

What is the expected output? What do you see instead?
- From the website, it suggests that the menu options are available and that 
the new node can be changed to a history node.

Note: If the SCXML data already contains a history node, then the node is 
correctly displayed in the editor in light/dark purple as appropriate to the 
deep/shallow attribute.

What version of the product are you using? On what operating system?
SVN revision: 170
Windows 7 Enterprise 32 bit.
Java Version 7 Update 9 (build 1.7.0_09-b05)

Original issue reported on code.google.com by miketrah...@gmail.com on 11 Jan 2013 at 9:09

Attachments:

GoogleCodeExporter commented 9 years ago 
i interpreted the w3c working draft to say that history nodes cannot be 
children of the root container node but i may be wrong. I disabled the check in 
version 172. Let me know if that works for you and i'll mark this as fixed. 
Does the scxml interpreter digest and use correctly top level history nodes 
like the one you created above?

thanks

Original comment by fmorbini on 14 Jan 2013 at 10:55

GoogleCodeExporter commented 9 years ago 
It makes total sense to not allow History nodes at the root container as 
previous to your "fix" in version 172. Using the steps below (still using 
version 170) I can create History Nodes correctly:
1. File->New SCXML
2. Create new node
3. Select the new node and Toggle Cluster.
4. Select the new node and add a new child node
5. Select the new node and Toggle Deep/Shallow History is now available.

My recommendation is to leave the behaviour as per version 170.

Thanks,

Mike

Original comment by miketrah...@gmail.com on 15 Jan 2013 at 9:13

GoogleCodeExporter commented 9 years ago 
marking as invalid. Reverted tmp changes in r172.

Original comment by fmorbini on 15 Jan 2013 at 5:44