Merge pull request #397 from MuckRock/368-document-components

Document components

.gitignore

@@ -39,3 +39,4 @@ tests/fixtures/development.json
 
 scratch/
 dist/
+storybook-static/

src/common/dialog/DocumentPickerDialog.svelte

@@ -13,15 +13,13 @@
   }
 </script>
 
-<div>
-  <div class="mcontent">
-    <div class="documents">
-      <Documents dialog={true} embed={true} on:pick={handlePick} />
-    </div>
+<div class="mcontent">
+  <div class="documents">
+    <Documents dialog={true} embed={true} on:pick={handlePick} />
   </div>
 </div>
 
-<style lang="scss">
+<style>
   .documents {
     padding: 0 10px;
   }

src/common/stories/Logo.stories.svelte

@@ -5,6 +5,8 @@
   export const meta = {
     title: "Common / Logo",
     component: Logo,
+    tags: ["autodocs"],
+    parameters: { layout: "centered" },
   };
 
   const args = {

src/pages/app/Document.svelte

@@ -168,17 +168,23 @@
               </Link>
             {/each}
             {#if document.dataPoints.length > 0 && document.editAccess}
-              <span class="pencil" on:click={() => editData([document])}>
+              <button
+                class="pencil buttonLike"
+                on:click={() => editData([document])}
+              >
                 {@html pencilSvg}
-              </span>
+              </button>
             {/if}
           {/if}
         </div>
         {#if document.highlights != null && document.highlights.length > 0 && !closeHighlights}
           <div class="hinfo">
-            <span class="x" on:click={() => (closeHighlights = true)}>
+            <button
+              class="x buttonLike"
+              on:click={() => (closeHighlights = true)}
+            >
               {@html closeSimpleSvg}
-            </span>
+            </button>
 
             {#if moreToExpand}
               <span>
@@ -189,9 +195,12 @@
                   },
                 })}
               </span>
-              <span class="padleft" on:click={() => (expandHighlights = true)}>
+              <button
+                class="padleft buttonLike"
+                on:click={() => (expandHighlights = true)}
+              >
                 {$_("document.showAll")}
-              </span>
+              </button>
             {:else}
               <span>
                 {$_("document.matchingPages", {
@@ -239,9 +248,12 @@
 
         {#if document.noteHighlights != null && document.noteHighlights.length > 0 && !closeNoteHighlights}
           <div class="hinfo">
-            <span class="x" on:click={() => (closeNoteHighlights = true)}>
+            <button
+              class="x buttonLike"
+              on:click={() => (closeNoteHighlights = true)}
+            >
               {@html closeSimpleSvg}
-            </span>
+            </button>
 
             <span>Notes matching the query</span>
           </div>

src/pages/app/fixtures/addon_runs.json

@@ -0,0 +1 @@
+{ "next": null, "previous": null, "results": [] }

src/pages/app/fixtures/document.json

@@ -0,0 +1,46 @@
+{
+  "id": 1,
+  "access": "public",
+  "admin_noindex": false,
+  "asset_url": "https://s3.documentcloud.org/",
+  "canonical_url": "https://www.documentcloud.org/documents/1-a-i-g-bailout-the-inspector-generals-report",
+  "created_at": "2010-02-22T19:48:08.738905Z",
+  "data": {},
+  "description": "Neil Barofsky's report concludes that officials overseeing the rescue of the American International Group might have overpaid other banks to wrap up A.I.G.'s financial obligations.",
+  "edit_access": false,
+  "file_hash": "",
+  "noindex": false,
+  "language": "eng",
+  "organization": {
+    "id": 1,
+    "avatar_url": "https://cdn.muckrock.com/static/images/avatars/organization.png",
+    "individual": false,
+    "name": "DocumentCloud",
+    "slug": "dcloud",
+    "uuid": "99875da4-7b70-4150-b854-7ba5a3951f99"
+  },
+  "original_extension": "pdf",
+  "page_count": 47,
+  "page_spec": "612.0x792.0:0-46",
+  "projects": [46386, 6],
+  "publish_at": null,
+  "published_url": "",
+  "related_article": "",
+  "revision_control": false,
+  "slug": "a-i-g-bailout-the-inspector-generals-report",
+  "source": "Office of the Special Inspector General for T.A.R.P.",
+  "status": "success",
+  "title": "A.I.G. Bailout: The Inspector General's Report",
+  "updated_at": "2020-11-10T16:23:31.154198Z",
+  "user": {
+    "id": 1,
+    "avatar_url": "https://cdn.muckrock.com/static/images/avatars/profile.png",
+    "name": "Jeremy Ashkenas",
+    "organization": 24523,
+    "organizations": [24523],
+    "admin_organizations": [24523],
+    "username": "JeremyAshkenas_VfDHJpUE",
+    "uuid": "48175058-8eb8-44b8-943c-a705036ea840",
+    "verified_journalist": false
+  }
+}