Skip to content
This repository was archived by the owner on Jun 17, 2026. It is now read-only.

[codex] Allow decimal custom playback speeds#636

Closed
muttyqt15 wants to merge 1 commit into
siddharthvaddem:mainfrom
muttyqt15:codex/custom-playback-speed-decimals
Closed

[codex] Allow decimal custom playback speeds#636
muttyqt15 wants to merge 1 commit into
siddharthvaddem:mainfrom
muttyqt15:codex/custom-playback-speed-decimals

Conversation

@muttyqt15

@muttyqt15 muttyqt15 commented May 21, 2026

Copy link
Copy Markdown

Summary

  • allow the custom playback speed field to accept decimal values and sub-1 speeds
  • preserve non-preset values like 1.1x instead of rounding them to whole numbers
  • add regression tests for custom playback speed parsing and formatting

Root cause

The custom speed input stripped all non-digits with replace(/\D/g, "") and initialized non-preset values with Math.round(value), so values like 1.1 could not be entered or displayed correctly.

Impact

Users can enter decimal speeds like 1.1x, 0.9x, and other values up to 16x through the custom playback speed control, matching the existing 2-decimal playback speed model.

Validation

  • npm test -- src/components/video-editor/customPlaybackSpeed.test.ts
  • npx tsc --noEmit

Fixes #635.

@coderabbitai

coderabbitai Bot commented May 21, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8991b3af-2d7d-4d61-840c-7dacc7b9dcfa

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@muttyqt15

Copy link
Copy Markdown
Author

Superseded by #637 with cleaned-up PR metadata.

@muttyqt15 muttyqt15 closed this May 21, 2026
@muttyqt15 muttyqt15 deleted the codex/custom-playback-speed-decimals branch May 21, 2026 20:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Custom Playback Speed input does not accept decimal values like 1.1×

1 participant