Create viewport topology module
Bug #1090479 reported by
Gary Poster
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| juju-gui |
Fix Released
|
High
|
Benjamin Saller | ||
Bug Description
Separate out the viewport code from the mongo module created from bug 1077067.
The module will still be subscribed to aggregated db events.
No new tests needed: only port existing tests (see bug 1083933 for new tests of legacy viewport code).
Related branches
lp:~bcsaller/juju-gui/viewport
- Juju GUI Hackers: Pending requested
-
Diff: 3047 lines (+1092/-1156)27 files modifiedapp/app.js (+10/-6)
app/assets/javascripts/d3-components.js (+10/-7)
app/modules-debug.js (+6/-1)
app/templates/overview.handlebars (+1/-1)
app/views/charm-panel.js (+3/-2)
app/views/environment.js (+13/-13)
app/views/login.js (+1/-3)
app/views/topology/mega.js (+0/-59)
app/views/topology/panzoom.js (+71/-61)
app/views/topology/topology.js (+57/-61)
app/views/topology/viewport.js (+43/-100)
app/views/utils.js (+9/-7)
lib/views/stylesheet.less (+5/-0)
package.json (+2/-2)
test/index.html (+53/-52)
test/test_app.js (+41/-18)
test/test_app_hotkeys.js (+47/-45)
test/test_d3_components.js (+0/-1)
test/test_environment_view.js (+7/-7)
test/test_login.js (+3/-4)
test/test_model.js (+2/-1)
test/test_notifications.js (+469/-454)
test/test_notifier_widget.js (+98/-95)
test/test_panzoom.js (+15/-26)
test/test_service_config_view.js (+1/-1)
test/test_topology.js (+1/-0)
undocumented (+124/-129)
| Changed in juju-gui: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| Changed in juju-gui: | |
| assignee: | nobody → Benjamin Saller (bcsaller) |
| milestone: | none → 0.1.3+build.302 |
| status: | Triaged → In Progress |
| Changed in juju-gui: | |
| status: | In Progress → Fix Released |
To post a comment you must log in.
