Skip to content

Remove DOMContentLoaded Eventlistener

What does this MR do?

This Merge Request solves these issues: 1 2 3 4 5 6

The DOMContentLoaded has been removed, it's no longer needed and slowing down the rendering of the page. See this epic for reference.

Screenshots and URL's

1 Issue: 1.
GDK Link: http://gitlab.localdev:3000/twitter/typeahead-js/-/requirements_management/requirements

Issue: 2 Could not be reproduced by the changes are minor and can be assumed save.

3 Issue: 3.
GDK Link: http://gitlab.localdev:3000/admin/push_rule

4 Issue: 4.
GDK Link: http://gitlab.localdev:3000/admin/users/$YOURUSERNAME/edit

5 Issue: 5.
GDK Link: http://gitlab.localdev:3000/admin/groups/new

6 Issue: 6.
GDK Link: http://gitlab.localdev:3000/admin/groups/$YOURGROUPNAME/edit

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Jannik Lehmann

Merge request reports