From 835cf78a2bd7a099f0462000ef71fa6c20c4e8b8 Mon Sep 17 00:00:00 2001
From: cdtank <cdtank@440791e1-c2bf-4c1e-ad7c-caf3f8b0ebe8>
Date: Fri, 9 Jul 2010 02:01:22 +0000
Subject: [PATCH] 1. Made minor style change for short sparkline. 2. Added more
 conditions & appropriate text for edge cases like 0 co-authorships.

---
 .../entity/sparklineVisualization.jsp         |  2 +-
 .../templates/visualization/person_level.jsp  | 60 +++++++++----------
 2 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/webapp/web/templates/entity/sparklineVisualization.jsp b/webapp/web/templates/entity/sparklineVisualization.jsp
index dfefabedb..f32b460fc 100644
--- a/webapp/web/templates/entity/sparklineVisualization.jsp
+++ b/webapp/web/templates/entity/sparklineVisualization.jsp
@@ -81,7 +81,7 @@
 			margin-left:24%;
 			margin-top:-2%;
 			position:absolute;
-			width:380px;
+			/*width:380px;*/
 		}
 	</style>
 	
diff --git a/webapp/web/templates/visualization/person_level.jsp b/webapp/web/templates/visualization/person_level.jsp
index ace26c805..ede5e0ae4 100644
--- a/webapp/web/templates/visualization/person_level.jsp
+++ b/webapp/web/templates/visualization/person_level.jsp
@@ -162,7 +162,12 @@
 		    </c:when>
 		    <c:otherwise>
 		        </h2>
-		        <span id="no_coauthorships">Currently there are no multi-author papers for <span id="no_coauthorships_person">this author</span> in the VIVO database.</span>
+		        
+		        <c:if test='${numOfAuthors > 0}'>
+		        	<c:set var='authorsText' value='multi-author' />
+		        </c:if>
+		        
+		        <span id="no_coauthorships">Currently there are no ${authorsText} papers for <span id="no_coauthorships_person">this author</span> in the VIVO database.</span>
 		    </c:otherwise>
 		</c:choose>
 		
@@ -208,53 +213,48 @@
 
 </c:if>
 
-<div class="vis_stats">
+<c:if test='${numOfAuthors > 0}'>
 
-<h2 class="sub_headings">Tables</h2>
-
-	<div class="vis-tables">
-		<p id="publications_table_container" class="datatable">
-			${egoPubSparkline.table} 
-		</p>
-	</div>
+	<div class="vis_stats">
+	
+	<h2 class="sub_headings">Tables</h2>
 	
-	<c:if test='${numOfCoAuthorShips > 0}'>
-
 		<div class="vis-tables">
-			<p id="coauth_table_container" class="datatable"></p>
+			<p id="publications_table_container" class="datatable">
+				${egoPubSparkline.table} 
+			</p>
 		</div>
+		
+		<c:if test='${numOfCoAuthorShips > 0}'>
 	
-	</c:if>
+			<div class="vis-tables">
+				<p id="coauth_table_container" class="datatable"></p>
+			</div>
+		
+		</c:if>
+	
+	</div>
+</c:if>
 
 </div>
 
-
-</div>
-
-
-
 <script language="JavaScript" type="text/javascript">
 $(document).ready(function(){
 
-	<c:choose>
-	    <c:when test='${numOfCoAuthorShips > 0}'>
+		<c:if test='${numOfCoAuthorShips > 0}'>
 	    	$("#coauth_table_container").empty().html('<img id="loadingData" with="auto" src="${loadingImageLink}" />');
-	    </c:when>
-	    <c:otherwise>
-	    	setProfileName('no_coauthorships_person', $('#ego_label').text());
-	    </c:otherwise>
-	</c:choose>
-
-	
-	
+	    </c:if>
+	    	
 	processProfileInformation("ego_label", 
 							  "ego_moniker",
 							  "ego_profile_image",
 							  jQuery.parseJSON(getWellFormedURLs("${requestScope.egoURIParam}", "profile_info")));
 
-	
+	<c:if test='${numOfCoAuthorShips <= 0}'>
+		setProfileName('no_coauthorships_person', $('#ego_label').text());
+	</c:if>	
 
-	 
+