From 7042946bdf13af5b70dcdebe07435b09306c3bc2 Mon Sep 17 00:00:00 2001
From: Graham Knop <graham@plainblack.com>
Date: Thu, 5 Jul 2007 09:35:29 +0000
Subject: [PATCH] fix: CSS dragTrigger: multiple identical ID's on every page

---
 docs/changelog/7.x.x.txt           | 2 ++
 lib/WebGUI/Asset/Wobject/Layout.pm | 2 +-
 www/extras/draggable.css           | 2 +-
 www/extras/draggable.js            | 7 ++++---
 4 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/docs/changelog/7.x.x.txt b/docs/changelog/7.x.x.txt
index 416307af5..5068fe2c2 100644
--- a/docs/changelog/7.x.x.txt
+++ b/docs/changelog/7.x.x.txt
@@ -71,6 +71,8 @@
     http://www.webgui.org/bugs/tracker/session-variables-still-used-in-help-docs
  - fix: Editing a template sometimes fails (perlDreamer Consulting, LLC)
     http://www.webgui.org/bugs/tracker/editing-a-template-sometimes-fails
+ - fix: CSS dragTrigger: multiple identical ID's on every page
+    http://www.webgui.org/bugs/tracker/css-dragtrigger-multiple-identical-ids-on-every-page
 
 7.3.19
  - Fixed a formatting problem in the workflow editor screen.
diff --git a/lib/WebGUI/Asset/Wobject/Layout.pm b/lib/WebGUI/Asset/Wobject/Layout.pm
index 9a346ced2..7dc689a33 100644
--- a/lib/WebGUI/Asset/Wobject/Layout.pm
+++ b/lib/WebGUI/Asset/Wobject/Layout.pm
@@ -232,7 +232,7 @@ sub view {
 	if ($self->{_viewVars}{showAdmin} && $self->canEditIfLocked) {
 		# under normal circumstances we don't put HTML stuff in our code, but this will make it much easier
 		# for end users to work with our templates
-		$self->{_viewVars}{"dragger.icon"} = '<div id="dragTrigger" class="dragTrigger">'.$self->session->icon->drag('class="dragTrigger"').'</div>';
+		$self->{_viewVars}{"dragger.icon"} = '<div class="dragTrigger dragTriggerWrap">'.$self->session->icon->drag('class="dragTrigger"').'</div>';
 		$self->{_viewVars}{"dragger.init"} = '
 			<iframe id="dragSubmitter" style="display: none;" src="'.$self->session->url->extras('spacer.gif').'"></iframe>
 			<script type="text/javascript">
diff --git a/www/extras/draggable.css b/www/extras/draggable.css
index 3a12bdf4b..5d5bd846f 100644
--- a/www/extras/draggable.css
+++ b/www/extras/draggable.css
@@ -10,7 +10,7 @@ div.dragable:hover {
   padding: 0px;
   margin: 0px;
 }
-#dragTrigger{
+.dragTriggerWrap{
   width: 100%;
   height: 18px;
   font-size: 10px;
diff --git a/www/extras/draggable.js b/www/extras/draggable.js
index 28ec60942..62bfeed1b 100644
--- a/www/extras/draggable.js
+++ b/www/extras/draggable.js
@@ -58,11 +58,12 @@ function dragable_checkKeyEvent(e) {
 
 //goes up the parent tree until class is found. If not found, returns null
 function dragable_getObjectByClass(target,clazz) {
-    while (target.tagName!=topelement&&target.className!=clazz){
+    var classMatch = new RegExp("\\b" + clazz + "\\b");
+    while (target.tagName!=topelement && target.className.search(classMatch) == -1){
         target=dom? target.parentNode : target.parentElement
     }
 
-    if (target.className==clazz){
+    if (target.className.search(classMatch) != -1){
         return target;
     }else {
         return null;
@@ -202,7 +203,7 @@ function dragable_dragStart(e){
     e=dom? e : event;
     var fObj=dom? e.target : e.srcElement
 
-    if (fObj.className != "dragTrigger") {
+    if (fObj.className.search(/\bdragTrigger\b/) == -1) {
         return;
     }