Skip to content

Replace ThemeImage with ThemedImage and update image sources to use useBaseUrl#4512

Closed
clicktodev wants to merge 3 commits into
react:mainfrom
clicktodev:use-builtin-themed-image-component
Closed

Replace ThemeImage with ThemedImage and update image sources to use useBaseUrl#4512
clicktodev wants to merge 3 commits into
react:mainfrom
clicktodev:use-builtin-themed-image-component

Conversation

@clicktodev

@clicktodev clicktodev commented Feb 16, 2025

Copy link
Copy Markdown
Contributor

Transition from ThemeImage to ThemedImage for improved image handling and update image sources to utilize useBaseUrl for better path management.

edit: looks like the components need to be moved out of the pages folder so they don't get SSGed without the layout wrapper to avoid context error

@netlify

netlify Bot commented Feb 16, 2025

Copy link
Copy Markdown

Deploy Preview for react-native failed.

Name Link
🔨 Latest commit 68196c9
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/67b2006c96f93600083c66fe

@netlify

netlify Bot commented Feb 16, 2025

Copy link
Copy Markdown

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 025676e
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/67b20da589ad390008e6d964
😎 Deploy Preview https://deploy-preview-4512--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@clicktodev clicktodev marked this pull request as ready for review February 16, 2025 20:07

@jaamaalxyz jaamaalxyz left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job @hichemfantar

@clicktodev clicktodev marked this pull request as draft February 19, 2025 09:51
@Simek

Simek commented Aug 15, 2025

Copy link
Copy Markdown
Collaborator

Nice catch @clicktodev, thanks for working on those changes! 🙏

Sorry to see that this did not get over the finish line. Due to inactivity, and draft status of that contribution we decided to fix and ship this separately, since the issue has been brought up again in recent weeks.

Closing in favour of:

@Simek Simek closed this Aug 15, 2025
@clicktodev clicktodev deleted the use-builtin-themed-image-component branch August 15, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants