nandorsoma commented on PR #7878:
URL: https://github.com/apache/nifi/pull/7878#issuecomment-1785921412

   > @nandorsoma On further consideration, 300 KB seems too large for an 
initial loading image, given that the point of the loading screen is to render 
quickly before the rest of the content is ready.
   > 
   > 
   > 
   > I looked at optimizing the animated GIF, which consists of 90 frames, and 
I was able to shrink it down to about 40 KB with 11 frames. It is not quite as 
smooth, but it is one possible approach.
   > 
   > 
   > 
   > Thoughts?
   > 
   > 
   > 
   > 
![loading-optimized](https://github.com/apache/nifi/assets/20425862/f8615c4c-a536-4bcb-911a-495b5f73f615)
   > 
   > 
   
   Thanks for the feedback, @exceptionfactory. To be honest, I had some 
concerns about the initial 300kB as well. When I tested it on my machine with a 
blank canvas, the splash screen only appeared for a fraction of a second. It 
doesn't seem like the loading time is significantly affected by the animation's 
size in that scenario, although I do recognize this is an edge case. In a more 
typical scenario with a complex canvas and a longer loading time for NiFi, that 
300kB animation download shouldn't be the bottleneck. In that situation, having 
a smoother animation also appears to be more beneficial.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to