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

#109, Move triggering control strategies in threat explorer, #110 explain untriggered threats #171

Merged
merged 20 commits into from
May 21, 2024

Conversation

kenmeacham
Copy link
Contributor

@kenmeacham kenmeacham commented May 14, 2024

Closes #109, Closes #110, Closes #172

@kenmeacham kenmeacham linked an issue May 14, 2024 that may be closed by this pull request
@kenmeacham kenmeacham marked this pull request as ready for review May 14, 2024 14:44
@kenmeacham kenmeacham requested a review from scp93ch May 14, 2024 14:44
…sing deprecated CSG type. Now a CSG can have several types for different threats.
@kenmeacham kenmeacham marked this pull request as draft May 16, 2024 09:33
@kenmeacham kenmeacham self-assigned this May 16, 2024
@kenmeacham
Copy link
Contributor Author

After disussion, @scp93ch preferred that the "Triggering Control Strategies" panel title should be coloured red when activated, and green otherwise. This now looks as follows:

image

We may need to revise the colour scheme further, as the light green colouring of the CSG panel may not now make sense?

@scp93ch
Copy link
Member

scp93ch commented May 16, 2024

We may need to revise the colour scheme further, as the light green colouring of the CSG panel may not now make sense?

Yeah, that looks pretty weird. I suggest just removing the light green when a CSG is active in the triggering control strategies panel. In the context of the threat that is triggered, the triggering CSG is kind of a bad thing so the red accordion title is okay. I think that colouring the background pink or something might be going too far, because the CSG isn't all bad - it does do something useful.

Copy link

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@kenmeacham kenmeacham marked this pull request as ready for review May 20, 2024 14:42
@kenmeacham
Copy link
Contributor Author

We may need to revise the colour scheme further, as the light green colouring of the CSG panel may not now make sense?

Yeah, that looks pretty weird. I suggest just removing the light green when a CSG is active in the triggering control strategies panel. In the context of the threat that is triggered, the triggering CSG is kind of a bad thing so the red accordion title is okay. I think that colouring the background pink or something might be going too far, because the CSG isn't all bad - it does do something useful.

The color scheme has now been revamped, via pull request #174

I think this branch should now be ready to merge in, if you approve @scp93ch

@kenmeacham kenmeacham merged commit c31b86a into dev May 21, 2024
3 checks passed
@kenmeacham kenmeacham deleted the 110-explain-untriggered-threats branch May 21, 2024 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants