709eee2579
Real bug, not cosmetic: setAllSetsDone/toggleSet/setExerciseDone still auto-called logWorkoutCompleted() whenever doneSets == totalSets — leftover pre-redesign behavior that fought the new completion card. Ticking the last box (or tapping 'Check all sets') silently counted the day and snapped the UI to the green 'done' takeover without the user ever hitting Finish. Removed the auto-log from all three set-mutation methods. Only the card's explicit Finish Workout/Finish Anyway (-> markWorkoutDone) now counts a day as done. Check all sets just checks boxes. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>