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

Bar graph selection #2409

Merged
merged 6 commits into from
Sep 18, 2024
Merged

Bar graph selection #2409

merged 6 commits into from
Sep 18, 2024

Conversation

bgoldowsky
Copy link
Contributor

@bgoldowsky bgoldowsky commented Sep 17, 2024

PT-188129114

Implements selection features:

  • if a bar includes a case that is selected, it will be highlighted with a green border.
  • clicking on a bar selects all cases that make up that bar

Note that modifier keys are not implemented as part of this PR, nor is any behavior for clicking or dragging on the background of the bar graph tile, so you cannot deselect cases that you selected by clicking on a bar.

Base automatically changed from 188129121-bar-graph-sync to 188127118-bar-graph-link September 17, 2024 17:05
Base automatically changed from 188127118-bar-graph-link to master September 17, 2024 18:17
Copy link

codecov bot commented Sep 17, 2024

Codecov Report

Attention: Patch coverage is 75.00000% with 20 lines in your changes missing coverage. Please review.

Project coverage is 86.28%. Comparing base (7a65c8c) to head (6a918e6).
Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
src/plugins/bar-graph/bar-graph-content.ts 61.53% 17 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2409      +/-   ##
==========================================
- Coverage   86.31%   86.28%   -0.03%     
==========================================
  Files         739      739              
  Lines       37955    38018      +63     
  Branches     9680     9701      +21     
==========================================
+ Hits        32759    32802      +43     
- Misses       4900     4917      +17     
- Partials      296      299       +3     
Flag Coverage Δ
cypress ?
cypress-regression 78.02% <74.02%> (-0.02%) ⬇️
cypress-smoke 27.88% <0.00%> (-0.06%) ⬇️
jest 48.75% <61.53%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

cypress bot commented Sep 17, 2024

collaborative-learning    Run #13781

Run Properties:  status check passed Passed #13781  •  git commit 6a918e611d: Merge branch 'master' into 188129114-bar-graph-select
Project collaborative-learning
Branch Review 188129114-bar-graph-select
Run status status check passed Passed #13781
Run duration 13m 53s
Commit git commit 6a918e611d: Merge branch 'master' into 188129114-bar-graph-select
Committer Boris Goldowsky
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 3
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 111
View all changes introduced in this branch ↗︎

Copy link
Contributor

@emcelroy emcelroy left a 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 👍

@bgoldowsky bgoldowsky merged commit e31b61e into master Sep 18, 2024
17 of 19 checks passed
@bgoldowsky bgoldowsky deleted the 188129114-bar-graph-select branch September 18, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants