www

commit e8ce1d30d2bdd0e9bc9e3e9d5260d5d214e6fea9
parent c351486ed26b69aa24be74b4757608f54488122b
Author: Simon Kornblith <simon@simonster.com>
Date:   Thu,  9 Apr 2009 07:16:41 +0000

make fake window border gray when window is inactive on OS X


Diffstat:
M
chrome/content/zotero/overlay.js
 | 
20
++++++++------------
M
chrome/skin/default/zotero/overlay.css
 | 
13
++++++++++++-

2 files changed, 20 insertions(+), 13 deletions(-)
diff --git a/chrome/content/zotero/overlay.js b/chrome/content/zotero/overlay.js
@@ -306,8 +306,7 @@ var ZoteroPane = new function()
 		zoteroSplitter.setAttribute('hidden', !makeVisible);
 		
 		// Restore fullscreen mode if necessary
-		var fullScreenMode = document.getElementById('zotero-tb-fullscreen').getAttribute('fullscreenmode') == 'true';
-		if (makeVisible && fullScreenMode) {
+		if (makeVisible && isFullScreen()) {
 			this.fullScreen(true);
 		}
 		
@@ -361,7 +360,7 @@ var ZoteroPane = new function()
 			document.getElementById('content').setAttribute('collapsed', false);
 			
 			// turn off full window mode, if it was on
-			_setFullWindowMode(false)
+			_setFullWindowMode(false);
 			
 			// Return focus to the browser content pane
 			window.content.window.focus();
@@ -378,21 +377,21 @@ var ZoteroPane = new function()
 	
 	function fullScreen(set)
 	{
-		var fs = document.getElementById('zotero-tb-fullscreen');
+		var zPane = document.getElementById('zotero-pane');
 		
 		if (set != undefined) {
-			var makeFullScreen = set;
+			var makeFullScreen = !!set;
 		}
 		else {
-			var makeFullScreen = fs.getAttribute('fullscreenmode') != 'true';
+			var makeFullScreen = zPane.getAttribute('fullscreenmode') != 'true';
 		}
 		
 		// Turn Z-pane flex on to stretch to window in full-screen, but off otherwise so persist works
-		document.getElementById('zotero-pane').setAttribute('flex', makeFullScreen ? "1" : "0");
+		zPane.setAttribute('flex', makeFullScreen ? "1" : "0");
 		document.getElementById('content').setAttribute('collapsed', makeFullScreen);
 		document.getElementById('zotero-splitter').setAttribute('hidden', makeFullScreen);
-		fs.setAttribute('fullscreenmode', makeFullScreen);
 		
+		zPane.setAttribute('fullscreenmode', makeFullScreen);
 		_setFullWindowMode(makeFullScreen);
 	}
 	
@@ -410,7 +409,6 @@ var ZoteroPane = new function()
 				titlebarcolorState = document.documentElement.getAttribute("activetitlebarcolor");
 				document.documentElement.removeAttribute("activetitlebarcolor");
 			}*/
-			if(Zotero.isMac) document.getElementById("zotero-pane").style.borderTop = "1px solid black";
 			if(document.title != "Zotero") {
 				titleState = document.title;
 				document.title = "Zotero";
@@ -426,7 +424,6 @@ var ZoteroPane = new function()
 			/*if(Zotero.isMac) {
 				document.documentElement.setAttribute("activetitlebarcolor", titlebarcolorState);
 			}*/
-			if(Zotero.isMac) document.getElementById("zotero-pane").style.borderTop = "";
 			if(document.title == "Zotero") document.title = titleState;
 			
 			if(toolbarCollapseState) {
@@ -439,8 +436,7 @@ var ZoteroPane = new function()
 	}
 	
 	function isFullScreen() {
-		var fs = document.getElementById('zotero-tb-fullscreen');
-		return fs.getAttribute('fullscreenmode') == 'true'
+		return document.getElementById('zotero-pane').getAttribute('fullscreenmode') == 'true';
 	}
 	
 	
diff --git a/chrome/skin/default/zotero/overlay.css b/chrome/skin/default/zotero/overlay.css
@@ -24,6 +24,17 @@
 	overflow: hidden;
 }
 
+/* these are hacks do display a window separator in full screen mode on OS X */
+window[activetitlebarcolor] #zotero-pane[fullscreenmode="true"]
+{
+	border-top: 1px solid #A3A3A3;
+}
+
+window[active="true"][activetitlebarcolor] #zotero-pane[fullscreenmode="true"]
+{
+	border-top: 1px solid black;
+}
+
 #zotero-collections-pane
 {
 	min-width: 150px;
@@ -240,7 +251,7 @@
 	list-style-image: url('chrome://zotero/skin/toolbar-fullscreen-top.png');
 }
 
-#zotero-tb-fullscreen[fullscreenmode="true"]
+#zotero-pane[fullscreenmode="true"] #zotero-tb-fullscreen
 {
 	background: #666666;
 	-moz-border-radius: 6px;