From 9858514bb4605070096637e87a8109bceb0d6d85 Mon Sep 17 00:00:00 2001 From: David Palmer Date: Wed, 21 Jun 2023 16:15:02 +1200 Subject: [PATCH 1/5] wrap video embeds in the ratio container This should correctly size the embedded video iframe to the full available width, which looks better and is compatible with mobile devices. Also add the "allowfullscreen" modifier to the iframe so that the video can be expanded to the browser's fullscreen mode. Also add the post.embed_title as the iframe "title" attribute. --- src/shared/components/post/metadata-card.tsx | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/shared/components/post/metadata-card.tsx b/src/shared/components/post/metadata-card.tsx index e6a864a..3c6f6bb 100644 --- a/src/shared/components/post/metadata-card.tsx +++ b/src/shared/components/post/metadata-card.tsx @@ -75,10 +75,14 @@ export class MetadataCard extends Component< )} {this.state.expanded && post.embed_video_url && ( - +
+ +
)} ); From c16c00db0dffbe9503c3c61cb8067614f8424549 Mon Sep 17 00:00:00 2001 From: David Palmer Date: Wed, 21 Jun 2023 16:28:27 +1200 Subject: [PATCH 2/5] add a prefix to hint to screenreaders what this iframe is --- src/shared/components/post/metadata-card.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/shared/components/post/metadata-card.tsx b/src/shared/components/post/metadata-card.tsx index 3c6f6bb..80ce173 100644 --- a/src/shared/components/post/metadata-card.tsx +++ b/src/shared/components/post/metadata-card.tsx @@ -80,7 +80,7 @@ export class MetadataCard extends Component< allowFullScreen className="post-metadata-iframe" src={post.embed_video_url} - title={post.embed_title} + title={"Embedded Video: " + post.embed_title} > )} From f7df8314990b3f0152993eac42d125a65b54552b Mon Sep 17 00:00:00 2001 From: Dessalines Date: Wed, 21 Jun 2023 19:48:43 -0400 Subject: [PATCH 3/5] v0.18.0-rc.5 --- package.json | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/package.json b/package.json index 9a28546..1055d85 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "lemmy-ui", - "version": "0.18.0-rc.4", + "version": "0.18.0-rc.5", "description": "An isomorphic UI for lemmy", "repository": "https://github.com/LemmyNet/lemmy-ui", "license": "AGPL-3.0", @@ -22,9 +22,16 @@ "translations:update": "git submodule update --remote --recursive" }, "lint-staged": { - "*.{ts,tsx,js}": ["prettier --write", "eslint --fix"], - "*.{css, scss}": ["prettier --write"], - "package.json": ["sortpack"] + "*.{ts,tsx,js}": [ + "prettier --write", + "eslint --fix" + ], + "*.{css, scss}": [ + "prettier --write" + ], + "package.json": [ + "sortpack" + ] }, "dependencies": { "@babel/plugin-proposal-decorators": "^7.21.0", From 924e6706993c5d291aa9f269ad933d83da4a4147 Mon Sep 17 00:00:00 2001 From: David Palmer Date: Thu, 22 Jun 2023 12:23:36 +1200 Subject: [PATCH 4/5] remove embed prefix until a translation can be added --- src/shared/components/post/metadata-card.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/shared/components/post/metadata-card.tsx b/src/shared/components/post/metadata-card.tsx index 0b28b28..bc6576a 100644 --- a/src/shared/components/post/metadata-card.tsx +++ b/src/shared/components/post/metadata-card.tsx @@ -80,7 +80,7 @@ export class MetadataCard extends Component< allowFullScreen className="post-metadata-iframe" src={post.embed_video_url} - title={"Embedded Video: " + post.embed_title} + title={post.embed_title} > )} From 3664b0600b3a196209248ff7c5898a78ee4c0795 Mon Sep 17 00:00:00 2001 From: SleeplessOne1917 Date: Thu, 22 Jun 2023 01:04:28 +0000 Subject: [PATCH 5/5] (Hopefully) fix webmanifest different origin issue (#1457) Co-authored-by: Dessalines --- src/server/index.tsx | 2 +- src/server/utils/create-ssr-html.tsx | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/server/index.tsx b/src/server/index.tsx index 144b596..b65506b 100644 --- a/src/server/index.tsx +++ b/src/server/index.tsx @@ -25,7 +25,7 @@ if (!process.env["LEMMY_UI_DISABLE_CSP"] && !process.env["LEMMY_UI_DEBUG"]) { server.get("/robots.txt", RobotsHandler); server.get("/service-worker.js", ServiceWorkerHandler); -server.get("/manifest", ManifestHandler); +server.get("/manifest.webmanifest", ManifestHandler); server.get("/css/themes/:name", ThemeHandler); server.get("/css/themelist", ThemesListHandler); server.get("/*", CatchAllHandler); diff --git a/src/server/utils/create-ssr-html.tsx b/src/server/utils/create-ssr-html.tsx index 569d83a..ae766b3 100644 --- a/src/server/utils/create-ssr-html.tsx +++ b/src/server/utils/create-ssr-html.tsx @@ -77,7 +77,7 @@ export async function createSsrHtml( /> - +