Small modification to the way styles are assigned to authors in author listing on add authors form

productMods/edit/forms/addAuthorsToInformationResource.jsp

@@ -308,7 +308,7 @@ SPARQL queries for existing values. --%>
     Collections.sort(authorships, comp);
         
     int maxRank = 0;
-    int authorshipCount = 0;  
+    int authorshipCount = authorships.size();  
 
     // for ( ObjectPropertyStatement stmt : authorshipStmts) {
     //     Individual authorship = stmt.getObject();
@@ -319,7 +319,6 @@ SPARQL queries for existing values. --%>
     
 <%    
     for ( Individual authorship : authorships ) {
-        authorshipCount++;
 
         request.setAttribute("authorshipUri", authorship.getURI());
         request.setAttribute("authorshipName", authorship.getName());
@@ -340,6 +339,7 @@ SPARQL queries for existing values. --%>
         request.setAttribute("numericRank", numericRank);
        
         request.setAttribute("author", authorship.getRelatedIndividual(vivoCore + "linkedAuthor")); 
+
 %> 
         <li class="authorship">
             <%-- span.author will be used in the next phase, when we display a message that the author has been

productMods/edit/forms/css/addAuthorsToInformationResource.css

@@ -9,10 +9,10 @@
     margin-bottom: .75em;
 }
 
-#authorships.dd li,
-#authorships.ui-sortable li {
+#authorships li.dd {
     padding-left: 1em;
     background: url("../images/sortable_icon.png") no-repeat left center;  
+    cursor: move;  
 }
 
 #authorships .authorNameWrapper {
@@ -20,10 +20,6 @@
     width: 15em;
 }
 
-#authorships.dd li {
-    cursor: move;
-}
-
 #showAddForm span.or {
     display: none;
 }

productMods/edit/forms/js/addAuthorsToInformationResource.js

@@ -223,7 +223,6 @@ var addAuthorForm = {
             // or click on the selection with the mouse. This appears to confuse some users.
             select: function(event, ui) {
                 addAuthorForm.showSelectedAuthor(ui); 
-                return false;
             }
         });
 
@@ -309,6 +308,8 @@ var addAuthorForm = {
             return;
         }
         
+        authorships.addClass('dd');
+        
         $('.authorNameWrapper').each(function() {
             $(this).attr('title', 'Drag and drop to reorder authors');
         });
@@ -662,10 +663,9 @@ var addAuthorForm = {
     	authorships.sortable({ disable: true } );
         authorships.removeClass('dd');
         
-    	authorship.css('background', 'none');
-    	authorship.css('padding-left', '0');
+        authorship.removeClass('dd');  
               
-    	authorNameWrapper.attr('title', '');
+    	authorNameWrapper.removeAttr('title');
     },
 
     // RY To be implemented later.