Beginning on October 25th, 2016, Persona will no longer be an option for authentication on BMO. For more details see Persona Deprecated.
Last Comment Bug 715163 - Mouse click + drag causing Tilt to Translate instead of Rotate on First Open
: Mouse click + drag causing Tilt to Translate instead of Rotate on First Open
Product: Firefox
Classification: Client Software
Component: Developer Tools: Inspector (show other bugs)
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: Victor Porof [:vporof][:vp]
: Patrick Brosset <:pbro>
Depends on: 718425
Blocks: 718973
  Show dependency treegraph
Reported: 2012-01-04 09:03 PST by Rob Campbell [:rc] (:robcee)
Modified: 2012-01-28 16:09 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

v1 (2.51 KB, patch)
2012-01-17 02:34 PST, Victor Porof [:vporof][:vp]
rcampbell: review+
Details | Diff | Splinter Review

Description Rob Campbell [:rc] (:robcee) 2012-01-04 09:03:34 PST

1. Open Tilt on a page.
2. Click with mouse (trackpad) and drag around

Expected results:

Page is rotated.


Sometimes the page translates instead of rotates.

Not sure why this is happening but it seems to only happen the first time Tilt is opened. Subsequent drag events work as expected.
Comment 1 Victor Porof [:vporof][:vp] 2012-01-14 02:48:07 PST
I really can't reproduce this :(
Comment 2 Victor Porof [:vporof][:vp] 2012-01-14 13:14:59 PST
I may have found a reason for this: Tilt is opened using Ctr+W (or Ctr+M before bug 711966). Ctrl+Click changes left click to right click. Right click + drag = translate, while left click + drag = rotate.

So whenever you do Ctrl+Click + drag you get translation. Is this the problem?
Comment 3 Rob Campbell [:rc] (:robcee) 2012-01-16 08:53:29 PST
Hm! Maybe. It usually happens after I've opened Tilt and start dragging. There could be something to the right-click to translate function. All I know is that it's confusing when it happens. I'll watch my fingers the next time I see it.
Comment 4 Victor Porof [:vporof][:vp] 2012-01-16 09:20:40 PST
I could disable all events for a split second when you open Tilt. This would allow the initial animation to finish (and also even avoid any of that weird znear/zfar clipping if you have fast fingers). How about that?
Comment 5 Rob Campbell [:rc] (:robcee) 2012-01-16 11:20:54 PST
Can we do this and increase the speed of the intro animation? We want the UI to be responsive within 50ms.
Comment 6 Victor Porof [:vporof][:vp] 2012-01-16 11:23:28 PST
It's a plan.
Comment 7 Victor Porof [:vporof][:vp] 2012-01-17 02:34:21 PST
Created attachment 589134 [details] [diff] [review]
Comment 8 Rob Campbell [:rc] (:robcee) 2012-01-25 07:00:55 PST
did you want review on this patch? No request present and it's a blocker for bug 718973.
Comment 9 Victor Porof [:vporof][:vp] 2012-01-25 07:06:51 PST
(In reply to Rob Campbell [:rc] (robcee) from comment #8)
> did you want review on this patch? No request present and it's a blocker for
> bug 718973.

Yes please :)
Comment 10 Rob Campbell [:rc] (:robcee) 2012-01-25 11:47:31 PST
Comment 11 Tim Taubert [:ttaubert] 2012-01-25 23:58:02 PST
Comment 12 Rob Campbell [:rc] (:robcee) 2012-01-27 07:33:37 PST
Comment on attachment 589134 [details] [diff] [review]

[Approval Request Comment]
Regression caused by (bug #): New Feature
User impact if declined: Occasional accidental tranlation
Testing completed (on m-c, etc.): m-c
Risk to taking this patch (and alternatives if risky): Pretty minimal changes.

Note You need to log in before you can comment on or make changes to this bug.