mershon-enterprises / wh-hero

Interactive data collection and visualization of WirelessHART mesh data
https://mershon.enterprises
GNU General Public License v3.0
0 stars 0 forks source link

Feature create mesh view #19

Closed darinh closed 9 years ago

darinh commented 9 years ago

@kevinmershon please review -- Blocked -- I branched off of feature-create-measurement-history-view, so that one needs to be reviewed and merged before you'll get an accurate diff for this one.

kevinmershon commented 9 years ago

@darinh For this screen I really don't think it's a good idea to have Transmitter 1, 2, 3, listed below because Gateways can have up to 200 transmitters.

In the long run, I'd prefer to have a pinch-zoom functionality on the D3 graph, and a search-by-name box that will let the user pick the node and show it in the mesh, with text labels under it and its neighbors for the transmitter name and MAC address. For the ISA show, though, I'd just be happy with pinch-to-zoom and text labels for all nodes.

darinh commented 9 years ago

@kevinmershon thanks for the feedback. One issue I ran into with testing was that swiping right and panning right both revealed the menu, which could be problematic if there will be zooming and panning. Here are two options that could solve this problem:

Requirements:

kevinmershon commented 9 years ago

Yeah, let's just go with a hamburger menu to fix the swiping issue. It's easy and standard.

kevinmershon commented 9 years ago

@darinh This is definitely closer to what I had in mind. Instead of worrying about pinch-to-zoom, could we superimpose +/- buttons, and use gestures for panning?

darinh commented 9 years ago

This is complete apart from the zooming / panning. Scroll was disabled, which makes dragging the nodes easier. Selecting a node from the dropdown list will highlight it in the graph and display detail.

@kevinmershon please review this again when you have a chance.

kevinmershon commented 9 years ago

Looks great.