Permalink
Browse files

MDL-25676 Add back missing test for editing_assign class

This had caused the assign roles button disappeared in AJAX view.

Note - this does not fix the issue with the cloned buttons being resized
(MDL-25682)

Signed-off-by: Andrew Robert Nicols <andrew.nicols@luns.net.uk>
Signed-off-by: Dan Poltawski <dan.poltawski@luns.net.uk>
  • Loading branch information...
1 parent f056cb5 commit 1a0d7bd7852e3257b7e5c89fd92f8ba527cd1457 @andrewnicols andrewnicols committed with danpoltawski Dec 14, 2010
Showing with 8 additions and 0 deletions.
  1. +8 −0 lib/ajax/section_classes.js
@@ -631,6 +631,7 @@ resource_class.prototype.init_buttons = function() {
var moveLeft = false;
var moveRight = false;
var updateButton = null;
+ var assignButton = null;
// for RTL support
var isrtl = (document.getElementsByTagName("html")[0].dir=="rtl");
@@ -642,6 +643,8 @@ resource_class.prototype.init_buttons = function() {
moveRight = true;
} else if (buttons[x].className == 'editing_update') {
updateButton = buttons[x].cloneNode(true);
+ } else if (buttons[x].className == 'editing_assign') {
+ assignButton = buttons[x].cloneNode(true);
} else if (buttons[x].className == 'editing_groupsnone') {
this.groupmode = this.NOGROUPS;
} else if (buttons[x].className == 'editing_groupsseparate') {
@@ -717,6 +720,11 @@ resource_class.prototype.init_buttons = function() {
commandContainer.appendChild(button);
this.groupButton = button;
}
+
+ // Add the assign roles button back in
+ if (assignButton != null) {
+ commandContainer.appendChild(assignButton);
+ }
};

0 comments on commit 1a0d7bd

Please sign in to comment.