Project

General

Profile

Actions

Cleanup #49220

closed

Tasks #49211: mgr/dashboard: accessibility, performance and security check for dashboard

mgr/dashboard: avoid using document.write()

Added by Avan Thakkar about 3 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
General
Target version:
% Done:

0%

Tags:
Backport:
pacific,octopus,nautilus
Reviewed:
Affected Versions:
Pull request ID:

Description

For users on slow connections, external scripts dynamically injected via `document.write()` can delay page load by tens of seconds. https://developers.google.com/web/updates/2016/08/removing-document-write


Related issues 4 (0 open4 closed)

Precedes Dashboard - Cleanup #50655: mgr/dashboard: dashboard not working with a URL prefixResolvedAvan Thakkar

Actions
Copied to Dashboard - Backport #49326: pacific: mgr/dashboard: avoid using document.write()ResolvedAvan ThakkarActions
Copied to Dashboard - Backport #49327: nautilus: mgr/dashboard: avoid using document.write()ResolvedAvan ThakkarActions
Copied to Dashboard - Backport #49328: octopus: mgr/dashboard: avoid using document.write()ResolvedAvan ThakkarActions
Actions #1

Updated by Avan Thakkar about 3 years ago

  • Pull request ID set to 39372
Actions #2

Updated by Ernesto Puerta about 3 years ago

Nice suggestion too! Chrome is always complaining about the use of document.write(). However that part of code was autogenerated by Angular. Perhaps is it time to regenerate the index.html?

Actions #3

Updated by Ernesto Puerta about 3 years ago

  • Tracker changed from Tasks to Cleanup
  • Status changed from In Progress to Fix Under Review
  • Backport set to pacific,octopus,nautilus

Given this has a positive impact on page load speed, added backports.

Actions #4

Updated by Ernesto Puerta about 3 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #5

Updated by Backport Bot about 3 years ago

  • Copied to Backport #49326: pacific: mgr/dashboard: avoid using document.write() added
Actions #6

Updated by Backport Bot about 3 years ago

  • Copied to Backport #49327: nautilus: mgr/dashboard: avoid using document.write() added
Actions #7

Updated by Backport Bot about 3 years ago

  • Copied to Backport #49328: octopus: mgr/dashboard: avoid using document.write() added
Actions #8

Updated by Ernesto Puerta about 3 years ago

  • Status changed from Pending Backport to Resolved
Actions #9

Updated by Ernesto Puerta almost 3 years ago

  • Precedes Cleanup #50655: mgr/dashboard: dashboard not working with a URL prefix added
Actions

Also available in: Atom PDF