SnpDeveloper commented on issue #711:
URL: 
https://github.com/apache/cordova-plugin-camera/issues/711#issuecomment-846812829


   Thanks @PieterVanPoyer I have tested heic conversion, all seems to work 
fine. 
   
   But here are some of my observations, feel free to comment my thoughts
   
   1. bitmap conversion does a very excellent job. 
        It converts GIF, BMP, JPG, PNG, HEIC to desired format (JPG in mycase)
        It doesnt convert TIF, HEIF formats.
        
   2. While failing to convert on bitmap I am getting a clear exception (App is 
not crashing, again as i would except and a good thing) so easy for me to show 
File format not support error message in my app.
   
   New changes you have made make sense preseving the encoding mimeType (JPG) 
extension.. a nice touch.
   
   What i dont understand is why we are restricting support only to JPG, PNG 
and HEIC, as mentioned before GIF, BMP, JPG, PNG, HEIC formats are all working.
   
   Correct me if I have missed anything.
   


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to