Project

General

Profile

Actions

Defect #9775

closed

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

Added by John Kubiatowicz over 12 years ago. Updated about 12 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 #1

Updated by Jean-Philippe Lang over 12 years ago

  • Category set to SCM
  • Status changed from New to Resolved
  • Target version set to 1.3.1
  • Affected version (unused) set to 1.3.0
  • Affected version set to 1.3.0

Fixed in r8209.

Actions #2

Updated by Jean-Philippe Lang about 12 years ago

  • Status changed from Resolved to Closed
  • Resolution set to Fixed

Merged.

Actions

Also available in: Atom PDF