From 38325ad6d181b276ecf851062be8cda1e58105b7 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Wed, 6 Feb 2013 11:09:45 +0400
Subject: [PATCH] remove tt-rss tabcontrol

---
 include/functions.php |  2 +-
 index.php             |  5 +--
 js/feedlist.js        |  4 ---
 js/viewfeed.js        | 72 ++++++-------------------------------------
 tt-rss.css            | 12 +++-----
 5 files changed, 15 insertions(+), 80 deletions(-)

diff --git a/include/functions.php b/include/functions.php
index c32d31f6..b2d93acd 100644
--- a/include/functions.php
+++ b/include/functions.php
@@ -2910,7 +2910,7 @@
 
 			$rv['content'] .= "<div class=\"postReply\" id=\"POST-$id\">";
 
-			$rv['content'] .= "<div onclick=\"return postClicked(event, $id)\"
+			$rv['content'] .= "<div onclick=\"return hlClicked(event, $id)\"
 				class=\"postHeader\" id=\"POSTHDR-$id\">";
 
 			$entry_author = $line["author"];
diff --git a/index.php b/index.php
index 0d0646c7..6ddf5de6 100644
--- a/index.php
+++ b/index.php
@@ -153,9 +153,7 @@
 </div>
 
 <div dojoType="dijit.layout.BorderContainer" region="center" id="header-wrap" gutters="false">
-<div dojoType="dijit.layout.TabContainer" region="center" id="content-tabs">
-<div dojoType="dijit.layout.BorderContainer" region="center" id="content-wrap"
-	title="<?php echo __("News") ?>">
+<div dojoType="dijit.layout.BorderContainer" region="center" id="content-wrap">
 
 <div id="toolbar" dojoType="dijit.layout.ContentPane" region="top">
 	<div id="main-toolbar" dojoType="dijit.Toolbar">
@@ -249,7 +247,6 @@
 </div>
 </div>
 </div>
-</div>
 
 <?php db_close($link); ?>
 
diff --git a/js/feedlist.js b/js/feedlist.js
index f4196a72..72bc436c 100644
--- a/js/feedlist.js
+++ b/js/feedlist.js
@@ -107,10 +107,6 @@ function viewfeed(feed, method, is_cat, offset, background, infscroll_req) {
 			} */
 		}
 
-		if (offset == 0 && !background)
-			dijit.byId("content-tabs").selectChild(
-				dijit.byId("content-tabs").getChildren()[0]);
-
 		if (!background) {
 			_viewfeed_last = get_timestamp();
 
diff --git a/js/viewfeed.js b/js/viewfeed.js
index 3ffa134e..a950ec7a 100644
--- a/js/viewfeed.js
+++ b/js/viewfeed.js
@@ -1372,10 +1372,6 @@ function zoomToArticle(event, id) {
 	try {
 		var cached_article = cache_get("article: " + id);
 
-		if (dijit.byId("ATAB-" + id))
-			if (!event || !event.shiftKey)
-				return dijit.byId("content-tabs").selectChild(dijit.byId("ATAB-" + id));
-
 		if (dijit.byId("ATSTRTIP-" + id))
 			dijit.byId("ATSTRTIP-" + id).destroyRecursive();
 
@@ -1388,11 +1384,6 @@ function zoomToArticle(event, id) {
 				id: 'ATAB-' + id,
 				closable: true });
 
-			dijit.byId("content-tabs").addChild(article_pane);
-
-			if (!event || !event.shiftKey)
-				dijit.byId("content-tabs").selectChild(article_pane);
-
 			if ($("PTITLE-" + id))
 				article_pane.attr('title', $("PTITLE-" + id).innerHTML);
 
@@ -1420,11 +1411,6 @@ function zoomToArticle(event, id) {
 							id: 'ATAB-' + id,
 							closable: true });
 
-						dijit.byId("content-tabs").addChild(article_pane);
-
-						if (!event || !event.shiftKey)
-							dijit.byId("content-tabs").selectChild(article_pane);
-
 						if ($("PTITLE-" + id))
 							article_pane.attr('title', $("PTITLE-" + id).innerHTML);
 					}
@@ -1612,7 +1598,8 @@ function cdmClicked(event, id) {
 			}
 
 			toggleUnread(id, 0, false);
-			zoomToArticle(event, id);
+
+			openArticleInNewWindow(id);
 		}
 
 		request_counters();
@@ -1624,45 +1611,18 @@ function cdmClicked(event, id) {
 	return false;
 }
 
-function postClicked(event, id) {
-	try {
-
-		if (!event.ctrlKey) {
-			return true;
-		} else {
-			postOpenInNewTab(event, id);
-			return false;
-		}
-
-	} catch (e) {
-		exception_error("postClicked");
-	}
-}
-
-function hlOpenInNewTab(event, id) {
-	toggleUnread(id, 0, false);
-	zoomToArticle(event, id);
-}
-
-function postOpenInNewTab(event, id) {
-	closeArticlePanel(id);
-	zoomToArticle(event, id);
-}
-
 function hlClicked(event, id) {
 	try {
 		if (event.which == 2) {
 			view(id);
 			return true;
-		} else if (event.altKey) {
+		} else if (event.ctrlKey) {
+			toggleSelected(id, true);
+			toggleUnread(id, 0, false);
 			openArticleInNewWindow(id);
-		} else if (!event.ctrlKey) {
-			view(id);
 			return false;
 		} else {
-			toggleSelected(id);
-			toggleUnread(id, 0, false);
-			zoomToArticle(event, id);
+			view(id);
 			return false;
 		}
 
@@ -1775,17 +1735,9 @@ function headlineActionsChange(elem) {
 
 function closeArticlePanel() {
 
-	var tabs = dijit.byId("content-tabs");
-	var child = tabs.selectedChildWidget;
-
-	if (child && tabs.getIndexOfChild(child) > 0) {
-		tabs.removeChild(child);
-		child.destroy();
-	} else {
-		if (dijit.byId("content-insert"))
-			dijit.byId("headlines-wrap-inner").removeChild(
-				dijit.byId("content-insert"));
-	}
+	if (dijit.byId("content-insert"))
+		dijit.byId("headlines-wrap-inner").removeChild(
+			dijit.byId("content-insert"));
 }
 
 function initHeadlinesMenu() {
@@ -1836,12 +1788,6 @@ function initHeadlinesMenu() {
 				openArticleInNewWindow(this.getParent().callerRowId);
 			}}));
 
-		menu.addChild(new dijit.MenuItem({
-			label: __("View in a tt-rss tab"),
-			onClick: function(event) {
-				hlOpenInNewTab(event, this.getParent().callerRowId);
-				}}));
-
 		menu.addChild(new dijit.MenuSeparator());
 
 		menu.addChild(new dijit.MenuItem({
diff --git a/tt-rss.css b/tt-rss.css
index 5f4f69f9..b636140d 100644
--- a/tt-rss.css
+++ b/tt-rss.css
@@ -1079,8 +1079,10 @@ div.fatalError textarea {
 
 #content-wrap {
 	padding : 0px;
-	border-width : 0px;
-	margin : 0px;
+	border-width : 1px 0px 1px 1px;
+	border-style : solid;
+  	border-color : #c0c0c0;
+	margin : 25px 0px 0px 0px;
 }
 
 #feeds-holder {
@@ -1111,12 +1113,6 @@ div.fatalError textarea {
 	display : none;
 }
 
-#content-tabs {
-	margin : 0px;
-	padding : 0px;
-	border-width : 0px;
-}
-
 #toolbar {
 	padding : 0px;
 	margin : 0px;
-- 
2.39.2