mariusmuntean / ChartJs.Blazor

Brings Chart.js charts to Blazor
https://www.iheartblazor.com/
MIT License
676 stars 151 forks source link

Please migrate Newtonsoft.Json to System.Text.Json, to reduce download szie #201

Open keerthirajap opened 2 years ago

keerthirajap commented 2 years ago

Hi Team,

Please remove dependence of Newtonsoft.Json, and migrate json processing to System.Text.Json. It will save few kb of download size and improve performance.

Microsoft recommends it.

https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-migrate-from-newtonsoft-how-to?pivots=dotnet-6-0

SeppPenner commented 2 years ago

The problem is that System.Text.Json still is not equivalent to Newtonsoft.Json (and if you ask me has still childhood problems even with simple types...). Just my opinion, I'm not using System.Text.Json anywhere because of these issues.