dotnetthailand.github.io icon indicating copy to clipboard operation
dotnetthailand.github.io copied to clipboard

Remove warning `warn the option sizeByPixelDensity is deprecated and should not be used. It will be removed in the next major release of Gatsby.`

Open kaiwanyawit-chawankul opened this issue 3 years ago • 4 comments

warn the option sizeByPixelDensity is deprecated and should not be used. It will be removed in the next major release of Gatsby.

image

Should we change this deprecated function?

CC: @aaronamm @mildronize

kaiwanyawit-chawankul avatar Feb 08 '22 03:02 kaiwanyawit-chawankul

@kaiwanyawit-chawankul Thanks for your suggestion, my opinion it doesn't hurry to fix this issue, because in order to migrate Gatsby with major release is not easy, due to a huge dependency of this project. this project still use Gatsby 2.x,

Answer your question, if we can migrate from Gatsby 2.x to 3.x, this deprecated function will be more easy.

mildronize avatar Feb 11 '22 07:02 mildronize

@kaiwanyawit-chawankul Thanks for your suggestion and agree with @mildronize for his idea. I think we should consider to migrate our website to Gatsby version 4 as our first priority.

However, I don't know if you would like to help us on migration. Please let me know if you would like to try or not. I am happy to support you but I accept this might be a huge PR, LoL.

aaronamm avatar Feb 11 '22 10:02 aaronamm

Hi @mildronize @aaronamm,

This error came from gatsby-plugin-sharp for warning about sizeByPixelDensity which is have nothing to do with the use in gatsby-remark-images

image

That mean we don't have to worry about it. Based on those links below. Maybe I am wrong.

https://diff.intrinsic.com/gatsby-plugin-sharp/2.0.36/2.0.37 https://github.com/gatsbyjs/gatsby/issues/23376 https://github.com/gatsbyjs/gatsby/issues/12743

What do you think?

kaiwanyawit-chawankul avatar Feb 24 '22 16:02 kaiwanyawit-chawankul

I think @kaiwanyawit-chawankul you can be true, could you please remove that line?

mildronize avatar Apr 15 '22 13:04 mildronize