fix(classifier): reset hidden marks when the subject changes

[eatyourgreens]

Reset the Hide Previous Marks toggle when a drawing task is reset. This should fix a bug where task.hidingIndex is remembered across subjects, which means that a volunteer won't see the first few marks that they draw on a new subject.

That affects both drawing and transcription tasks, as both use the same code to store task.hidingIndex across subjects.

Screen.Recording.2024-05-18.at.11.15.46.mov

Please request review from @zooniverse/frontend team or an individual member of that team.

Package

• lib-classifier/plugins/tasks/drawing
• lib-classifier/plugins/tasks/experimental/transcription

Linked Issue and/or Talk Post

• should fix Hide previous marks persists to next subject, but will show 2nd mark #6062.

How to Review

You can test this on WildWatch Burrowing Owl. See the video above.
https://localhost:8080/?project=sandiegozooglobal/wildwatch-burrowing-owls&env=production&demo=true

Checking a transcription workflow would also be a good idea, as the same code is used by both transcription and drawing tasks. I’ve included tests for both tasks in this PR.

Checklist

PR Creator - Please cater the checklist to fit the review needed for your code changes.
PR Reviewer - Use the checklist during your review. Each point should be checkmarked or discussed before PR approval.

General

• Tests are passing locally and on Github
• Documentation is up to date and changelog has been updated if appropriate
• You can yarn panic && yarn bootstrap or docker-compose up --build and FEM works as expected
• FEM works in all major desktop browsers: Firefox, Chrome, Edge, Safari (Use Browserstack account as needed)
• FEM works in a mobile browser

General UX

Example Staging Project: i-fancy-cats

• All pages of a FEM project load: Home Page, Classify Page, and About Pages
• Can submit a classification
• Can sign-in and sign-out
• The component is accessible
  • Can be used with a screen reader BBC guide to VoiceOver
  • Can be used from the keyboard WebAIM guide to keyboard testing
  • It is passing accessibility checks in the storybook

Bug Fix

• The PR creator has listed user actions to use when testing if bug is fixed
• The bug is fixed
• Unit tests are added or updated

[coveralls]

coverage: 80.383% (-0.01%) from 80.397%
when pulling 0611eb2 on eatyourgreens:fix-reset-hidden-marks
into 59d5240 on zooniverse:master.

[goplayoutside3]

Looks good thank you!