icingaweb2-module-businessprocess icon indicating copy to clipboard operation
icingaweb2-module-businessprocess copied to clipboard

Add showFullscreen and showCompact for dashboard

Open NavidSassan opened this issue 3 years ago • 7 comments

Re-implementation of https://github.com/Icinga/icingaweb2-module-businessprocess/pull/294.

NavidSassan avatar Sep 29 '22 09:09 NavidSassan

Thank you for your pull request. Before we can look at it, you'll need to sign a Contributor License Agreement (CLA).

Please follow instructions at https://icinga.com/company/contributor-agreement to sign the CLA.

After that, please reply here with a comment and we'll verify.

Contributors that have not signed yet: @NavidSassan

  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Please contact us if you think this is the case.

  • If you signed the CLA as a corporation, your GitHub username may not have been submitted to us. Please reach out to the responsible person in your organization.

cla-bot[bot] avatar Sep 29 '22 09:09 cla-bot[bot]

Corporation CLA signed

NavidSassan avatar Sep 29 '22 10:09 NavidSassan

@cla-bot check

bobapple avatar Sep 29 '22 10:09 bobapple

@sukhwinder33445, is your request about the cosmetics and not about the content and information shown? Please confirm.

If not, I fear, that your request goes directly against our needs and why we sponsor this code!

Our requirement is to be able to see the process titles inside of the configs. In our use case, It's quite useless to know, on a big wall mounted sign, that X business critical processes inside a department, clinic, branch or team are kaput because the next question in mind is which one and there isn't a mouse or any other input available to find out without, then and there.

Please, have a look at the original pull request Add a nice dashboard for kiosk mode for a explanation and a mock-up.

slalomsk8er avatar Jul 31 '23 20:07 slalomsk8er

Hi @slalomsk8er, The request is also about the content and the information that should be displayed.

The main reason why we should not show the titles of the nodes in each configuration is that we would then have problems with scalability. A large environment can have multiple configurations with multiple (hundreds) of nodes. In this case, only a few configurations with their node titles and states can be displayed on the dashboard page.

In the following screenshot, you can see the current state of the full screen view with multiple nodes. As you can see, there is already less space available. The nodes can also have titles of different lengths, which can make the state badges unequal in length. The configuration containers have unequal height and width. localhost_icingaweb2_businessprocess_showFullscreen=1 view=compact

This is one of the reasons why we remove the titles of the nodes in the mockup. The small status badges can still contain a link to the node and have a tooltip that shows the name of the node. A display that hangs on the wall and provides a full screen view can show you the current status of the nodes in all given configurations. To get more information, the user can open this view on a local device and perform further investigation.

sukhwinder33445 avatar Aug 01 '23 09:08 sukhwinder33445

@sukhwinder33445 then we have a problem as showing the titles of the business processes is the whole point of this sponsored feature!

This is what it actually looks in production:

Screenshot_20230801_123022

It maybe not perfect but works well for us and as far as I know, we're currently the only users so please don't let the perfect be the enemy of the good.

slalomsk8er avatar Aug 01 '23 11:08 slalomsk8er