-
Notifications
You must be signed in to change notification settings - Fork 271
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
Disable autocommit and manually commit filters in study view #4502
Conversation
summary of feedback during refactorers stand-up:
|
fa85a45
to
c87289b
Compare
* TODO: A better solution would be a uniform list of filters | ||
* but this would require some refactoring. | ||
*/ | ||
getBrowserWindow().hesitantPillStore = {} as PillStore; |
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.
why not put these in the StudyView context?
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.
also, is "hesitant" a typo?
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.
Fixed in 60e31e2
Pill stores are now in study view store, and study view store is passed to pills using the context
@pvannierop @BasLee i did some testing here. I think there are probably some more tweaks that we would want to make to the UI before going live with this for our own users.
I think a feature like this should be manually tested thoroughly by someone on your team who is not a developer. It should also be well covered by e2e tests. |
@alisman #4502 (comment) |
☝️Feedback from community call on March 7 Relevant changes:
|
0a337c5
to
9b5b740
Compare
…submission Update screenshots
d5f1923
to
b1205a3
Compare
Add a checkbox to the study page to disable 'auto submitting' of filters; and add a submit button that allows users to submit filters manually. (Netlify preview)
'Autosubmitting' of filters is enabled by default. It can be activated with the 'Manually submit' option, available in a new settings menu
Problem
When a user adds a new filter on the study page, it is automatically added and processed: data is requested from the backend and the panes are re-rendered. When studies grow and datasets get larger, study view panes become less responsive, making it quite laborious to add a few filters.
Solution
This PR adds a checkbox and a submit button. The checkbox allows users to disable 'autosubmitting'. New filters are now queued. Once the new submit button is clicked, queued filters are submitted all at once, only triggering a single rerender.
Disabling autocommit
When disabling autosubmit, filters are added to the queue. Panes are only updated when the submit button is clicked.
Visualizing submitted and queued filters
When disabling autosubmit, the filter bar color codes 1) newly added filters; 2) already applied filters; and 3) filters that are queued for deletion.
TODO