We need a nice icon for the server monitoring app

[schiessle]

Would be good to have a nice icon for the server monitoring app in the apps listing

cc @nextcloud/designers

[MariusBluem]

What do you think about somehting like this:
http://fontawesome.io/icon/pie-chart/

[enoch85]

Other suggestions:

https://www.iconfinder.com/icons/638239/business_graph_monitor_report_icon#size=128
https://www.iconfinder.com/icons/1031365/beat_beats_heartbeat_monitor_screen_icon#size=128
https://www.iconfinder.com/icons/399143/graph_health_monitor_screen_tv_icon#size=128

[enoch85]

I can donate the cost btw. $1, take it from my Paypal account :D

[MorrisJobke]

I think we will not buy an icon, but will do a simple one ourselves ... cc @jancborchardt

[MorrisJobke]

I think @Bugsbane also did some icons in the past

[schiessle]

Decided to move it to the admin page, so no icon for the apps menu needed. But would probably still be nice to have a icon for the apps listing

[Bugsbane]

I'd rather make a new icon myself than buy one personally. Can you show me a screenshot or at least a description of what the app actually monitors, so I have a better conceptual understanding of what we're trying to capture?

[Bugsbane]

Actually, I had a bit of time, so I ran something up while I was waiting, based on the idea of a heartbeat, or monitoring the wellbeing of your install:

app icon:

...and in a mocked up screenshot (I replaced the generic cog icon for the "First Run Wizard" app, with the new potential Server Monitor app icon.

Let me know what you think, and if I'm on the right track.

[schiessle]

@Bugsbane looks really nice! 👍 Do you want to create a pull request to add the icon? Thanks!

[Bugsbane]

Done. Sorry, I accidentally committed straight to master, instead of forking/branching first. Anyway, take a look and let me know if you need anything (like the color) changed.

[LukasReschke]

Done. Sorry, I accidentally committed straight to master, instead of forking/branching first.

I protected the branches now so that this won't be possible anymore in the future.

[Bugsbane]

For future reference @LukasReschke , what's the protocol for committing? Branch, commit and PR? Fork, commit and PR? (In which case what happens when you're working on multiple things and you want to commit the changes for one?) Fork, branch your fork and then PR?

[LukasReschke]

Branch, commit and PR <- this. :-)

[MorrisJobke]

backported to stable10: stable10 5b6dd3c

[jancborchardt]

@Bugsbane hehe, awesome icon work again! :)

[Bugsbane]

Thanks! I wish I could say I planned this consciously, but I just noticed the icon has a large "N" hidden in it (as in "N" for Nextcloud 😉 )

[jancborchardt]

@Bugsbane :DD