diff --git a/app/policies/plan_policy.rb b/app/policies/plan_policy.rb
index 870cd41..136ae45 100644
--- a/app/policies/plan_policy.rb
+++ b/app/policies/plan_policy.rb
@@ -14,7 +14,7 @@
   end
 
   def share?
-    @plan.editable_by?(@user.id)
+    @plan.editable_by?(@user.id) 
   end
 
   def export?
diff --git a/app/views/plans/_navigation.html.erb b/app/views/plans/_navigation.html.erb
index d21eab0..1319bba 100644
--- a/app/views/plans/_navigation.html.erb
+++ b/app/views/plans/_navigation.html.erb
@@ -13,14 +13,17 @@
       </a>
     </li>
   <% end %>
-  <% if plan.editable_by?(current_user) %>
-    <li role="presentation" class="<%= (isActivePage(share_plan_path(plan)) ? 'active' : '') %>">
-      <a href="<%= share_plan_path(plan) %>" role="tab" aria-controls="content"><%= _('Share') %></a>
+  <% if plan.users.include?(current_user) %>
+    <% if plan.editable_by?(current_user) %>
+      <li role="presentation" class="<%= (isActivePage(share_plan_path(plan)) ? 'active' : '') %>">
+        <a href="<%= share_plan_path(plan) %>" role="tab" aria-controls="content"><%= _('Share') %></a>
+      </li>
+    <% end %>
+
+    <li role="presentation" class="<%= (isActivePage(download_plan_path(plan)) ? 'active' : '') %>">
+      <a href="<%= download_plan_path(plan) %>" role="tab" aria-controls="content"><%= _('Download') %></a>
     </li>
   <% end %>
-  <li role="presentation" class="<%= (isActivePage(download_plan_path(plan)) ? 'active' : '') %>">
-    <a href="<%= download_plan_path(plan) %>" role="tab" aria-controls="content"><%= _('Download') %></a>
-  </li>
 </ul>
 <div class="tab-content">
   <div id="content" role="tabpanel" class="tab-pane active">