Project

General

Profile

Actions

Defect #9775

closed

app/views/repository/_revision_graph.html.erb sets window.onload directly..

Added by John Kubiatowicz about 13 years ago. Updated almost 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
SCM
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

The subject says it all. The partial _revision_graph.html.erb (in app/views/repository) sets window.onload directly. As a result of setting window.onload directly, it toasts other onload events on the repository page (currently messing with a plugin that I have).

Instead of:

window.onload = function() { stuff; };

should use the prototype syntax used elsewhere in Redmine:

Event.observe(window,"load",function(){ stuff; });

I believe that this started showing up when I pulled down the 1.2.2 updates (not entire sure of timeline)

Actions

Also available in: Atom PDF