Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Cooja] (Multi-)Mote visualization updates #760

Merged
merged 9 commits into from Aug 1, 2014

Conversation

ejoerns
Copy link
Contributor

@ejoerns ejoerns commented Jul 31, 2014

This introduces several fixes and extensions to Coojas Visualizer plugin.

  • Fix: Selected motes were highlighted only if UDGMVisualizer was active.
    Now a Mote is always highlighted with a blue circle and a semi-transparent gray overlay which still allows other skins to modify the mote fill color
  • Enable multi-mote deletion
  • Mote context menus handle group selections
  • Key control (ESC aborts any action, DEL removes mote(s)
  • Several visual and behavior fixes (see commits)

Enrico Joerns added 9 commits July 11, 2014 00:41
Motes were highlighted only by the UDGMVisualizerSkin before.
Now highlighting moved from individual VisualizerSkin to Visualizer.

A selected mote is highlighted with a blue circle
and a semi-transparent gray overlay.
Previously if multiple motes were selected
and the context menu of a non-selected mote
was used to delete, not the clicked mote
but the other selected ones were removed.

This behavior is fixed so that if another
mote is clicked only this is deleted.
… mote menu for move, too

Note that this does not yet implement the correct behavior of beginMoveReques()
…action

and reses actions state and cursor appearance

Also renamed UNKNOWN state to NONE as it is a more adequate name for the new behavior
…ESCAPE key and delete motes using DELETE key
This also fixes visualization issues such as
remaining transmission range underlays for removed motes
@fros4943
Copy link
Member

fros4943 commented Aug 1, 2014

Looks great! 👍

fros4943 added a commit that referenced this pull request Aug 1, 2014
[Cooja] (Multi-)Mote visualization updates
@fros4943 fros4943 merged commit 648d357 into contiki-os:master Aug 1, 2014
@ejoerns ejoerns deleted the pull-req/cooja-multi-mote-vis branch September 10, 2014 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants