sardell commented on issue #1334: METRON-1997: Replace Threat Triage Score Field Slider with Text Box URL: https://github.com/apache/metron/pull/1334#issuecomment-477583578 @ruffle1986 Just a few more things: - I noticed that when I add a threat triage rule without a name, the rule and score adjustment are shown in the list of rules. If I add a rule with a name, the name and score adjustment are shown and the rule is hidden. I think this can be confusing and hard to parse if a user has many rules because of the way we are displaying the data right now. I think it might be better to use a table to display the rule information. If there isn't a name for a rule, that rule's name column remains blank. - I also noticed that in the Threat Triage Rules in the read-only pane there is a word break.  This can be quickly resolved by removing the bootstrap col classes from the form labels and values and applying it to the wrapping div. - Your testing instructions include a test button, but it looks like that was removed. Could you just strikeout or remove that part of your instructions?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
