Closed Bug 1079434 Opened 10 years ago Closed 5 years ago

Use template tag instead of setting innerHTML grid.js's _createSiteFragment method

Categories

(Firefox :: New Tab Page, defect)

Product:
Firefox
Component:
New Tab Page
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: evold, Unassigned)

References

Details

At the moment we use innerHTML in grid.js to create the html for each tile, this makes making updates to the html harder and also makes understanding the code base much harder, we should use the template tag instead imo.
Assignee: nobody → evold
Depends on: 1080029
I gave this a shot and ran into bug 1080029
Assignee: evold → nobody
The plan is to rewrite about:newtab in HTML. We might as well wait for this I guess.
Depends on: 1021653
Depends on: 1167601

Hello!

This bug has been closed due to inactivity and/or the potential for this bug to no longer be an issue with the new Discovery Stream-powered New Tab experience.

Please help us triage by reopening if this issue still persists and should be addressed.

Thanks!

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.