-
Notifications
You must be signed in to change notification settings - Fork 298
chore(developer-hub): dual image mode enable and use in OIS #3055
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
5 Skipped Deployments
|
darkSrc="/images/ois/OIS_Rewards_Example_Dark_1.png" | ||
lightSrc="/images/ois/OIS_Rewards_Example_Light_1.png" | ||
alt="Example 1 - Only Publisher Stake" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As per the fuma docs, referencing Images with URL
.lightImage { | ||
display: block; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
created this module, and added them in a folder to follow the same pattern.
 | ||
<br /> | ||
<DualModeImage | ||
darkSrc="/images/ois/OIS_Rewards_Example_Dark_1.png" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be better to import the images and pass the import here. Next can do some automatic image optimization when you import images which it cannot do when you use image urls. Also you can colocate the images with where they're used which is nice
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see, I was using image URL as per the fuma docs https://fumadocs.dev/docs/headless/custom-source#images
If there would be optimization by using the import, lets do it.
Summary
Rationale
How has this been tested?