Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Tab tray screenshots flicker when loaded in #14343

Closed
yoasif opened this issue Aug 28, 2020 · 6 comments
Closed

Tab tray screenshots flicker when loaded in #14343

yoasif opened this issue Aug 28, 2020 · 6 comments
Labels
🐞 bug Crashes, Something isn't working, .. Feature:Tabs pin Issues, features, improvements that are still valid

Comments

@yoasif
Copy link
Contributor

yoasif commented Aug 28, 2020

Steps to reproduce

  1. Tap tab switcher icon

Expected behavior

Screenshots should be present on initial display.

Actual behavior

I see placeholders briefly, then a flicker as the screenshots are loaded. Worse still, the screenshots are not present if I open the tray a few times in succession.

See video:

recording-20200828-022847

Device information

  • Android device: Moto G (XT1031)
  • Fenix version: latest nightly

┆Issue is synchronized with this Jira Task

@yoasif yoasif added the 🐞 bug Crashes, Something isn't working, .. label Aug 28, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Aug 28, 2020
@liuche liuche added Feature:Tabs and removed needs:triage Issue needs triage labels Aug 28, 2020
@liuche
Copy link
Contributor

liuche commented Aug 28, 2020

Do you know @jonalmeida if there's something that happens with the AC thumbnails loading (are we reloading them every time) or if there's something else we could do on the Fenix side re: flicker?

@jonalmeida
Copy link
Contributor

jonalmeida commented Aug 28, 2020

@liuche If by "flicker" you mean fade in, then yes this is the default loading animation behaviour of the RecyclerView. If we want to change this, we can look into non-default state to show when the thumbnail is loading from disk or some other technique to not show the empty thumbnail state. For Fennec, I can see it's just a blank white square and the image loads without an animation.

Otherwise, I don't see any flicker in the gif above. @yoasif could you clarify?

Worse still, the screenshots are not present if I open the tray a few times in succession.

This sounds like a separate bug that I'm more interested in knowing the STR for, although it sounds like a race.

@yoasif
Copy link
Contributor Author

yoasif commented Aug 31, 2020

@jonalmeida If I switch windows in macOS or Windows, I never see a generic "window" icon - I just see the windows when switching between them. Fennec also manages to show screenshots every time i load up its tab view without ever loading in a placeholder. That is the flicker I am referring to.

If Fennec shows white at any point, I am not seeing it, so it may be caching it (which is what I would have expected on subsequent loads of the tab tray in Fenix).

@stale
Copy link

stale bot commented Feb 27, 2021

See: #17373 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Feb 27, 2021
@stale stale bot closed this as completed Mar 6, 2021
@jonalmeida
Copy link
Contributor

With #19809, this bug is noticeable so let's see if there's anything we can do about it with the new implementation of the tabs tray.

@jonalmeida jonalmeida reopened this Jun 4, 2021
@stale stale bot removed the wontfix label Jun 4, 2021
@jonalmeida jonalmeida added the pin Issues, features, improvements that are still valid label Aug 17, 2021
@boek boek changed the title [Bug] Tab tray screenshots flicker when loaded in Tab tray screenshots flicker when loaded in Jan 27, 2023
@boek
Copy link
Contributor

boek commented Jan 27, 2023

Moved to bugzilla: https://bugzilla.mozilla.org/show_bug.cgi?id=1812912

Change performed by the Move to Bugzilla add-on.

@boek boek closed this as completed Jan 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. Feature:Tabs pin Issues, features, improvements that are still valid
Projects
None yet
Development

No branches or pull requests

4 participants