style: close button on left side of preview modal

ubiquity · Dec 4, 2023 · 3fc439d · 3fc439d
1 parent 60bb4be
commit 3fc439d
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 14 deletions.
diff --git a/src/home/home.ts b/src/home/home.ts
@@ -14,7 +14,6 @@ if (cachedIssues) {
   const fullIssues = fetchCachedIssuesFull();
 
   if (!fullIssues) {
-
     fetchIssuesFull(cachedIssues)
       .then((downloaded) => {
         localStorage.setItem("githubIssuesFull", JSON.stringify(downloaded));

diff --git a/src/home/render-github-issues.ts b/src/home/render-github-issues.ts
@@ -19,8 +19,8 @@ const previewBodyInner = document.createElement("div");
 previewBodyInner.classList.add("preview-body-inner");
 
 // Assemble the preview box
-previewHeader.appendChild(title);
 previewHeader.appendChild(closeButton);
+previewHeader.appendChild(title);
 previewBody.appendChild(previewBodyInner);
 previewContent.appendChild(previewHeader);
 previewContent.appendChild(previewBody);
@@ -186,9 +186,5 @@ function previewIssue(issuePreview: GitHubIssueWithNewFlag) {
 
 // Function to find an issue by URL
 function findIssueByUrl(issues: GitHubIssueWithNewFlag[], url: string) {
-  console.trace({
-    issues,
-    url,
-  });
   return issues.find((issue) => issue.html_url === url);
 }
diff --git a/static/style/inverted-style.css b/static/style/inverted-style.css
@@ -460,11 +460,11 @@
     margin: 16px;
     /* text-align: center; */
   }
-  .preview a{
+  .preview a {
     word-break: break-all;
   }
   .preview-body-inner {
-    width: calc( 100% - 48px);
+    width: calc(100% - 48px);
     max-width: 640px;
     overflow: scroll;
   }
@@ -496,7 +496,7 @@
     max-height: calc(100vh - 130px);
     pointer-events: none;
   }
-  .close-preview svg{
+  .close-preview svg {
     fill: #00000080;
   }
   .preview.active {

diff --git a/static/style/style.css b/static/style/style.css
@@ -457,14 +457,14 @@
     padding-left: 12px;
   }
   .preview h1 {
-    margin: 16px;
+    margin: 8px;
     /* text-align: center; */
   }
-  .preview a{
+  .preview a {
     word-break: break-all;
   }
   .preview-body-inner {
-    width: calc( 100% - 48px);
+    width: calc(100% - 48px);
     max-width: 640px;
     overflow: scroll;
   }
@@ -496,7 +496,7 @@
     max-height: calc(100vh - 130px);
     pointer-events: none;
   }
-  .close-preview svg{
+  .close-preview svg {
     fill: #ffffff80;
   }
   .preview.active {
@@ -507,7 +507,7 @@
   .preview-header {
     display: flex;
     align-items: center;
-    justify-content: space-between;
+    /* justify-content: space-between; */
     margin-bottom: 8px;
   }