Closed Bug 984409 Opened 6 years ago Closed 4 years ago

Context menu: go to change wallpaper, has 8 reflows

Categories

(Firefox OS Graveyard :: Gaia::Homescreen, defect)

x86
macOS
defect
Not set

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: janjongboom, Unassigned)

References

Details

Attachments

(1 file)

I don't know yet if this is actually a problem, but when you long tap on the homescreen it creates 3 reflows, then when you click 'Change wallpaper' it creates another 5. Should be doable in 2 reflows I'd presume.
Blocks: 984408
Assignee: nobody → crdlc
Status: NEW → ASSIGNED
Attached file Patch v1
There are two different scenarios:

1) THE FIRST LOAD

- Reflow when the fragment is added to DOM
- Reflow when the fragment is translated
- Reflow when the button "wallpaper" is pressed (action-menu BB)
- Reflow when the button "wallpaper" is released (action-menu BB)

Total: 4 reflows

2) CONSECUTIVE TIMES

- Reflow when the button "wallpaper" is pressed (action-menu BB)
- Reflow when the button "wallpaper" is released (action-menu BB)

Total 2 reflows

Also, if we open the context menu and close it clicking on home button:

1) THE FIRST LOAD

- Reflow when the fragment is added to DOM
- Reflow when the fragment is translated

Total: 2 reflows

2) CONSECUTIVE TIMES

Total: 0 reflows
Attachment #8392883 - Flags: review?(21)
Comment on attachment 8392883 [details]
Patch v1

Please reask r? once review comments are addressed.
Attachment #8392883 - Flags: review?(21)
Assignee: crdlc → nobody
Status: ASSIGNED → NEW
Mass update: Resolve wontfix all issues with legacy homescreens.

As of 2.6 we have a new homescreen and having these issues open is confusing. All issues will block bug 1231115 so we can use that to re-visit any of these if needed.
Blocks: 1231115
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.