After I swipe close a tab, I see the old tab briefly flash back into place before its killed.
This happens when clicking on the 'X' to close a tab too. I'm seeing it on a Galaxy Q running 2.3, it feels very glitchy whenever I see it.
tracking-fennec: --- → ?
Created attachment 780514 [details] [diff] [review] Patch This makes us reset our transforms when the view is recycled instead of when the animation ends. I'm guessing this is from bug 868460, but I don't really see any changes there. Just moving things around. I suspected the setHasTransientState calls were the causing problems, but removing them didn't fix anything.
Comment on attachment 780514 [details] [diff] [review] Patch Grr. qdiff is failing me. Not the right patch.
Created attachment 780516 [details] [diff] [review] Patch
Comment on attachment 780516 [details] [diff] [review] Patch Review of attachment 780516 [details] [diff] [review]: ----------------------------------------------------------------- Interesting, I added code in PropertyAnimator that sets any animated view as having transient state (looks for setHasTransientState() calls there) which would cause the animated views to not be recycled by TwoWayView (at least for API level 16 and later). Maybe there's a bug in TwoWayView. In any case, this is needed for older Android versions. ::: mobile/android/base/TabsTray.java @@ +349,5 @@ > > TabRow tab = (TabRow)view.getTag(); > final int tabId = tab.id; > + if (mOriginalSize == 0) > + mOriginalSize = (isVertical ? view.getHeight() : view.getWidth()); Please add a comment explaining the assumption that all list items have the same size.
Attachment #780516 - Flags: review?(lucasr.at.mozilla) → review+
Maybe it's the case to uplift this to Aurora least?
Assignee: nobody → wjohnston
Status: NEW → ASSIGNED
OS: Linux → Android
Hardware: x86_64 → ARM
Wes - Does this happen on Beta or Release?
tracking-fennec: ? → 24+
I only see this on nightly.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
You need to log in before you can comment on or make changes to this bug.