Opened 8 years ago

Closed 8 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: dev
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 by kapil garg, 8 years ago

I can work on this.

comment:2 by Abhishek Gautam, 8 years ago

Owner: set to Abhishek Gautam
Status: new → assigned

I am working on this.

comment:3 by Tim Graham, 8 years ago

Has patch: set
Patch needs improvement: set

​PR

comment:4 by Tim Graham <timograham@…>, 8 years ago

Resolution: → fixed
Status: assigned → closed

In 941b8691:

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

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