Added scrollbars to the profiles list so it's never taller than the i…

…frame.
georgebrock · May 13, 2009 · b42872e · b42872e
1 parent 7eb5b03
commit b42872e
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 8 deletions.
diff --git a/xfndiscovery.css b/xfndiscovery.css
@@ -63,8 +63,6 @@
 #xfn-discovery h5
 {
 	width:auto;
-	float:left;
-	clear:left;
 	font-size:1em;
 	margin:1em 0 0;
 	padding:0 1em;
@@ -82,14 +80,20 @@
 	cursor:pointer;
 }
 
-#xfn-discovery ul.profiles
+#xfn-discovery div.profiles-container
 {
 	margin:1em 0 0 0;
-	padding:0;
-	list-style:none;
 	width:25%;
 	float:left;
 	clear:left;
+	height:480px;
+	overflow:auto;
+}
+
+#xfn-discovery ul.profiles
+{
+	padding:0;
+	list-style:none;
 }
 
 #xfn-discovery ul.profiles a

diff --git a/xfndiscovery.user.js b/xfndiscovery.user.js
@@ -272,9 +272,12 @@ XFNDiscovery.UI = {
 				.append("<h4>More user profiles</h4>")
 				.append($iframe)
 				.append($iframeAlternative)
-				.append($profileList)
-				.append($unknownTitle)
-				.append($unknownProfileList);
+				.append(
+					$("<div></div>")
+						.addClass("profiles-container")
+						.append($profileList)
+						.append($unknownTitle)
+						.append($unknownProfileList));
 
 			for(var i = 0, p; i < XFNDiscovery.profiles.length; i++)
 			{