From 7ad882ff2de98ae325477d6c14cee18a1c0f88bf Mon Sep 17 00:00:00 2001 From: Mayhem Date: Fri, 10 May 2013 05:47:19 +0200 Subject: [PATCH] Only add the "Show announcement" to the header's menu if there is an annoucement on this page. This simplifies the open function. --- src/Miscellaneous/PSAHiding.coffee | 25 +++++++++++-------------- 1 file changed, 11 insertions(+), 14 deletions(-) diff --git a/src/Miscellaneous/PSAHiding.coffee b/src/Miscellaneous/PSAHiding.coffee index ada322e4e..9f69780b0 100644 --- a/src/Miscellaneous/PSAHiding.coffee +++ b/src/Miscellaneous/PSAHiding.coffee @@ -5,20 +5,6 @@ PSAHiding = $.addClass doc, 'hide-announcement' $.addClass doc, 'hide-announcement-enabled' - entry = - type: 'header' - el: $.el 'a', - textContent: 'Show announcement' - className: 'show-announcement' - href: 'javascript:;' - order: 50 - open: -> - if $.id('globalMessage')?.hidden - return true - false - $.event 'AddMenuEntry', entry - - $.on entry.el, 'click', PSAHiding.toggle $.on d, '4chanXInitFinished', @setup setup: -> $.off d, '4chanXInitFinished', PSAHiding.setup @@ -28,6 +14,17 @@ PSAHiding = $.rmClass doc, 'hide-announcement-enabled' return + entry = + type: 'header' + el: $.el 'a', + textContent: 'Show announcement' + className: 'show-announcement' + href: 'javascript:;' + order: 50 + open: -> psa.hidden + $.event 'AddMenuEntry', entry + $.on entry.el, 'click', PSAHiding.toggle + PSAHiding.btn = btn = $.el 'a', innerHTML: '[ - ]' title: 'Hide announcement.'