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 +4 -9
2 changed files with 4 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 15fb4cabdf - Show all commits

4
packages/app/src/Element/Text.css
View File

@ -113,3 +113,7 @@
height: 100%;
display: block;
}
.gallery:not(:first-child), img:not(:first-child), video:not(:first-child), .link-preview-container:not(:first-child) {
margin-top: 10px;
}

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

@ -170,9 +170,6 @@ export default function Text({
}
if (element.type === "media" && element.mimeType?.startsWith("image")) {
if (i > 0) {
chunks.push(<div className="mt10" />);
}
if (disableMedia ?? false) {
chunks.push(<DisableMedia content={element.content} />);
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
} else {
@ -234,9 +231,6 @@ 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 {
@ -253,9 +247,6 @@ 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)} />,
);