Kieran/snort
5
14
Fork 14
Code Issues 37 Pull Requests 1 Packages Projects Releases 24 Wiki Activity

Profile urls, scrollbar, ProfilePage refactoring #646

Merged
mmalmi merged 10 commits from mmalmi/snort:main into main 2023-10-09 18:41:32 +00:00
Conversation 2 Commits 10 Files Changed 9 +9
1 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 224960a11f - Show all commits

9
packages/app/src/Element/Text.tsx
View File

@ -170,6 +170,9 @@ export default function Text({
}
if (element.type === "media" && element.mimeType?.startsWith("image")) {
if (i > 0) {
chunks.push(<div className="mt10" />);
mmalmi marked this conversation as resolved Outdated
fernandoporazzi commented 2023-10-09 09:03:33 +00:00
Outdated
Review
Copy Link

This is just for margin top? Can't we control that without adding an empty div?

This is just for margin top? Can't we control that without adding an empty div?
Kieran commented 2023-10-09 09:05:30 +00:00
Outdated
Review
Copy Link

Yea probably best to use css actually

Yea probably best to use css actually
}
if (disableMedia ?? false) {
chunks.push(<DisableMedia content={element.content} />);
} else {
@ -231,6 +234,9 @@ export default function Text({
element.type === "media" &&
(element.mimeType?.startsWith("audio") || element.mimeType?.startsWith("video"))
) {
if (i > 0) {
chunks.push(<div className="mt10" />);
}
if (disableMedia ?? false) {
chunks.push(<DisableMedia content={element.content} />);
} else {
@ -247,6 +253,9 @@ export default function Text({
chunks.push(<CashuNuts token={element.content} />);
}
if (element.type === "link" || (element.type === "media" && element.mimeType?.startsWith("unknown"))) {
if (i > 0 && !disableLinkPreview) {
chunks.push(<div className="mt10" />);
}
chunks.push(
<HyperText link={element.content} depth={depth} showLinkPreview={!(disableLinkPreview ?? false)} />,
);