diff --git a/browser/common.py b/browser/common.py
index d353609..4e8900f 100644
--- a/browser/common.py
+++ b/browser/common.py
@@ -246,6 +246,8 @@ class BaseView(GenericView):
     @Lazy
     def xeditable(self):
         ct = getattr(self.context, 'contentType', '')
+        if ct == 'application/pdf':
+            return False
         if ct.startswith('text/'):
             return self.request.principal.id == 'rootadmin'
         return canWrite(self.context, 'title')
diff --git a/browser/resource_macros.pt b/browser/resource_macros.pt
index a33beef..c4cf78e 100644
--- a/browser/resource_macros.pt
+++ b/browser/resource_macros.pt
@@ -49,7 +49,7 @@
          tal:attributes="href string:${view/url}/.target${view/targetId}/view">
         Download
       </a>
-      <tal:xedit condition="view/xeditable"> |
+      <tal:xedit condition="item/xeditable"> |
         <a href="#" title="Edit with External Editor"
            tal:define="url string:${view/url}/.target${view/targetId}"
            tal:attributes="href string:$url/external_edit">