Availability-map #1464

Merged
merged 3 commits into from Jul 17, 2015

Projects

None yet

4 participants

@Rosiak
Contributor
Rosiak commented Jul 16, 2015

This PR consists 3 things:

  • A availability/heat map. This is just a VERY basic version. There is room for improvement, but hey gotta start somewhere!
    http://snag.gy/2Eo5g.jpg
  • Introduction of the Maps submenu, the idea is to "move" the maps "links" to here.
    This PR only introduces the availability/heat map entry in the submenu.
    http://snag.gy/U4iVI.jpg
  • Removed the old "Network Map" menu shortcut, since it just linked to the overview page.
@Rosiak Rosiak Availability-map
- Availability-map
- Maps submenu
- Remove old network-map menu entry
eb87668
@f0o
Member
f0o commented Jul 17, 2015

Awesome work!

Should the Network-Map link from the Devices-dropdown also be moved to the Maps submenu?
Is linking to the device-page rather than the uptime-graph more useful?

Could you use generate_url() on line 37 of html/pages/availability-map.inc.php please ?

@f0o f0o self-assigned this Jul 17, 2015
@f0o f0o added the WebUI label Jul 17, 2015
@Rosiak Rosiak Minor corrections
- Made some changes according to f0o’s recommendation
8d09a66
@Rosiak
Contributor
Rosiak commented Jul 17, 2015

Thx for input @f0o just made the changes.

@f0o
Member
f0o commented Jul 17, 2015

I'm tagging @laf for a comment.

👍, I'd merge this PR :)

@laf
Member
laf commented Jul 17, 2015

I think if we can change the device map link in print-menubar to be Network Map then all good

@Rosiak Rosiak Change submenu entry name
- Device -> Network
ef23f47
@laf laf merged commit 5b53597 into librenms:master Jul 17, 2015

2 checks passed

Auto-Deploy Build finished.
Details
Scrutinizer No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment