Bug 126301 – [Gtk] Add fast-forward and reverse media controls

• Component: New Bugs
• Status: RESOLVED INVALID

[Brendan Long]

[Gtk] Add fast-forward and reverse media controls

[Brendan Long]

Created attachment 220123 [details]
Patch

[Brendan Long]

Created attachment 220124 [details]
Screenshot of new controls

Here's a screenshot of the new controls.

I'm not sure if you guys will want this, but someone on my project did, so I figured I'd offer it to you. I think the various media control patches would need to be rebaselined, but I'm on Fedora, so I don't think the pixel tests would pass if I did it.

[Brendan Long]

Philippe, what do you think of this?

[Philippe Normand]

Adding Xabier in CC.

I don't think this patch is worth the effort for the current/old controls but it'd be good to have it in the new controls :)

[Xabier Rodríguez Calvar]

(In reply to comment #4)
> Adding Xabier in CC.
> 
> I don't think this patch is worth the effort for the current/old controls but it'd be good to have it in the new controls :)

Agree. Anyway, I'm adding Jon McCann to see what he thinks about it.

[Brendan Long]

(In reply to comment #4)
> I don't think this patch is worth the effort for the current/old controls but it'd be good to have it in the new controls :)

How would I turn these new controls on?

[Xabier Rodríguez Calvar]

(In reply to comment #6)
> (In reply to comment #4)
> > I don't think this patch is worth the effort for the current/old controls but it'd be good to have it in the new controls :)
> 
> How would I turn these new controls on?

You can't just turn them on now. I am working on a patch for bug 123097, which I added as a dependency today. I don't think I will change the important bits of the code too much so I think you can safely use my latest patch as a starting point. Rebasing after some changes I am planning to do shouldn't be too complicated. Other thing you can do is waiting a bit until I finished with bug 123097, which should happen in one or two days.

[Xabier Rodríguez Calvar]

Jon, can you have a look at the screenshot that Brendan uploaded and give your opinion about it?

[Brendan Long]

(In reply to comment #7)
> You can't just turn them on now. I am working on a patch for bug 123097, which I added as a dependency today. I don't think I will change the important bits of the code too much so I think you can safely use my latest patch as a starting point. Rebasing after some changes I am planning to do shouldn't be too complicated. Other thing you can do is waiting a bit until I finished with bug 123097, which should happen in one or two days.

Ok, I'll just wait until you're done.

[William Jon McCann]

Can you describe a bit about why they would be needed? And why it is better than using the position slider?

[Brendan Long]

(In reply to comment #10)
> Can you describe a bit about why they would be needed? And why it is better than using the position slider?

These change the playback speed, not position. We wanted it to make testing the DLNA "Remote UI" spec more convenient, but I don't know if normal people would be interested.

[Xabier Rodríguez Calvar]

(In reply to comment #11)
> > Can you describe a bit about why they would be needed? And why it is better than using the position slider?
> 
> These change the playback speed, not position. We wanted it to make testing the DLNA "Remote UI" spec more convenient, but I don't know if normal people would be interested.

Jon, if you don't find them interesting, we can just add them with a compilation conditional or something.

[Brendan Long]

(In reply to comment #12)
> Jon, if you don't find them interesting, we can just add them with a compilation conditional or something.

You don't have to add them at all if you don't think they're useful. You won't hurt my feelings ;)

[Xabier Rodríguez Calvar]

(In reply to comment #13)
> > Jon, if you don't find them interesting, we can just add them with a compilation conditional or something.
> 
> You don't have to add them at all if you don't think they're useful. You won't hurt my feelings ;)

:D I wasn't thinking about your feelings but about your needs in combination to what the designers (Jon in this case) think is best.

[Martin Robinson]

Is this patch still valid with the media controls rewrite?

[Brendan Long]

Probably not.

[Brady Eidson]

Comment on attachment 220123 [details]
Patch

Assuming that patches for review since 2013 are stale, r-