Closed felideon closed 3 weeks ago
The changes introduce a new optional property, sequential
, to the SurveyTaskList
component and its related stories, allowing for sequential management of survey tasks. Additionally, a surveyBlocked
property is added to the SingleSurveyTask
component, which controls user interactions based on the survey's availability. CSS modifications enhance the user interface by adjusting cursor behavior for interactive elements. These updates collectively improve the components' functionality and user experience in handling survey tasks.
File | Change Summary |
---|---|
src/components/container/SurveyTaskList/SurveyTaskList.* |
Added sequential property to SurveyTaskListProps , updated logic for task blocking, and modified storybook components to include sequential . |
src/components/presentational/SingleSurveyTask/SingleSurveyTask.* |
Introduced surveyBlocked property to manage interactions, updated click handlers, and modified story arguments accordingly. |
src/components/presentational/SingleSurveyTask/SingleSurveyTask.css |
Added a new CSS rule for .mdhui-single-survey-task.incomplete-expanded.active to enhance cursor behavior. |
sequenceDiagram
participant User
participant SurveyTaskList
participant SingleSurveyTask
User->>SurveyTaskList: Request to view tasks
SurveyTaskList->>SingleSurveyTask: Render tasks with sequential and surveyBlocked props
alt If surveyBlocked is true
SingleSurveyTask->>User: Disable interaction
else
SingleSurveyTask->>User: Enable interaction
end
Storybook screenshots:
@felideon The way it is implemented, a new view would be needed if there was ever a desire to complete non sequentially. Is that going to be ok?
I connected with @felideon on my previous question about how to not make this binary. In the near future, there will be support for filtering tasks in the task list by category. Having options like that will make it easier to support a mix of non seq and sequential on the same view
Overview
Paves way for https://github.com/CareEvolution/MyDataHelpsViewBuilder/issues/111.
Security
REMINDER: All file contents are public.
No security risks come to mind.
Checklist
Testing
Documentation
Consider "Squash and merge" as needed to keep the commit history reasonable on
main
.Reviewers
Assign to the appropriate reviewer(s). Minimally, a second set of eyes is needed ensure no non-public information is published. Consider also including:
Summary by CodeRabbit
New Features
sequential
property to enhance task presentation in the survey task components.surveyBlocked
property to manage user interactions based on survey availability.Bug Fixes
Style