Merge pull request #131 from dipamsen/fix-trash

Fixes to trash page

Author: harshkhandeparkar

frontend/src/pages/TrashPage.tsx

@@ -157,14 +157,14 @@ function TrashPage() {
 														{paper.course_code ? `(${paper.course_code})` : ""}
 													</div>
 													<div className="pills">
-														<div className="pill">{paper.year}</div>
-														<div className="pill">{paper.exam}</div>
-														<div className="pill">{paper.semester}</div>
+														{paper.year && <div className="pill">{paper.year}</div>}
+														{paper.exam && <div className="pill">{paper.exam}</div>}
+														{paper.semester && <div className="pill">{paper.semester}</div>}
 														{paper.note !== "" && <div className="pill">{paper.note}</div>}
 													</div>
-													<button className="btn">
+													<a className="btn" href={paper.filelink} target="_blank">
 														<FaFilePdf size="1.2em" />
-													</button>
+													</a>
 												</div>
 											</div>
 										))}

frontend/src/pages/styles/trash_page.scss

@@ -44,6 +44,7 @@
 		display: flex;
 		gap: 2rem;
 		justify-content: space-around;
+		margin-bottom: 20px;
 
 		.papers-panel {
 			width: 80%;
@@ -84,7 +85,7 @@
     .trash-paper {
       display: flex;
       align-items: center;
-      gap: 0.3em;
+      gap: 0.5em;
     }
   }
 
@@ -113,7 +114,7 @@
 
 		&.icon-btn {
 			display: flex;
-			gap: 0.3em;
+			gap: 0.5em;
 			align-items: center;
 		}