SimpleMobileTools / Simple-Camera

Quick photo and video camera with a flash, customizable aspect ratio.
https://www.simplemobiletools.com
GNU General Public License v3.0
750 stars 288 forks source link

add camera timer #379

Closed KryptKode closed 1 year ago

KryptKode commented 1 year ago

Notes

https://user-images.githubusercontent.com/25648077/204021657-6c243440-4baa-4214-beba-5b5388ed6b94.mp4

tibbi commented 1 year ago

remove the back flash and front blinking, it isnt needed. Keep the numbers countdown. The "x" on the shutter should be at the proper position since the beginning, remove the rotating animation. That is used at FAB buttons.

KryptKode commented 1 year ago

What about the sound? @tibbi shall we keep or remove it?

tibbi commented 1 year ago

lets add some more silent beep, not that high frequency. Something deeper.

KryptKode commented 1 year ago

Any idea where I can get such a sound resource?

tibbi commented 1 year ago

not really, I could just google too. Look for something with a free license though. But try fixing the other issues first, I can replace the audio myself easily too.

KryptKode commented 1 year ago

Hi @tibbi . I addressed all the changes

For sound, I would address this in another pull request. I found some free sound resources here

tibbi commented 1 year ago

maybe the beeps could speed up before the capture, like other apps do it. Just so the user knows it is coming. Also it is unrelated to this task, but I just noticed that portrait photos keep showing upside down in the latest gallery. It shows up well initially, then as the zoomable subsampling-scale-imageview shows up, it is upside down.

tibbi commented 1 year ago

and also make sure the top icons are evenly aligned, if Flash is not visible. I guess that other buttons are always visible

camera

KryptKode commented 1 year ago

Hi @tibbi I addressed the issue of the top icons not being evenly spaced. I would address all beep sound-related changes in another pull request to reduce the scope of changes if you don't mind.

About the issue with images in portrait mode being upside down, I am using 6.25.0 Pro of Gallery and it works fine. I would investigate with the latest gallery app and make the changes in a future pull request.

Screenshot
tibbi commented 1 year ago

guess Ill check the upside down bug myself too, if you cannot reproduce it. Anyway, that isnt related to this task.

KryptKode commented 1 year ago

Yes, I couldn't reproduce it now with the latest master. Maybe if you could tell me the steps and the settings you have on both the camera and gallery apps, it might help.

tibbi commented 1 year ago

I keep reinstalling the apps, so everything is the default. It will be some rotation value somewhere, will take a look at it.

tibbi commented 1 year ago

apparently it breaks only if "Flip front camera photos horizontally" is enabled

tibbi commented 1 year ago

changes from ORIENTATION_ROTATE_270 to ORIENTATION_TRANSVERSE

tibbi commented 1 year ago

it keeps rotating chaotically in other galleries too, sometimes to the side too. There is likely something wrong with setting those exif values. Maybe simply rotating the image itself and using orientation_normal would be better