sneak/AutistMask
1
0
Fork 0
Code Issues 60 Pull Requests Actions Packages Projects Releases Wiki Activity

feat: speed up copy-flash timing by ~25% — follow-up to #113 #121

Merged
sneak merged 1 commits from fix/issue-100-faster-copy-flash into main 2026-03-01 01:21:25 +01:00
Conversation 2 Commits 1 Files Changed 2
+4 -4
clawbot commented 2026-03-01 01:17:19 +01:00
Collaborator
Copy Link
Copy Source

Follow-up to PR #113 per sneak's feedback: the copy-flash animation felt laggy.

Changes:

  • Active phase: 100ms → 75ms
  • Fade transition: 300ms → 225ms
  • Cleanup delay: 350ms → 275ms

Total animation time reduced from ~450ms to ~300ms (~25% faster).

Refs #100

Follow-up to PR #113 per sneak's feedback: the copy-flash animation felt laggy. Changes: - Active phase: 100ms → 75ms - Fade transition: 300ms → 225ms - Cleanup delay: 350ms → 275ms Total animation time reduced from ~450ms to ~300ms (~25% faster). Refs #100
clawbot added the needs-review label 2026-03-01 01:17:19 +01:00
clawbot self-assigned this 2026-03-01 01:17:19 +01:00
clawbot added 1 commit 2026-03-01 01:17:20 +01:00
feat: speed up copy-flash timing by ~25%
check / check (push) Successful in 20s
Details
d78af3ec80 
Reduce active phase from 100ms to 75ms, fade transition from 300ms to
225ms, and cleanup delay from 350ms to 275ms for snappier feedback.

Refs #100
clawbot commented 2026-03-01 01:17:28 +01:00
Author
Collaborator
Copy Link
Copy Source

Rework agent: applied ~25% timing speedup per sneak's review feedback on PR #113. Docker build passes. Ready for review.

Rework agent: applied ~25% timing speedup per sneak's review feedback on PR #113. Docker build passes. Ready for review.
clawbot removed their assignment 2026-03-01 01:17:33 +01:00
clawbot added merge-ready and removed needs-review labels 2026-03-01 01:18:20 +01:00
sneak was assigned by clawbot 2026-03-01 01:18:20 +01:00
clawbot commented 2026-03-01 01:18:35 +01:00
Author
Collaborator
Copy Link
Copy Source

Review: ✅ PASS

Changes are correct and match the stated intent:

  • Active phase: 100ms → 75ms ✓
  • CSS fade transition: 300ms → 225ms ✓
  • JS cleanup delay: 350ms → 275ms ✓

No linter config, test, or Makefile modifications. docker build . passes. Only timing constants changed in the expected files (main.css and helpers.js).

Marking as merge-ready.

## Review: ✅ PASS Changes are correct and match the stated intent: - Active phase: 100ms → 75ms ✓ - CSS fade transition: 300ms → 225ms ✓ - JS cleanup delay: 350ms → 275ms ✓ No linter config, test, or Makefile modifications. `docker build .` passes. Only timing constants changed in the expected files (`main.css` and `helpers.js`). Marking as `merge-ready`.
sneak merged commit 5f01d9f111 into main 2026-03-01 01:21:25 +01:00
sneak deleted branch fix/issue-100-faster-copy-flash 2026-03-01 01:21:25 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No labels merge-ready
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees sneak
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#121