Fix Line Graph Legend Position (Close #100) #107
Merged
b72zhou
merged 5 commits from b72zhou-line-graph-legend
into main
3 months ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'b72zhou-line-graph-legend'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Fix Line Graph Legend Positionto Fix Line Graph Legend Position (Close #100) 4 months agoI got rid of other unused variables/props in the component. I also modified the LineGraph legend so that it is only displayed centered at a fixed space from the graph because I found it more simple than having a left and top offset of the graph and legend. Let me know if that is good with you @b72zhou Otherwise, LGTM! 🚀
I just noticed a small thing.
flexDirection: legendPosition === "right" ? "row" : "column-reverse",
}}
>
<div className={styles.container}>
What is
styles.container
for? Where is thecontainer
class?sry a class was there but deleted, got rid of it
2720728e46
into main 3 months agoReviewers
2720728e46
.