ErikWegner / imoin

Monitor your nagios or icinga instance in Firefox, Chrome, Opera, Vivaldi and Edge.
https://imoin.qwirl.eu/
BSD 2-Clause "Simplified" License
4 stars 0 forks source link

Make output more compact and efficient #16

Closed majurgens closed 3 years ago

majurgens commented 6 years ago

Compare this imoin screenshot: image

vs the old plugin: image

If you assume that filtering options become available shortly, then the older output is still slightly more compact. For example,

1) I don't think that including the host check results, just because there are services on that host included, adds value. Removing that would save more space. Perhaps some people want it. Perhaps make that an option.

2) I'm not sure that we need to have different font sizes to make headings more visible if the data was columnar. like the older version. That would save space as well as making it quicker to read.

3) I suspect that most nagios users are used to the main alarm colours and hence the space currently used to show that status could be saved by just simply making the alarm colour as the text background colour (like the old plugin). I'm not 100% sure if we need the status field (like the old one) at all.

4) Adding the duration timestamp would also be pretty useful (like the old one)

Compare this imoin status icon: image

vs the old plugin: image

5) now, i don't know if this is possible with the new Firefox changes, but, while the Imoin status icon is more compact than the old one, the old one manages to convey more information before the user has to interact with it. If possible, can we please show the numbers of different alarm types in the status icon.

majurgens commented 6 years ago

For "help wanted" - what language are you coding in?

ErikWegner commented 6 years ago

The part that produces the panel content is written in plain JavaScript (see html/panel.js). The main code is written in TypeScript.

If the panel should allow switching to different kinds of display templates, using a templating engine would be nice.

For the panel icon:

What help is appreciated?

majurgens commented 6 years ago

Ok, I've done a bit of coding in javascript. Typescript does not look much of a leap.

For the panel icon: I think those limitations severely restrict what I was suggesting in that area.

Help: I could probably do some of that but I would need help on getting the development environment setup, checking out, committing (push, pull) etc (I have hardly done any thing with Github).

ErikWegner commented 6 years ago

Please contact me at https://ewus.de/en/contact and we will start with a direkt mail exchange.

ErikWegner commented 6 years ago

For the most easy start, you can directly edit the script and style inside this codepen:

https://codepen.io/erikwegner/pen/RywqjR

ErikWegner commented 6 years ago

To work on a design, check out the vue-branch and run panel-dev.

drook commented 6 years ago

I think the current layout also needs improvements:

So, concluding, I think the output in the "errors/warnings" tab should be like this, grouped within the instance name (because currenlty it's hard to distinguish the instances):

instance name hostname | servicename | duration | extended output

PING OK - Packet loss = 0%, RTA = 0.26 ms

I think this should be fixed too. May be this is merely the issue with "nagios HTML pаrsing" method, but still.

And thanks a lot for doing such a cool work.

drook commented 6 years ago

Actually changing only one CSS property makes the output far more easy-readable (inline displaying):

.hostcheckinfo,
.service .info {
    color: #888;
    padding-left: 1ex;
    display: inline;
}
ErikWegner commented 6 years ago

Option added to beta version

ErikWegner commented 5 years ago

A very simple approach to use jsx as templates: https://codepen.io/erikwegner/project/full/ZqNNOw/ Feel free to fork the project and suggest a layout, have a look at hostrow.jsx and servicerow.jsx.

ErikWegner commented 3 years ago

No layout sugested, closing now