-
Notifications
You must be signed in to change notification settings - Fork 172
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add duration knob #934
feat: add duration knob #934
Conversation
feat: add tests
67dc797
to
33fb773
Compare
… feat/duration_knob
Hi @YoussefRaafatNasry can you please take a look/review? |
doc: add duration field to example chore: remove decoration label from duration field chore: fix typo
7e831d3
to
73d6932
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really love how professional this PR is! 💙
You have done a good job with neat implementation, adding tests, Changelog item, even a video and an example.
I've added some nitpick comments that will make the code style more matching to ours, and will improve the UX!
Can't wait to get this merged and released!
Made the changes and added their respective test groups. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One little thing I forgot to mention, but we will really appreciate if you could add the new knobs to the docs as well.
@YoussefRaafatNasry I have made the changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job @Mastersam07!
Adds duration knob.
Duration should be entered in milliseconds (1000 milliseconds == 1 second).
List of issues which are fixed by the PR
#900
Screenrecord
durationField.mov
Checklist
///
).If you need help, consider asking for advice on Discord.