Skip to content

Conversation

@ResistanceTo
Copy link

No description provided.

@ResistanceTo
Copy link
Author

Add a configurable countdown option that allows a few seconds for video recording preparation

@ResistanceTo ResistanceTo changed the title Add Chinese localization Add a configurable countdown option May 31, 2024
Copy link
Owner

@Mnpn Mnpn left a comment

Choose a reason for hiding this comment

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

Hi, thanks for the PR! This looks pretty good - and thanks for translating!

  • I think it would probably be best if the countdown value was to be limited to e.g. 0-99, because you can currently enter negative values and also cause arithmetic overflows (crashes)
  • (Bonus idea that you don't have to add for me to merge the PR) It would be nice if it were possible to cancel a countdown, e.g. by clicking the recording indicator (or pressing the set keybind again)

@ResistanceTo
Copy link
Author

The first issue regarding the value range (0-99) has been fixed. As for the second suggestion, I personally don't think it's a good idea. We already have a 'start recording' shortcut key, and adding a 'skip countdown to start recording' feature seems to defeat the purpose of having a countdown. Our default countdown value is set to 0.

@Mnpn
Copy link
Owner

Mnpn commented Jun 7, 2024

Ah, I think you misunderstood, I meant that it should be possible to cancel the recording itself during a countdown; say you have a 10 second countdown, and started a recording you didn't mean to, there is no way to stop and you have to wait for it to finish counting down :P But this looks good so I'll go ahead and merge it -- thanks again for the PR!

@Mnpn Mnpn merged commit 7c01c65 into Mnpn:main Jun 7, 2024
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.

2 participants