From 32c70d19abbbe6ca85e778a3711e39d6e3873727 Mon Sep 17 00:00:00 2001 From: Gregor Schmidt Date: Fri, 25 Feb 2011 18:49:58 +0100 Subject: [PATCH] adding $() wrappers around native nodes to please IE fixes #18653 --- assets/javascripts/reporting/group_bys.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/assets/javascripts/reporting/group_bys.js b/assets/javascripts/reporting/group_bys.js index 87e3efd3d3..2aec5a6c48 100644 --- a/assets/javascripts/reporting/group_bys.js +++ b/assets/javascripts/reporting/group_bys.js @@ -12,7 +12,7 @@ Reporting.GroupBys = { axis = "rows"; } - var selected_container = btn.form.select("#group_by_" + axis)[0]; + var selected_container = $(btn.form).select("#group_by_" + axis)[0]; var group_by_container = btn.form.group_by_container; var source_container, target_container; @@ -31,7 +31,7 @@ Reporting.GroupBys = { attach_sort_button: function (direction, axis) { var btn = $$(".buttons.group_by.sort.sort" + direction + ".sort-" + axis)[0]; - var box = btn.form.select("#group_by_" + axis)[0]; + var box = $(btn.form).select("#group_by_" + axis)[0]; btn.observe("click", function () { if (direction === "Up") { moveOptionUp(box);