From eabc0564cd09a5ce77c00777e34cc1bfc7ee56d4 Mon Sep 17 00:00:00 2001
From: Aleksander Machniak <alec@alec.pl>
Date: Fri, 05 Jul 2013 04:40:32 -0400
Subject: [PATCH] Help plugin: CSS fixes, content frame scrollbar fix, about page (classic skin) fix

---
 plugins/help/skins/larry/help.css              |    1 -
 skins/classic/templates/about.html             |    7 +++++++
 skins/larry/templates/about.html               |    4 ++++
 plugins/help/help.php                          |    3 ++-
 plugins/help/skins/classic/templates/help.html |    6 +++---
 plugins/help/skins/classic/help.css            |    2 +-
 6 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/plugins/help/help.php b/plugins/help/help.php
index 66c4256..ab665d0 100644
--- a/plugins/help/help.php
+++ b/plugins/help/help.php
@@ -82,7 +82,8 @@
                 if (is_readable($this->home . '/content/about.html')) {
                     return @file_get_contents($this->home . '/content/about.html');
                 }
-                $src = $rcmail->config->get('help_about_url', $rcmail->url(array('_task' => 'settings', '_action' => 'about')));
+                $default = $rcmail->url(array('_task' => 'settings', '_action' => 'about', '_framed' => 1));
+                $src     = $rcmail->config->get('help_about_url', $default);
                 break;
 
             case 'license':
diff --git a/plugins/help/skins/classic/help.css b/plugins/help/skins/classic/help.css
index 8f67f11..ad5b7da 100644
--- a/plugins/help/skins/classic/help.css
+++ b/plugins/help/skins/classic/help.css
@@ -7,7 +7,7 @@
 
 .help-box
 {
-  overflow: auto;  
+  overflow: auto;
   background-color: #F2F2F2;
 }
 
diff --git a/plugins/help/skins/classic/templates/help.html b/plugins/help/skins/classic/templates/help.html
index 8a398e7..5bc3420 100644
--- a/plugins/help/skins/classic/templates/help.html
+++ b/plugins/help/skins/classic/templates/help.html
@@ -7,9 +7,9 @@
 <script type="text/javascript">
 function help_init_settings_tabs()
 {
-    var action, tab = '#helptabdefault';
+    var action, tab = '#helptabindex';
     if (window.rcmail && (action = rcmail.env.action)) {
-        tab = '#helptab' + (action ? action : 'default');
+        tab = '#helptab' + (action ? action : 'index');
     }
     $(tab).addClass('tablink-selected');
 }
@@ -21,7 +21,7 @@
 <roundcube:include file="/includes/header.html" />
 
 <div id="tabsbar">
-<span id="helptabdefault" class="tablink"><roundcube:button name="helpdefault" href="?_task=help" type="link" label="help.help" title="help.help" /></span>
+<span id="helptabindex" class="tablink"><roundcube:button name="helpindex" href="?_task=help" type="link" label="help.help" title="help.help" /></span>
 <span id="helptababout" class="tablink"><roundcube:button name="helpabout" href="?_task=help&amp;_action=about" type="link" label="help.about" title="help.about" class="tablink" /></span>
 <span id="helptablicense" class="tablink"><roundcube:button name="helplicense" href="?_task=help&amp;_action=license" type="link" label="help.license" title="help.license" class="tablink" /></span>
 <roundcube:container name="helptabs" id="helptabsbar" />
diff --git a/plugins/help/skins/larry/help.css b/plugins/help/skins/larry/help.css
index c2e369a..d9af6e1 100644
--- a/plugins/help/skins/larry/help.css
+++ b/plugins/help/skins/larry/help.css
@@ -39,7 +39,6 @@
 	background: url(help.png) center -130px no-repeat;
 }
 
-.iframebox.help_about,
 .iframebox.help_license {
 	overflow: auto;
 }
diff --git a/skins/classic/templates/about.html b/skins/classic/templates/about.html
index 519acf7..429dfcf 100644
--- a/skins/classic/templates/about.html
+++ b/skins/classic/templates/about.html
@@ -7,11 +7,15 @@
 </head>
 <body>
 
+<roundcube:if condition="!request:_framed" />
+
 <roundcube:include file="/includes/taskbar.html" />
 <roundcube:include file="/includes/header.html" />
 <roundcube:include file="/includes/settingstabs.html" />
 
 <div id="mainscreen" class="box darkbg crop">
+<roundcube:endif />
+
 <div class="readtext">
 <div id="license">
 <roundcube:object name="aboutcontent" />
@@ -29,7 +33,10 @@
 <roundcube:object name="pluginlist" id="pluginlist" class="records-table" cellspacing="0" />
 
 </div>
+
+<roundcube:if condition="!request:_framed" />
 </div>
+<roundcube:endif />
 
 </body>
 </html>
diff --git a/skins/larry/templates/about.html b/skins/larry/templates/about.html
index 301c301..e2bd0b0 100644
--- a/skins/larry/templates/about.html
+++ b/skins/larry/templates/about.html
@@ -4,7 +4,11 @@
 <title><roundcube:object name="pagetitle" /></title>
 <roundcube:include file="/includes/links.html" />
 </head>
+<roundcube:if condition="request:_framed" />
+<body class="iframe fullheight">
+<roundcube:else />
 <body class="ui-widget-content">
+<roundcube:endif />
 
 <div class="readtext">
 <roundcube:object name="aboutcontent" />

--
Gitblit v1.9.1