Menu

[PATCH] Added zoom by pinch gesture

irix
2015-04-13
2015-04-16
  • irix

    irix - 2015-04-13

    Hello,

    I added a small useful feature to the djview4. Could you please review this patch?

    Attached patch allows to use pinch zoom gesture to fine adjust zoom level. It does not conflict with mouse wheel zooming. Behaviour is available for Qt 4.6 and disabled by macro check for earlier versions. Tested on OS X 10.10, Qt 4.8.6 and djview 4.10.2 (master branch).


    Best regards,
    Eugene Seliverstov

     
  • irix

    irix - 2015-04-13

    Also made a pull request because a message did not show up for twenty minutes. Sorry for the duplicate.

     
    • Leon Bottou

      Leon Bottou - 2015-04-14

      On Monday, April 13, 2015 11:14:31 PM irix wrote:

      Also made a pull request because a message did not show up for twenty
      minutes. Sorry for the duplicate.

      A variant of your code is in git (tested with qt-5.2). I had to be more
      careful about canceled gestures and about the cumulative effect of pinch
      updates. I wish it worked under linux.

      • L.
       
  • irix

    irix - 2015-04-14

    Thank you. Checked latest commit, it works on Mac, cannot check Linux (no multitouch trackpad).

     
  • irix

    irix - 2015-04-14

    A little off-topic. I help maintaining djview at Homebrew. I saw a git tag for version 4.10.2 that fixes some critical build issues on OS X. Sf.net "Files" section does not contain this release. Is it an official or intermediate version? Could this release be placed at "Files"?

     
    • Leon Bottou

      Leon Bottou - 2015-04-14

      This is a tag in git.
      Do you actually need a file for homebrew or can you target a git tag?

      I have a question about homebrew. My mac has a haswell processor with plenty of new instructions (avx2, etc.) But I do not want homebrew to compile for this processor only because I use it to generate distributable binaries. I had to hack the homebrew code to prevent tuning for the machine (both the std and super envs), essentially pretending that I am building bottles. Is there a better way to achieve this?

      • L.

      On Apr 14, 2015, at 11:28 AM, irix theirix@users.sf.net wrote:

      A little off-topic. I help maintaining djview at Homebrew. I saw a git tag for version 4.10.2 that fixes some critical build issues on OS X. Sf.net "Files" section does not contain this release. Is it an official or intermediate version? Could this release be placed at "Files"?

      [PATCH] Added zoom by pinch gesture

      Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/djvu/discussion/103286/

      To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

       
      • irix

        irix - 2015-04-14

        For the Homebrew it's probably better to have an official release with a tarball. Using a git tag for a unreleased version is not widely used. Now I am using a 4.10 release and a few patches.

        You can build on your machine a portable binary targetting older CPU (core2 for 64-bit or core for 32-bit systems) by building a bottle. Easiest way is using the --build-bottle, for example brew install --build-bottle djview4.

         
        • Leon Bottou

          Leon Bottou - 2015-04-16

          I'll put a 4.10.2 out soon.
          Can I see your brew recipes somewhere (mine still uses 4.8).
          Best,
          - L.

          On Apr 14, 2015, at 6:13 PM, irix theirix@users.sf.net wrote:

          For the Homebrew it's probably better to have an official release with a tarball. Using a git tag for a unreleased version is not widely used. Now I am using a 4.10 release and a few patches.

          You can build on your machine a portable binary targetting older CPU (core2 for 64-bit or core for 32-bit systems) by building a bottle. Easiest way is using the --build-bottle, for example brew install --build-bottle djview4.

          [PATCH] Added zoom by pinch gesture

          Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/djvu/discussion/103286/

          To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

           
  • irix

    irix - 2015-04-16

    Thank you, it would be nice as it will remove unneeded patches in the homebrew recipe.
    My pull request on 4.10 is here https://github.com/Homebrew/homebrew/pull/38594

     
    • Leon Bottou

      Leon Bottou - 2015-04-16

      Also you can pass the configure arguments to autogen.sh and let it do everything.
      Note that djvulibre also moved to automate, using the same script.
      - L.

      On Apr 16, 2015, at 4:30 PM, irix theirix@users.sf.net wrote:

      Thank you, it would be nice as it will remove unneeded patches in the homebrew recipe.
      My pull request on 4.10 is here https://github.com/Homebrew/homebrew/pull/38594

      [PATCH] Added zoom by pinch gesture

      Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/djvu/discussion/103286/

      To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

       
  • irix

    irix - 2015-04-16

    Yes, I noticed it, very convenient. I followed the way separating autogen and configure calls for consistency with other Homebrew recipes.

     
    • Leon Bottou

      Leon Bottou - 2015-04-16

      Just added djview-4.10.3.tar.gz with pinch support as well.
      - L.

      On Apr 16, 2015, at 5:17 PM, irix theirix@users.sf.net wrote:

      Yes, I noticed it, very convenient. I followed the way separating autogen and configure calls for consistency with other Homebrew recipes.

      [PATCH] Added zoom by pinch gesture

      Sent from sourceforge.net because you indicated interest in https://sourceforge.net/p/djvu/discussion/103286/

      To unsubscribe from further messages, please visit https://sourceforge.net/auth/subscriptions/

       

Log in to post a comment.