-
Notifications
You must be signed in to change notification settings - Fork 601
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
feat: chart interactivity: click points on the graph to select the corresponding jobs #3851
feat: chart interactivity: click points on the graph to select the corresponding jobs #3851
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.
👍 Looks good to me! Reviewed everything up to 87bdace in 1 minute and 43 seconds
More details
- Looked at
286
lines of code in4
files - Skipped
0
files when reviewing. - Skipped posting
1
drafted comments based on config settings.
1. frontend/src/lib/components/RunChart.svelte:144
- Draft comment:
ThehighlightSelectedPoints
function modifies thedata
object directly, which might not trigger a re-render in Svelte. Consider using an assignment todata
or a specific reactive statement to ensure UI updates. - Reason this comment was not posted:
Confidence of 30% on close inspection, compared to threshold of 50%.
Workflow ID: wflow_ouYtsevyIcu768NV
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
Deploying windmill with
|
Latest commit: |
5c7b8e9
|
Status: | ✅ Deploy successful! |
Preview URL: | https://6941cb38.windmill.pages.dev |
Branch Preview URL: | https://win-166-chart-interactivity.windmill.pages.dev |
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.
👍 Looks good to me! Incremental review on b4d7590 in 6 minutes and 25 seconds
More details
- Looked at
22
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
0
drafted comments based on config settings.
Workflow ID: wflow_WVzPJzsSsyhiie5K
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and 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.
👍 Looks good to me! Incremental review on 5c7b8e9 in 5 minutes and 18 seconds
More details
- Looked at
13
lines of code in1
files - Skipped
0
files when reviewing. - Skipped posting
0
drafted comments based on config settings.
Workflow ID: wflow_lfi3hT7Go4MrHTtR
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet
mode, and more.
Summary:
Enhanced the
RunChart.svelte
component to support interactive multiple job selections with dynamic visual feedback and updated interaction handling.Key points:
selectedId
toselectedIds
allowing multiple selections.RunChart.svelte
to support multiple job selections by clicking on graph points.backgroundColor
in datasets to dynamically change based on selection.highlightSelectedPoints
function to handle point highlighting and dynamically change point colors based on selection.onClick
event to updateselectedIds
based on user interaction.Generated with ❤️ by ellipsis.dev