← Back to team overview

desktop-packages team mailing list archive

[Bug 873866] Re: --disable-gestures does not work

 

** Description changed:

+ SRU Justification:
+ ==================
+ [Impact]
+ Gesture support in applications currently requires a tradeoff when dealing with two touch scrolling on trackpads. There may be users who would prefer to operate evince without gesture support. Currently, --disable-gestures/-g does not work.
+ 
+ [Development Fix]
+ The bug has been fixed in the development branch by modifying the libgrip gestures distro patch. Version 3.2.1-0ubuntu2 released in precise has the fix.
+ 
+ [Stable Fix]
+ See http://bazaar.launchpad.net/~ubuntu-desktop/evince/ubuntu/revision/133.
+ 
+ [Test Case]
+ Run evince at the console. The warning about the "enable-gestures" property will show. Run with --disable-gestures and the warning will still appear, and gestures will still be enabled. The list of evince gestures are: two touch pinch to zoom and rotate on touchpads and touchscreens, and two touch drag to scroll on touchscreens. Note this only works with multitouch gesture capable hardware, such as Macbook trackpads and Magic Trackpads
+ 
+ [Regression Potential]
+ Minimal. The patch is extremely straightforward. The enable-gestures property is now checked before registering for gestures, and the unnecessary G_PARAM_CONSTRUCTION_ONLY flag on the property has been removed.
+ 
+ Original bug report:
+ ====================
  The --disable-gestures flag does not work due to two issues:
  
  1. The enable-gestures gobject class property of the EvView object is created with the G_PARAM_CONSTRUCT_ONLY flag.
  2. The property value is not checked before registering for gestures.
  
  The first issue causes the following warning in the console:
  
  (evince:23456): GLib-GObject-WARNING **: g_object_set_valist: construct
  property "enable-gestures" for object `EvView' can't be set after
  construction
  
  ProblemType: Bug
  DistroRelease: Ubuntu 11.10
  Package: evince 3.2.0-0ubuntu1
  ProcVersionSignature: Ubuntu 3.0.0-12.20-generic 3.0.4
  Uname: Linux 3.0.0-12-generic x86_64
  ApportVersion: 1.23-0ubuntu3
  Architecture: amd64
  Date: Fri Oct 14 02:10:49 2011
  ExecutablePath: /usr/bin/evince
  InstallationMedia: Ubuntu 11.04 "Natty Narwhal" - Release amd64 (20110427.1)
  ProcEnviron:
   SHELL=/bin/bash
   PATH=(custom, no user)
   LANG=en_CA.UTF-8
   LANGUAGE=en_CA:en
  SourcePackage: evince
  UpgradeStatus: Upgraded to oneiric on 2011-10-13 (0 days ago)

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/873866

Title:
  --disable-gestures does not work

Status in “evince” package in Ubuntu:
  Fix Committed
Status in “evince” source package in Oneiric:
  Triaged

Bug description:
  SRU Justification:
  ==================
  [Impact]
  Gesture support in applications currently requires a tradeoff when dealing with two touch scrolling on trackpads. There may be users who would prefer to operate evince without gesture support. Currently, --disable-gestures/-g does not work.

  [Development Fix]
  The bug has been fixed in the development branch by modifying the libgrip gestures distro patch. Version 3.2.1-0ubuntu2 released in precise has the fix.

  [Stable Fix]
  See http://bazaar.launchpad.net/~ubuntu-desktop/evince/ubuntu/revision/133.

  [Test Case]
  Run evince at the console. The warning about the "enable-gestures" property will show. Run with --disable-gestures and the warning will still appear, and gestures will still be enabled. The list of evince gestures are: two touch pinch to zoom and rotate on touchpads and touchscreens, and two touch drag to scroll on touchscreens. Note this only works with multitouch gesture capable hardware, such as Macbook trackpads and Magic Trackpads

  [Regression Potential]
  Minimal. The patch is extremely straightforward. The enable-gestures property is now checked before registering for gestures, and the unnecessary G_PARAM_CONSTRUCTION_ONLY flag on the property has been removed.

  Original bug report:
  ====================
  The --disable-gestures flag does not work due to two issues:

  1. The enable-gestures gobject class property of the EvView object is created with the G_PARAM_CONSTRUCT_ONLY flag.
  2. The property value is not checked before registering for gestures.

  The first issue causes the following warning in the console:

  (evince:23456): GLib-GObject-WARNING **: g_object_set_valist:
  construct property "enable-gestures" for object `EvView' can't be set
  after construction

  ProblemType: Bug
  DistroRelease: Ubuntu 11.10
  Package: evince 3.2.0-0ubuntu1
  ProcVersionSignature: Ubuntu 3.0.0-12.20-generic 3.0.4
  Uname: Linux 3.0.0-12-generic x86_64
  ApportVersion: 1.23-0ubuntu3
  Architecture: amd64
  Date: Fri Oct 14 02:10:49 2011
  ExecutablePath: /usr/bin/evince
  InstallationMedia: Ubuntu 11.04 "Natty Narwhal" - Release amd64 (20110427.1)
  ProcEnviron:
   SHELL=/bin/bash
   PATH=(custom, no user)
   LANG=en_CA.UTF-8
   LANGUAGE=en_CA:en
  SourcePackage: evince
  UpgradeStatus: Upgraded to oneiric on 2011-10-13 (0 days ago)

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/evince/+bug/873866/+subscriptions


References