-
Notifications
You must be signed in to change notification settings - Fork 148
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
Using Image
from next/Image
all the time for all images.
#686
Labels
Comments
JayaKrishnaNamburu
added
enhancement
New feature or request
help wanted
Extra attention is needed
labels
May 31, 2022
NextJS released a new update for the |
JayaKrishnaNamburu
changed the title
Using
Using Jul 19, 2022
Image
from `next/Image
all the time for all images.Image
from next/Image
all the time for all images.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now, we are using
next/image
only if the user specifieswidth
andheight
alone without any additional attributes. This is because,next/Image
is not completely pure component at the moment. Because it is bringing some side effects tot he layout with it. Since we can't pass thestyles
that users set into the component.https://github.com/teleporthq/teleport-code-generators/blob/development/packages/teleport-plugin-jsx-next-image/src/index.ts#L11
Here is the issue on the
NextJS
GitHub on the same topicvercel/next.js#18312
Case 1
It is not working out of the box all the time, since we allow users to free-style on the properties they use for the images. But the
next/image
need to be used with some constraintsUsual - https://codesandbox.io/s/pf9mt4
next/Image - https://codesandbox.io/s/travel-agency-forked-h28f9p
The text was updated successfully, but these errors were encountered: