diff --git a/app/policies/plan_policy.rb b/app/policies/plan_policy.rb
index 11dd045..78c674b 100644
--- a/app/policies/plan_policy.rb
+++ b/app/policies/plan_policy.rb
@@ -14,7 +14,7 @@
   end
 
   def share?
-    @plan.readable_by?(@user.id) && Role.find_by(user_id: @user.id, plan_id: @plan.id).active
+    @plan.editable_by?(@user.id) && Role.find_by(user_id: @user.id, plan_id: @plan.id).active
   end
 
   def export?
@@ -25,6 +25,10 @@
     @plan.readable_by?(@user.id) && Role.find_by(user_id: @user.id, plan_id: @plan.id).active
   end
 
+  def edit?
+    @plan.readable_by?(@user.id) && Role.find_by(user_id: @user.id, plan_id: @plan.id).active
+  end
+
   def update?
     @plan.editable_by?(@user.id) && Role.find_by(user_id: @user.id, plan_id: @plan.id).active
   end
diff --git a/app/views/plans/_navigation.html.erb b/app/views/plans/_navigation.html.erb
index 55658d0..6403257 100644
--- a/app/views/plans/_navigation.html.erb
+++ b/app/views/plans/_navigation.html.erb
@@ -9,9 +9,11 @@
       </a>
     </li>
   <% end %>
-  <li role="presentation" class="<%= (isActivePage(share_plan_path(plan)) ? 'active' : '') %>">
-    <a href="<%= share_plan_path(plan) %>" role="tab" aria-controls="content"><%= _('Share') %></a>
-  </li>
+  <% 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>
diff --git a/app/views/plans/show.html.erb b/app/views/plans/show.html.erb
index df98d71..f44bae3 100644
--- a/app/views/plans/show.html.erb
+++ b/app/views/plans/show.html.erb
@@ -10,7 +10,7 @@
     <% if @plan.editable_by?(current_user) %>
       <%= render partial: 'edit_details', layout: 'navigation', locals: {plan: @plan, visibility: @visibility} %>
     <% else %>
-      <%= render partial: 'show_details', locals: {plan: @plan, visibility: @visibility} %>
+      <%= render partial: 'show_details', layout: 'navigation', locals: { plan: @plan, visibility: @visibility } %>
     <% end %>
   </div>
 </div>
\ No newline at end of file