fix: batch load fills multiple identical objects

in case of an announce and a view, we get the attachments only once
otherwise and frontend may overwrite cache with the empty one. this is
wasteful because we clone everytime tho, TODO!

upub/core/src/selector/batch.rs

@@ -36,8 +36,10 @@ impl BatchFillable for Vec<RichActivity> {
 		}
 		for element in self.iter_mut() {
 			if let Some(ref object) = element.object {
-				if let Some(v) = map.remove(&object.internal) {
+				if let Some(v) = map.get(&object.internal) {
-					element.accept(v, tx).await?;
+					// TODO wasteful because we clone every time, but we cant do remove otherwise multiple
+					//      identical objects wont get filled (for example, a post boosted twice)
+					element.accept(v.clone(), tx).await?;
 				}
 			}
 		}

web/src/objects/item.rs

@@ -162,7 +162,7 @@ pub fn Object(
 			</tr>
 		</table>
 		{post}
-		<div class="mt-s ml-1 rev">
+		<div class="mb-s ml-1 rev">
 			{tag_badges}
 			{audience_badge}
 			<span style="white-space:nowrap">