[meta] Update our animations to use NineOldAndroids

RESOLVED FIXED

Status

()

Firefox for Android
General
RESOLVED FIXED
4 years ago
2 years ago

People

(Reporter: Margaret, Unassigned)

Tracking

Trunk
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

4 years ago
I filed bug 1044133 to add NineOldAndroids to the tree, so we should start to update our animations to use that instead of our own animation compatibility library.

Please set bugs to update individual animations to block this one.
(Reporter)

Updated

4 years ago
Depends on: 1044273
(Reporter)

Updated

4 years ago
Depends on: 1044275
I was thinking about this this morning and made myself nervous. This seems like core functionality that we're likely going to want to extend/change at times. At least, I've extended it before in bug 889620 (which never landed because of other things).

Are we getting ourselves into the same situations we've got with Picasso here where our work is going to wind up blocked on getting updates into a (basically non-maintained) animation library? What did we need here that our already written animation libraries couldn't support?
(Reporter)

Comment 2

4 years ago
Since NineOldAndroids doesn't seem to be actively developed anymore, we probably won't ever need to update it, so I would be fine with us patching it ourselves if we need to.

However, I think it has probably been used and tested much more than our homegrown animation support, so I don't know how much we would need to change.
(Reporter)

Updated

2 years ago
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.