diff --git a/chrome/content/zotero/bindings/attachmentbox.xml b/chrome/content/zotero/bindings/attachmentbox.xml index 4d7f98b343..e105ad643e 100644 --- a/chrome/content/zotero/bindings/attachmentbox.xml +++ b/chrome/content/zotero/bindings/attachmentbox.xml @@ -99,7 +99,6 @@ this.displayFileName = true; this.displayAccessed = true; this.displayNote = true; - this.displayButton = true; this.displayDateModified = true; break; @@ -346,28 +345,24 @@ // Note editor var noteEditor = this._id('attachment-note-editor'); - if (this.displayNote) { - if (this.displayNoteIfEmpty || this.item.getNote() != '') { - Zotero.debug("setting links on top"); - noteEditor.linksOnTop = true; - noteEditor.hidden = false; - - // Don't make note editable (at least for now) - if (this.mode == 'merge' || this.mode == 'mergeedit') { - noteEditor.mode = 'merge'; - noteEditor.displayButton = false; - } - else { - noteEditor.mode = this.mode; - } - noteEditor.parent = null; - noteEditor.item = this.item; + if (this.displayNote && (this.displayNoteIfEmpty || this.item.getNote() != '')) { + noteEditor.linksOnTop = true; + noteEditor.hidden = false; + + // Don't make note editable (at least for now) + if (this.mode == 'merge' || this.mode == 'mergeedit') { + noteEditor.mode = 'merge'; + noteEditor.displayButton = false; } + else { + noteEditor.mode = this.mode; + } + noteEditor.parent = null; + noteEditor.item = this.item; } else { noteEditor.hidden = true; } - if (this.displayButton) { selectButton.label = this.buttonCaption; @@ -572,40 +567,42 @@ - diff --git a/chrome/skin/default/zotero/bindings/attachmentbox.css b/chrome/skin/default/zotero/bindings/attachmentbox.css index 22093505b7..f86c106f4b 100644 --- a/chrome/skin/default/zotero/bindings/attachmentbox.css +++ b/chrome/skin/default/zotero/bindings/attachmentbox.css @@ -1,16 +1,16 @@ +#metadata { + padding: 5px 2px 2px 2px; +} + #title { font-weight: bold; - margin: 4px 5px 1px 5px !important; - padding: 0 2px 0 2px; /* Don't collapse blank attachment titles, since it prevents renaming */ min-height: 1.25em; } -#url -{ - margin: 1px 5px 2px 5px !important; - padding: 0 2px 0 2px !important; +#metadata > label { + margin: 6px 10px 4px !important; } #index-box diff --git a/chrome/skin/default/zotero/merge.css b/chrome/skin/default/zotero/merge.css index 08c46383fd..8a6df94035 100644 --- a/chrome/skin/default/zotero/merge.css +++ b/chrome/skin/default/zotero/merge.css @@ -74,6 +74,10 @@ zoteromergepane *[anonid="delete-box"] { -moz-box-pack: center; } +zoteromergepane .groupbox-body { + padding: 0; +} + zoteromergepane[selected=true] groupbox caption { color: red; font-weight: bold; @@ -96,3 +100,7 @@ zoteromergepane div[anonid=parent-row] { zoteromergepane div[anonid=parent-row] span { font-weight: bold; } + +zoteromergepane zoteronoteeditor { + margin: 10px 0 0; +} diff --git a/resource/tinymce/css/note-ui.css b/resource/tinymce/css/note-ui.css index ac73940dfe..509091d02d 100644 --- a/resource/tinymce/css/note-ui.css +++ b/resource/tinymce/css/note-ui.css @@ -24,6 +24,15 @@ html, body { right: 0; } +/* Adjustments for read-only mode */ +.mce-container.readonly .mce-container-body .mce-edit-area { + top: 0; +} + +.mce-container.readonly .mce-panel { + border: 0; +} + /* Shrink the buttons a bit */ .mce-btn-small button { padding-left: 3px !important; diff --git a/resource/tinymce/noteview.html b/resource/tinymce/noteview.html index 02a1723f51..b74a584367 100644 --- a/resource/tinymce/noteview.html +++ b/resource/tinymce/noteview.html @@ -11,15 +11,19 @@ content_css: "css/note-content.css", entity_encoding: "raw", fix_list_elements: true, - readonly: true, + toolbar: false, menubar: false, resize: false, statusbar: false, init_instance_callback: function (ed) { + ed.setMode('readonly'); + setLocale(ed); zoteroInit(ed); + + ed.getContainer().classList.add('readonly'); }, // More restrictive version of default set, with JS/etc. removed