Closed Bug 1440456 Opened 2 years ago Closed 2 years ago

Render context menus only when needed.

Categories

(Firefox :: New Tab Page, defect, P1)

defect

Tracking

()

RESOLVED FIXED
Firefox 60
Iteration:
60.4 - Mar 12
Tracking Status
firefox60 --- fixed

People

(Reporter: rrosario, Assigned: rrosario)

References

Details

Attachments

(1 file)

We currently render all the context menus for a page when it loads then we flip to visible via CSS if we need to show one. There are one for each link and soon for each section too.

Instead we should only render out to DOM when we need to show it (the user clicks on the context menu button).

Unclear if this is going to help performance, but it most definitely won't hurt it.

See discussion: https://github.com/mozilla/activity-stream/pull/3987#discussion_r169878203
Iteration: --- → 60.4 - Mar 12
Priority: -- → P1
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Blocks: 1441018
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.