Opened 2 years ago

Closed 2 years ago

#28008 closed Cleanup/optimization (fixed)

Remove getElementsByClassName() function in technical 500 template

Reported by: Tim Graham Owned by: Abhishek Gautam
Component: Error reporting Version: master
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: yes UI/UX: no

Description

Modern browsers have document.getElementsByClassName() so I don't think there's a need for a custom version.

Please wait until #28007 is completed before working on this to avoid a merge conflict.

Change History (4)

comment:1 Changed 2 years ago by kapil garg

I can work on this.

comment:2 Changed 2 years ago by Abhishek Gautam

Owner: set to Abhishek Gautam
Status: newassigned

I am working on this.

comment:3 Changed 2 years ago by Tim Graham

Has patch: set
Patch needs improvement: set

comment:4 Changed 2 years ago by Tim Graham <timograham@…>

Resolution: fixed
Status: assignedclosed

In 941b8691:

Fixed #28008 -- Replaced getElementsByClassName() JavaScript in debug view template.

Note: See TracTickets for help on using tickets.
Back to Top