have arrow removal hover effect working

pull/6827/head
Philipp Tessenow 14 years ago
parent a2a92dd0b7
commit 91ca4af0d5
  1. 12
      assets/javascripts/reporting/group_bys.js
  2. 4
      assets/stylesheets/reporting.css

@ -102,12 +102,24 @@ Reporting.GroupBys = {
}); });
}, },
arrow_removal_hover: function(arrow, status) {
if (status) {
$(arrow).addClassName('arrow_removal_hover');
}
else {
$(arrow).removeClassName('arrow_removal_hover');
}
},
init_arrow: function(group_by) { init_arrow: function(group_by) {
var arrow = new Element('img', { var arrow = new Element('img', {
'class': 'arrow in_row arrow_left', 'class': 'arrow in_row arrow_left',
'id': group_by.identify() + '_arrow' 'id': group_by.identify() + '_arrow'
}); });
//init_arrow_hover_effects(arrow); //init_arrow_hover_effects(arrow);
arrow.observe('mouseover', function() { Reporting.GroupBys.arrow_removal_hover(arrow, true) });
arrow.observe('mouseout', function() { Reporting.GroupBys.arrow_removal_hover(arrow, false) });
arrow.observe('mousedown', function() { Reporting.GroupBys.remove_group_by(arrow) });
return arrow; return arrow;
}, },

@ -304,7 +304,7 @@ fieldset#filter-settings table td > label {
background-image: url(../images/group_by_arrow_both_hover_left.png); background-image: url(../images/group_by_arrow_both_hover_left.png);
} }
.arrow_both_remove { .arrow_both.arrow_removal_hover {
background-image: url(../images/group_by_arrow_both_remove.png); background-image: url(../images/group_by_arrow_both_remove.png);
} }
@ -316,7 +316,7 @@ fieldset#filter-settings table td > label {
background-image: url(../images/group_by_arrow_left.png); background-image: url(../images/group_by_arrow_left.png);
} }
.arrow_left_remove { .arrow_left.arrow_removal_hover {
background-image: url(../images/group_by_arrow_left_remove.png); background-image: url(../images/group_by_arrow_left_remove.png);
} }

Loading…
Cancel
Save