From 09325c91de64d3bca545b456981d79a9eed6ee98 Mon Sep 17 00:00:00 2001
From: alemi <me@alemi.dev>
Date: Wed, 3 Jul 2024 05:58:29 +0200
Subject: [PATCH] feat(web): improved Document object view

doesn't work retroactively but new lemmy posts should look more like
lemmy posts
---
 web/index.html               | 11 +++++++++-
 web/src/components/object.rs | 41 ++++++++++++++++++++----------------
 2 files changed, 33 insertions(+), 19 deletions(-)

diff --git a/web/index.html b/web/index.html
index 35e5a968..596e4386 100644
--- a/web/index.html
+++ b/web/index.html
@@ -203,6 +203,16 @@
 			min-width: 2em;
 			max-width: 2em;
 		}
+		div.flex-pic-container {
+			flex: 1;
+			border: solid 3px #bf616a;
+			margin-right: .5em;
+		}
+		div.flex-pic {
+			background-size: cover;
+			margin: 5px;
+			height: calc(100% - 10px); /* TODO can we avoid this calc() without having this overflow??? */
+		}
 		.box {
 			border: 3px solid var(--accent);
 		}
@@ -215,7 +225,6 @@
 		img.attachment {
 			cursor: pointer;
 			height: 10em;
-			width: 100%;
 			border: 3px solid var(--accent);
 			padding: 5px;
 			object-fit: cover;
diff --git a/web/src/components/object.rs b/web/src/components/object.rs
index d65fa971..3df22c4e 100644
--- a/web/src/components/object.rs
+++ b/web/src/components/object.rs
@@ -38,7 +38,7 @@ pub fn Attachment(
 			view! {
 				<p class="center">
 					<img
-						class="attachment"
+						class="w-100 attachment"
 						class:expand=expand
 						src={move || if sensitive && !expand.get() {
 							URL_SENSITIVE.to_string()
@@ -78,18 +78,11 @@ pub fn Attachment(
 
 		"link" =>
 			view! {
-				<ul> // TODO kind of wasteful to make 1-item list, can we just add the same style to a div?
-					<li>
-						<a href={href.clone()} title={href.clone()} rel="noreferrer nofollow" target="_blank">
-							{Uri::pretty(&href)}
-						</a>
-					</li>
-				</ul>
-				{object.name().map(|name| {
-					view! {
-						<p class="center mt-0"><small>{name.to_string()}</small></p>
-					}
-				})}
+				<p class="center mt-s mb-s">
+					<a href={href.clone()} title={href.clone()} rel="noreferrer nofollow" target="_blank">
+						<input type="submit" class="w-100" value={Uri::pretty(&href)} title={object.name().unwrap_or_default().to_string()} />
+					</a>
+				</p>
 			}.into_view(),
 
 		_ => 
@@ -149,6 +142,15 @@ pub fn Object(object: crate::Object) -> impl IntoView {
 			</a>
 		});
 
+	let post_image = object.image().get().and_then(|x| x.url().id().str()).map(|x| view! {
+		<div class="flex-pic-container">
+			<a href={x.clone()} target="_blank">
+				<div class="flex-pic" style={format!("background-image: url('{x}')")}>
+				</div>
+			</a>
+		</div>
+	});
+
 	let post_inner = view! {
 		<Summary summary=object.summary().ok().map(|x| x.to_string()) >
 			<p inner_html={content}></p>
@@ -163,11 +165,14 @@ pub fn Object(object: crate::Object) -> impl IntoView {
 		}.into_view(),
 		// lemmy with Page, peertube with Video
 		Ok(apb::ObjectType::Document(t)) => view! {
-			<article class="ml-1 mr-1">
-				<h4 class="mt-s mb-1" title={t.as_ref().to_string()}>
-					<b>{object.name().unwrap_or_default().to_string()}</b>
-				</h4>
-				{post_inner}
+			<article class="ml-1 mr-1" style="display: flex">
+				{post_image}
+				<div style="flex: 3">
+					<h4 class="mt-s mb-1" title={t.as_ref().to_string()}>
+						<b>{object.name().unwrap_or_default().to_string()}</b>
+					</h4>
+					{post_inner}
+				</div>
 			</article>
 		}.into_view(),
 		// wordpress, ... ?