From 6810a1de4213fb1e4354e410e3cfa7e10df12db9 Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Tue, 11 Aug 2015 14:53:07 +0300 Subject: [PATCH] use dijit form widget for attachments dropdown --- classes/feeds.php | 5 +++-- css/tt-rss.css | 5 ++++- include/functions2.php | 38 +++++++++++++++----------------------- js/viewfeed.js | 14 -------------- 4 files changed, 22 insertions(+), 40 deletions(-) diff --git a/classes/feeds.php b/classes/feeds.php index ced6bbb4a..afc720957 100644 --- a/classes/feeds.php +++ b/classes/feeds.php @@ -701,12 +701,13 @@ class Feeds extends Handler_Protected { $always_display_enclosures = sql_bool_to_bool($line["always_display_enclosures"]); - $reply['content'] .= format_article_enclosures($id, $always_display_enclosures, $line["content"], sql_bool_to_bool($line["hide_images"])); - $reply['content'] .= ""; $reply['content'] .= "
"; + $reply['content'] .= format_article_enclosures($id, $always_display_enclosures, $line["content"], sql_bool_to_bool($line["hide_images"])); + $reply['content'] .= "
"; + foreach (PluginHost::getInstance()->get_hooks(PluginHost::HOOK_ARTICLE_LEFT_BUTTON) as $p) { $reply['content'] .= $p->hook_article_left_button($line); } diff --git a/css/tt-rss.css b/css/tt-rss.css index 38c255950..789319ccb 100644 --- a/css/tt-rss.css +++ b/css/tt-rss.css @@ -52,7 +52,6 @@ div.postReply div.postContent video { } div.postReply div.postEnclosures { - margin-top : 1em; color : #555; } @@ -1114,3 +1113,7 @@ body#ttrssMain #feedTree .dijitTreeRow img.dijitTreeExpandoLeaf { :focus { outline: none; } + +.dijitDropDownButton.attachments { + margin-bottom : 10px; +} \ No newline at end of file diff --git a/include/functions2.php b/include/functions2.php index 03b65b7b1..a6cb019f8 100644 --- a/include/functions2.php +++ b/include/functions2.php @@ -1314,21 +1314,6 @@ - "; } @@ -1446,10 +1431,13 @@ $rv['content'] .= "
"; $rv['content'] .= $line["content"]; - $rv['content'] .= format_article_enclosures($id, - sql_bool_to_bool($line["always_display_enclosures"]), - $line["content"], - sql_bool_to_bool($line["hide_images"])); + + if (!$zoom_mode) { + $rv['content'] .= format_article_enclosures($id, + sql_bool_to_bool($line["always_display_enclosures"]), + $line["content"], + sql_bool_to_bool($line["hide_images"])); + } $rv['content'] .= "
"; @@ -1988,8 +1976,10 @@ $rv .= "
"; } - $rv .= ""; + $rv .= "
"; + $rv .= ""; } return $rv; diff --git a/js/viewfeed.js b/js/viewfeed.js index 16b3bb1ee..85872556e 100644 --- a/js/viewfeed.js +++ b/js/viewfeed.js @@ -2347,20 +2347,6 @@ function displayArticleUrl(id) { } } -function openSelectedAttachment(elem) { - try { - var url = elem[elem.selectedIndex].value; - - if (url) { - window.open(url); - elem.selectedIndex = 0; - } - - } catch (e) { - exception_error("openSelectedAttachment", e); - } -} - function scrollToRowId(id) { try { var row = $(id);