Follow-up from "WIP: Resolve "Group Security Dashboard metrics MVC""
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
The following discussions from !8631 (merged) should be addressed:
-
@mikegreiling started a discussion: (+1 comment) I understand the need to use inline styles for the background color, but shouldn't the width and height values here be encapsulated in a class?
/cc @samdbeckham
-
@mikegreiling started a discussion: We should consider re-implementing the legend in HTML below the graph. There isn't really any reason for this to be done in SVG/Canvas and rendering it separately would allow us to make it more responsive and accessible. e.g. it won't need to scroll with the graph when it overflows.
/cc @adrielsantiago perhaps we can implement a generic legend component to use here.