Skip to content
This repository has been archived by the owner on Aug 27, 2022. It is now read-only.

implement countdown offset to compensate shutter-delay and cheese time #286

Merged
merged 1 commit into from
Aug 5, 2021

Conversation

Metropo
Copy link

@Metropo Metropo commented Aug 3, 2021

Prerequisites checklist

What is the purpose of this pull request? (put an "x" next to an item)

  • New feature

What changes did you make? (Give an overview)

I noticed, that some users that wanted to take pictures with the photobooth seemed to be confused or misled on the time the picture is taken. They assumed that at the moment the box displays "Cheeeeeese" and the live preview hides the image gets taken. This is probably encouraged by the mirror of the DSLR going down between live view and taking the picture.
A colleague suggested changing the countdown in a way, that it reaches 0 at the approximate time the photo is really taken. I implemented it using an offset that is added to the displayed countdown. It looks like this now
6, 5, 4, 3, Cheeeeeese [1 second] + shutter delay + Autofocus [1-2 seconds]
In my opinion, it is more intuitive this way. I would be glad to hear your opinion.

Is there anything you'd like reviewers to focus on?

@andi34
Copy link
Owner

andi34 commented Aug 3, 2021

Thanks! I'll take a closer look at it later today. Changes to the language files only need to be done to the english file, all other languages need to be translated at crowdin once submitted ( https://crowdin.com/project/photobooth ).

@Metropo
Copy link
Author

Metropo commented Aug 3, 2021

@andi34 Thanks for the information. I reverted the change to the german langauge file.

resources/lang/en.json Outdated Show resolved Hide resolved
resources/lang/en.json Outdated Show resolved Hide resolved
@Metropo
Copy link
Author

Metropo commented Aug 5, 2021

I fixed the issues and rebased onto current dev branch.

@andi34 andi34 merged commit 563f257 into andi34:dev Aug 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants