Skip to content

fix/timer-bug#137

Merged
RiyanMak merged 1 commit into
mainfrom
fix/timer-bug
May 1, 2026
Merged

fix/timer-bug#137
RiyanMak merged 1 commit into
mainfrom
fix/timer-bug

Conversation

@Youmanvi
Copy link
Copy Markdown
Contributor

@Youmanvi Youmanvi commented Apr 30, 2026

  • Fixed useRoundTimer where setInterval and cleanup were placed
    inside updateTimer instead of the useEffect body — the timer
    never actually started
  • Fixed minutes zero-padding in timer display (was showing 5:03
    instead of 05:03 when under 1 hour)
  • Removed stray \ before in Toolbar that broke the
    Previous Problem arrow icon
  • Changed stroke-width, stroke-linecap, stroke-linejoin to
    camelCase (strokeWidth, strokeLinecap, strokeLinejoin) across
    Toolbar SVGs so stroke styles render correctly in React

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bsg Ready Ready Preview, Comment Apr 30, 2026 1:18am

Request Review

@Eskan0r Eskan0r self-requested a review May 1, 2026 03:00
Copy link
Copy Markdown
Contributor

@Eskan0r Eskan0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@Eskan0r Eskan0r requested a review from RiyanMak May 1, 2026 03:03
@RiyanMak RiyanMak merged commit 0706758 into main May 1, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants