SnpDeveloper edited a comment 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)
        
   2. While failing to convert (TIF, HEIF formats) 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.
   
   Some of my customer's upload BMP and GIF as well... 
   
   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