Skip to content
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

WIP: Video Controls #129

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

WIP: Video Controls #129

wants to merge 2 commits into from

Conversation

Quackdoc
Copy link

Thought I would start implementing #120, So far there are no GUI controls, but these are the actual functions needed for Play/Pause, Getting time, and Setting time. Unfortunately I do not know the first thing about GTK, nor do I know where to start. but I hope this will help make Dev's job a little bit easier.

Play/Pause and time seeking works fine, but it really should be set to not work with GIFs, Playbin should take care of it, so it shouldn't cause bugs, but it would probably be best not to tempt fate.

Time seeking, and current time is currently done with percentages and nanoseconds, Im hoping this will make implementing the time/seek bar (if this is the route dev want's to go) a little bit easier, but reporting time isn't implemented yet (because I forgot, sorry), unless you want it in nanoseconds.

but using GST_FORMAT_TIME + GST_TIME_ARGS (ie. https://gstreamer.freedesktop.org/documentation/application-development/advanced/queryevents.html?gi-language=cpp) will be an easy implement.

I hope this helps the dev a little bit, Love the app.

@ahodesuka
Copy link
Owner

I can take care of the UI stuff. I already have a general idea of how I'd want it to look.
This should help me get started when I have some spare time to work on it.

@ahodesuka ahodesuka force-pushed the master branch 4 times, most recently from b47e09e to 2eb22dd Compare January 7, 2022 13:17
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.

None yet

2 participants