Show abilities to hide/unhide properly

app/models/ability.rb

@@ -43,8 +43,8 @@ class Ability
       if user.admin_role?
         can :approve, Screenshot
         can :destroy, Screenshot
-        can :hide, Screenshot, approved: true, hidden: false
+        can :hide, Screenshot
-        can :unhide, Screenshot, approved: true, hidden: true
+        can :unhide, Screenshot
         can :destroy, User
         can :view, Screenshot
         can :destroy, Package
@@ -52,8 +52,8 @@ class Ability
       end
       if user.moderator_role?
         can :approve, Screenshot
-        can :hide, Screenshot, approved: true, hidden: false
+        can :hide, Screenshot
-        can :unhide, Screenshot, approved: true, hidden: true
+        can :unhide, Screenshot
         can :view, Screenshot
       end
     end

app/views/packages/_admin_buttons.slim

@@ -7,13 +7,13 @@
       span class="label adminlabel #{@statuscolor}"
         = @statustext
 
-  - if can?(:hide, screenshot)
+  - if !screenshot.hidden && screenshot.approved && can?(:hide, screenshot)
     a.button.small.bordered.radius.warning[
       href=hide_screenshot_path(screenshot.id)
       onclick="return confirm('Really hide the screenshot from public view?');"
       ] #{fa_icon 'eye-slash'} Hide
 
-  - if can?(:unhide, screenshot)
+  - if screenshot.hidden && screenshot.approved && can?(:unhide, screenshot)
     a.button.small.bordered.radius.warning[
       href=unhide_screenshot_path(screenshot.id)
       ] #{fa_icon 'eye'} Un-Hide