search

sergiitk / pagerbeauty

:pager::sparkles: PagerDuty on-call widget for monitoring dashboard. Datadog and Grafana compatible
https://demo.pagerbeauty.sergii.org/v1/schedules/PDUQGYJ.html
MIT License
338 stars 27 forks source link

Avatar's size is too large and slow to load #112

Open geekeren opened 5 years ago

geekeren commented 5 years ago

The avatar loaded very very slowly, and I found the size with 2048x2048 may be very unnecessary.

image

https://github.com/sergiitk/pagerbeauty/blob/3416af7a6055016e209a905fa5f2882012f7c617/src/assets/javascripts/views/OnCallViews.jsx#L261

It shows the avatar takes up 3.5 MB!

image

I suggested to set the size to 128 x 128

sergiitk commented 5 years ago

Thank you for the feedback @geekeren. I used the larger image possible because PagerBeauty is supposed to be scalable. You're right, 2048x2048 feels like an overkill. However, I think 128x128 is too small too. The best way to go would be making it configurable. I'll do that, and set a more sensible default resolution.