Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HITS panel : new features, new design #153

Closed
wants to merge 9 commits into from

Conversation

jsm-oxa
Copy link

@jsm-oxa jsm-oxa commented Jun 11, 2013

Screenshot : http://hpics.li/d2c5ecc

  • possibility to display count or/and percent
  • percent available in vertical et horizontal list mode
  • set thresholds to display warning/critical

@jsm-oxa
Copy link
Author

jsm-oxa commented Jun 14, 2013

Now you can set thresholds for each query.

A screenshot !
http://hpics.li/d2c5ecc

I hope you will enjoy !

jsm-oxa

@rashidkpc
Copy link
Contributor

Ooo thats really cool. I'll review this soon and see what we can do to get it in.

@cdeszaq
Copy link

cdeszaq commented Jun 20, 2013

👍 This looks really great!
Can the thresholds be set based on % only, or can they use the counts as well?

@jsm-oxa
Copy link
Author

jsm-oxa commented Jun 20, 2013

They are based on the % only. Because I think that an alert must be triggered only if a value is too high compared with the rest of events.

@jsm-oxa jsm-oxa closed this Jun 24, 2013
@jsm-oxa jsm-oxa reopened this Jun 24, 2013
@jsm-oxa
Copy link
Author

jsm-oxa commented Jun 24, 2013

Sorry, I'm cleannig my repository and I accidentally deleted this branch, I re-opened it and I add "hide row"/"fix label display"

</div>
<h5>Queries</h5>
<div class="row-fluid">
<div class="span3">
<form style="margin-bottom: 0px">
<label class="small">Label</label>
<label class="small">Label</label>
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't the label here be inside the form? This same issue shows up a number of places.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,

When the label is inside the form, it is not displayed...I'm the only one who has this problem ?

@rashidkpc
Copy link
Contributor

Still like these changes, they unfortunately don't fix since the m3 refactor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants