Skip to content

Commit 729978a

Browse files
authored
refactor: drop image base url from cli and templates (#4563)
In all cli templates asset base url and image base url are the same so we can safely switch. Removed image loader from prebuild.ts. Dropped imageBaseUrl from sdk react context.
1 parent 8733d13 commit 729978a

File tree

51 files changed

+102
-181
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

51 files changed

+102
-181
lines changed

apps/builder/app/canvas/canvas.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,6 @@ const useElementsTree = (
119119
<ReactSdkContext.Provider
120120
value={{
121121
renderer: isPreviewMode ? "preview" : "canvas",
122-
imageBaseUrl: "/cgi/image/",
123122
assetBaseUrl: params.assetBaseUrl,
124123
imageLoader,
125124
resources: {},

apps/builder/app/routes/_canvas.canvas.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ export const loader = async ({ request }: LoaderFunctionArgs) => {
1616
}
1717

1818
const params: Params = {
19-
imageBaseUrl: "/cgi/image/",
2019
assetBaseUrl: env.ASSET_BASE_URL,
2120
};
2221

apps/builder/app/shared/nano-states/props.test.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -243,7 +243,6 @@ test("resolve asset prop values when params is provided", () => {
243243

244244
$params.set({
245245
assetBaseUrl: "/asset/",
246-
imageBaseUrl: "/image/",
247246
});
248247
expect(
249248
$propValuesByInstanceSelector.get().get(JSON.stringify(["box"]))
@@ -279,7 +278,6 @@ test("resolve page prop values when params is provided", () => {
279278

280279
$params.set({
281280
assetBaseUrl: "/asset/",
282-
imageBaseUrl: "/image/",
283281
});
284282
expect(
285283
$propValuesByInstanceSelector.get().get(JSON.stringify(["box"]))

fixtures/ssg-netlify-by-project-id/app/constants.mjs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,17 +16,17 @@ export const imageLoader = (props) => {
1616
}
1717

1818
if (process.env.NODE_ENV !== "production") {
19-
return imageBaseUrl + props.src;
19+
return assetBaseUrl + props.src;
2020
}
2121

2222
if (props.format === "raw") {
23-
return imageBaseUrl + props.src;
23+
return assetBaseUrl + props.src;
2424
}
2525

2626
// https://docs.netlify.com/image-cdn/overview/
2727
return (
2828
"/.netlify/images?url=" +
29-
encodeURIComponent(imageBaseUrl + props.src) +
29+
encodeURIComponent(assetBaseUrl + props.src) +
3030
"&w=" +
3131
props.width +
3232
"&q=" +

fixtures/ssg-netlify-by-project-id/pages/index/+Page.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,6 @@
11
import type { PageContext } from "vike/types";
22
import { ReactSdkContext } from "@webstudio-is/react-sdk/runtime";
3-
import {
4-
assetBaseUrl,
5-
imageBaseUrl,
6-
imageLoader,
7-
} from "../../app/constants.mjs";
3+
import { assetBaseUrl, imageLoader } from "../../app/constants.mjs";
84
import { Page } from "../../app/__generated__/_index";
95

106
const PageComponent = ({ data }: { data: PageContext["data"] }) => {
@@ -14,7 +10,6 @@ const PageComponent = ({ data }: { data: PageContext["data"] }) => {
1410
value={{
1511
imageLoader,
1612
assetBaseUrl,
17-
imageBaseUrl,
1813
resources,
1914
}}
2015
>

fixtures/ssg/app/constants.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,5 +15,5 @@ export const imageLoader = ({ src }) => {
1515
return src;
1616
}
1717

18-
return imageBaseUrl + src;
18+
return assetBaseUrl + src;
1919
};

fixtures/ssg/pages/another-page/+Page.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,6 @@
11
import type { PageContext } from "vike/types";
22
import { ReactSdkContext } from "@webstudio-is/react-sdk/runtime";
3-
import {
4-
assetBaseUrl,
5-
imageBaseUrl,
6-
imageLoader,
7-
} from "../../app/constants.mjs";
3+
import { assetBaseUrl, imageLoader } from "../../app/constants.mjs";
84
import { Page } from "../../app/__generated__/[another-page]._index";
95

106
const PageComponent = ({ data }: { data: PageContext["data"] }) => {
@@ -14,7 +10,6 @@ const PageComponent = ({ data }: { data: PageContext["data"] }) => {
1410
value={{
1511
imageLoader,
1612
assetBaseUrl,
17-
imageBaseUrl,
1813
resources,
1914
}}
2015
>

fixtures/ssg/pages/index/+Page.tsx

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,6 @@
11
import type { PageContext } from "vike/types";
22
import { ReactSdkContext } from "@webstudio-is/react-sdk/runtime";
3-
import {
4-
assetBaseUrl,
5-
imageBaseUrl,
6-
imageLoader,
7-
} from "../../app/constants.mjs";
3+
import { assetBaseUrl, imageLoader } from "../../app/constants.mjs";
84
import { Page } from "../../app/__generated__/_index";
95

106
const PageComponent = ({ data }: { data: PageContext["data"] }) => {
@@ -14,7 +10,6 @@ const PageComponent = ({ data }: { data: PageContext["data"] }) => {
1410
value={{
1511
imageLoader,
1612
assetBaseUrl,
17-
imageBaseUrl,
1813
resources,
1914
}}
2015
>

fixtures/webstudio-cloudflare-template/app/constants.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,5 @@ export const imageBaseUrl = "/assets/";
99
* @type {import("@webstudio-is/image").ImageLoader}
1010
*/
1111
export const imageLoader = ({ src }) => {
12-
return imageBaseUrl + src;
12+
return assetBaseUrl + src;
1313
};

fixtures/webstudio-cloudflare-template/app/routes/[another-page]._index.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ import {
3131
projectId,
3232
contactEmail,
3333
} from "../__generated__/[another-page]._index.server";
34-
import { assetBaseUrl, imageBaseUrl, imageLoader } from "../constants.mjs";
34+
import { assetBaseUrl, imageLoader } from "../constants.mjs";
3535
import css from "../__generated__/index.css?url";
3636
import { sitemap } from "../__generated__/$resources.sitemap.xml";
3737

@@ -329,7 +329,6 @@ const Outlet = () => {
329329
value={{
330330
imageLoader,
331331
assetBaseUrl,
332-
imageBaseUrl,
333332
resources,
334333
}}
335334
>

0 commit comments

Comments
 (0)