The default bug view has changed. See this FAQ.

Tilt should use mozAnimation for all its animations, instead of intervals

RESOLVED FIXED in Firefox 13

Status

()

Firefox
Developer Tools: Inspector
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: vporof, Assigned: vporof)

Tracking

unspecified
Firefox 13
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [tilt])

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

5 years ago
See more info in bug 718425.
(Assignee)

Updated

5 years ago
Assignee: nobody → vporof
Status: NEW → ASSIGNED
Whiteboard: [tilt]
(Assignee)

Comment 1

5 years ago
Created attachment 593883 [details] [diff] [review]
v1

A few global consts had to be changed slightly to achieve approximately the same animation timings.
Attachment #593883 - Flags: review?(rcampbell)
(Assignee)

Updated

5 years ago
Depends on: 723435
(Assignee)

Updated

5 years ago
Blocks: 723588
(Assignee)

Comment 2

5 years ago
Created attachment 594118 [details] [diff] [review]
v2

Fixed a few fails in try.
Attachment #593883 - Attachment is obsolete: true
Attachment #593883 - Flags: review?(rcampbell)
Attachment #594118 - Flags: review?(rcampbell)
Comment on attachment 594118 [details] [diff] [review]
v2

-const INTRO_TRANSITION_DURATION = 50;
-const OUTRO_TRANSITION_DURATION = 40;
+const INTRO_TRANSITION_DURATION = 1000;
+const OUTRO_TRANSITION_DURATION = 800;
 const INITIAL_Z_TRANSLATION = 400;
 const MOVE_INTO_VIEW_ACCURACY = 50;
 
what are these units? Comments please.

 const MOUSE_CLICK_THRESHOLD = 10;
-const MOUSE_INTRO_DELAY = 10;
+const MOUSE_INTRO_DELAY = 200;
 const ARCBALL_SENSITIVITY = 0.5;

these too.

Otherwise looks really solid. Could you throw this through try as well? R+ with a successful run. Thanks!
Attachment #594118 - Flags: review?(rcampbell) → review+
(Assignee)

Comment 4

5 years ago
(In reply to Rob Campbell [:rc] (robcee) from comment #3)
> Comment on attachment 594118 [details] [diff] [review]
> v2
> 
> -const INTRO_TRANSITION_DURATION = 50;
> -const OUTRO_TRANSITION_DURATION = 40;
> +const INTRO_TRANSITION_DURATION = 1000;
> +const OUTRO_TRANSITION_DURATION = 800;
>  const INITIAL_Z_TRANSLATION = 400;
>  const MOVE_INTO_VIEW_ACCURACY = 50;
>  
> what are these units? Comments please.

I can't really relate these to a unit of measurement. Just interpolation magic numbers. Definitely not pixels, not centimeters, or kilograms...

> 
>  const MOUSE_CLICK_THRESHOLD = 10;
> -const MOUSE_INTRO_DELAY = 10;
> +const MOUSE_INTRO_DELAY = 200;
>  const ARCBALL_SENSITIVITY = 0.5;
> 
> these too.
> 
> Otherwise looks really solid. Could you throw this through try as well? R+
> with a successful run. Thanks!

Try: https://tbpl.mozilla.org/?tree=Try&rev=97ff4518c7
(Assignee)

Updated

5 years ago
Whiteboard: [tilt] → [tilt][land-in-fx-team]
(In reply to Victor Porof from comment #4)
> (In reply to Rob Campbell [:rc] (robcee) from comment #3)
> > Comment on attachment 594118 [details] [diff] [review]
> > v2
> > 
> > -const INTRO_TRANSITION_DURATION = 50;
> > -const OUTRO_TRANSITION_DURATION = 40;
> > +const INTRO_TRANSITION_DURATION = 1000;
> > +const OUTRO_TRANSITION_DURATION = 800;
> >  const INITIAL_Z_TRANSLATION = 400;
> >  const MOVE_INTO_VIEW_ACCURACY = 50;
> >  
> > what are these units? Comments please.
> 
> I can't really relate these to a unit of measurement. Just interpolation
> magic numbers. Definitely not pixels, not centimeters, or kilograms...

I was thinking fractions of a second, frames or intervals. "Interpolation magic" is kind of unsatisfying.

> >  const MOUSE_CLICK_THRESHOLD = 10;
> > -const MOUSE_INTRO_DELAY = 10;
> > +const MOUSE_INTRO_DELAY = 200;
> >  const ARCBALL_SENSITIVITY = 0.5;
> > 
> > these too.
> > 
> > Otherwise looks really solid. Could you throw this through try as well? R+
> > with a successful run. Thanks!
> 
> Try: https://tbpl.mozilla.org/?tree=Try&rev=97ff4518c7

ah well. Land ho!
https://hg.mozilla.org/integration/fx-team/rev/3b55713894a6
Whiteboard: [tilt][land-in-fx-team] → [tilt][fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/3b55713894a6
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [tilt][fixed-in-fx-team] → [tilt]
Target Milestone: --- → Firefox 13
You need to log in before you can comment on or make changes to this bug.