1. Code cleanup

2. Made changes so that images in IE show up properly & scale properly.

webapp/web/js/visualization/personlevel/person_level.js

@@ -19,7 +19,6 @@ function getWellFormedURLs(given_uri, type) {
 			dataType: "text",
 			async: false,
 			success:function(data){
-			// console.log("COA - " + data);
 		}
 		}).responseText;
 
@@ -73,18 +72,15 @@ function getWellFormedURLs(given_uri, type) {
 $.fn.image = function(src, successFunc, failureFunc){
 	return this.each(function(){ 
 		var profileImage = new Image();
-		profileImage.src = src;
-		profileImage.width = 90;
 		profileImage.onerror = failureFunc;
 		profileImage.onload = successFunc;
-
+		profileImage.src = src;
 
 		return profileImage;
 	});
 };
 
 function setProfileImage(imageContainerID, rawPath, contextPath) {
-
 	
 	if (imageContainerID == "") {
 		return;
@@ -103,7 +99,7 @@ function setProfileImage(imageContainerID, rawPath, contextPath) {
 		imageContainer.empty();
 	}
 	);
-
+	
 }
 
 function setProfileMoniker(monikerContainerID, moniker, doEllipsis) {
@@ -317,4 +313,4 @@ function renderCoAuthorshipVisualization() {
 		document.write(alternateContent);  // insert non-flash content
 	}
 
-}
+}

webapp/web/templates/visualization/person_level.jsp

@@ -1,5 +1,4 @@
 <%-- $This file is distributed under the terms of the license in /doc/license.txt$ --%>
-
 <%@ page import="edu.cornell.mannlib.vitro.webapp.visualization.valueobjects.SparklineVOContainer"%>
 
 <%@ taglib uri="http://java.sun.com/jstl/core" prefix="c"%>
@@ -36,6 +35,11 @@
 
 <style type="text/css">
 
+#profileImage img{
+	width: 90px;
+	height: auto;
+}
+
 #body h1 {
 	margin:0.0em;
 } 
@@ -86,40 +90,24 @@ table.sparkline_wrapper_table td, th {
 
 #ego_label {
 	font-size:1.1em;
-	/*margin-left:100px;
-	margin-top:9px;
-	position:absolute;*/
-}
-
-#ego_moniker {
-	/*margin-left:100px;
-	margin-top:27px;
-	position:absolute;*/
 }
 
 #ego_profile_image {
 	float:left;
 	padding-right: 5px;
+}
 
-	/*width: 100px;*/
+#ego_profile_image img{
+	width: 90px;
+	height: auto;
 }
 
 #ego_sparkline {
 	cursor:pointer;
 	height:36px;
-	/*
-	margin-left:10px;
-	margin-top:69px;
-	position:absolute;*/
 	width:471px;
 }
 
-#dataPanel {
-	/*
-	float: left; 
-	width: 150px; 
-	visibility:hidden;*/
-}
 
 #coauthorships_table th {
 	vertical-align: top;
@@ -167,7 +155,6 @@ table.sparkline_wrapper_table td, th {
 			</div>
 			
 		<h2 class="sub_headings">Co-Author Network 
-				<%-- A simple if/else condition --%>
 		<c:choose>
 		    <c:when test="${numOfCoAuthorShips > 0 || numOfAuthors > 0}">
 		       <a href="${coAuthorshipDownloadFile}">(.GraphML File)</a></h2>
@@ -263,13 +250,6 @@ $(document).ready(function(){
 							  "ego_profile_image",
 							  jQuery.parseJSON(getWellFormedURLs("${requestScope.egoURIParam}", "profile_info")));
 
-	/*
-	if ($('#profileMoniker').text().length > 30) {
-		//$('#profileMoniker').text($('#profileMoniker').text().substr(0, 30) + "...");
-		console.log("what");
-	}*/
-
-	
 	<c:if test='${numOfCoAuthorShips <= 0}'>
 
 		if ($('#ego_label').text().length > 0) {
@@ -278,9 +258,5 @@ $(document).ready(function(){
 		
 	</c:if>	
 
-	
-
-	  
-
 });
-</script>
+</script>