-
Notifications
You must be signed in to change notification settings - Fork 30
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
{selectedResults.map((resultId) => ( | ||
<React.Fragment key={resultId}> | ||
{selectedLogKeys.map((logKey) => ( | ||
<Check |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't intend it. I tried with only one log. Sorry. 🙇
I adopt the same axis config instead of line setting.
In other words, the same name is grouped together, so in this example it is one main/loss
.
Thank you for your report.
Fixed 💪 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you
self-merge. |
cf. #302