sneak/AutistMask
1
0
Fork 0
Code Issues 1 Pull Requests 1 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

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
bot
 labels 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%
All checks were successful
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

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

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`.
clawbot removed the
bot
 label 2026-03-01 01:18:38 +01:00
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   
bot

Agent's turn to work

  
merge-ready

   
needs-checks

   
needs-rebase

   
needs-review

   
needs-rework
No Label
bot
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
sneak
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sneak/AutistMask#121
No description provided.
Delete Branch "fix/issue-100-faster-copy-flash"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?