After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 697896 - Add gestures
Add gestures
Status: RESOLVED OBSOLETE
Product: totem
Classification: Core
Component: Movie player
unspecified
Other Linux
: Normal normal
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on: 694435 724242
Blocks:
 
 
Reported: 2013-04-12 16:27 UTC by Bastien Nocera
Modified: 2018-05-24 10:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
backend: Use swipe gestures (2.77 KB, patch)
2014-02-11 12:36 UTC, Bastien Nocera
needs-work Details | Review
backend: Use swipe gestures (2.56 KB, patch)
2014-02-12 16:49 UTC, Bastien Nocera
none Details | Review
backend: Add support for zooming with pinch (2.47 KB, patch)
2014-11-25 20:06 UTC, Bastien Nocera
needs-work Details | Review

Description Bastien Nocera 2013-04-12 16:27:58 UTC
Ideas:
- 1-finger Swipe right/left -> skip (same as right/left arrows)
- 2-finger Swipe right/left -> next/previous tracks (same as n/b)
- 1-finger Swipe up/down -> change volume
- 2-finger Swipe up/down -> enable/disable remote display (when we have that feature...)
- zoom action (pinch movement) -> zoom in/out
- rotate action (2 finger rotation) -> rotate
Comment 1 Bastien Nocera 2014-02-11 12:36:31 UTC
Created attachment 268787 [details] [review]
backend: Use swipe gestures

1-finger Swipe right/left -> skip (same as right/left arrows)
1-finger Swipe up/down -> change volume

FIXME: But this doesn't work nicely. The gesture conflicts with the
tap gesture, and will generate first a tap when removing the finger
from the screen, then the swipe. So the controls are toggled each time
we swipe. Will need more work :/
Comment 2 Bastien Nocera 2014-02-11 12:46:14 UTC
Removing from 3.12 whiteboard, we won't be able to implement this without a significant amount of work on the gestures so that they don't conflict with each other.

Furthermore, even if we implemented this, it wouldn't change the UI.
Comment 3 Bastien Nocera 2014-02-12 16:49:06 UTC
Fixed in clutter! See bug 724242.
Comment 4 Bastien Nocera 2014-02-12 16:49:38 UTC
Created attachment 268938 [details] [review]
backend: Use swipe gestures

1-finger Swipe right/left -> skip (same as right/left arrows)
1-finger Swipe up/down -> change volume
Comment 5 Bastien Nocera 2014-02-13 17:59:09 UTC
(In reply to comment #0)
> Ideas:
> - 1-finger Swipe right/left -> skip (same as right/left arrows)

Done.

> - 2-finger Swipe right/left -> next/previous tracks (same as n/b)

Implementable.

> - 1-finger Swipe up/down -> change volume

We expect this to be done through a drag of some sort, which isn't that great for a gesture.

> - 2-finger Swipe up/down -> enable/disable remote display (when we have that
> feature...)

This is very gimmicky, and doesn't serve much of a purpose when we can have multiple targets. Unless you want it to look good in a demo.

> - zoom action (pinch movement) -> zoom in/out
> - rotate action (2 finger rotation) -> rotate

Those 2 are implementable, but we'd need to avoid arbitrary zooms or rotations, so they probably require custom gestures.
Comment 6 Bastien Nocera 2014-02-17 14:13:59 UTC
This wouldn't really impact on the UI either.
Comment 7 Bastien Nocera 2014-11-25 18:11:10 UTC
(In reply to comment #5)
> (In reply to comment #0)
> > Ideas:
> > - 2-finger Swipe right/left -> next/previous tracks (same as n/b)
> 
> Implementable.

Done.

> > - zoom action (pinch movement) -> zoom in/out

Double-tap is probably a good option for this, as it matches what you can do on bodies of text in browsers.
Comment 8 Bastien Nocera 2014-11-25 20:06:14 UTC
Created attachment 291491 [details] [review]
backend: Add support for zooming with pinch

XXX doesn't work that well unfortunately...
Comment 9 Bastien Nocera 2014-11-25 20:36:07 UTC
I've also had to revert the 2-finger swiping for now.

commit 3a0a07c8a068adb3f3e32cdaed5ad739cf687677
Author: Bastien Nocera <hadess@hadess.net>
Date:   Tue Nov 25 21:33:46 2014 +0100

    Revert "backend: Request next track when swiping with 2 fingers"
    
    This reverts commit 99dcbf8e9a97c261aa9620e038e5958dc1a9e975.
    
    We'll need to implement our own ClutterGestureAction for this to work.
Comment 10 GNOME Infrastructure Team 2018-05-24 10:44:18 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/totem/issues/69.