Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add LLB as a FlashMode, and remove existing LLB toggle related settings #269

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

yasith
Copy link
Member

@yasith yasith commented Oct 9, 2024

LLB Flash Toggle

  • Add Low Light Boost (LLB) as an additional Flash option
  • This will be combined with Implement LowLightBoost Capture #236 later on (NightLight icon outline vs filled when LLB is active)

@yasith yasith requested a review from Kimblebee October 9, 2024 17:35
@Kimblebee
Copy link
Collaborator

just curious, is there ever a scenario where someone would want LLB and Flash to be on simultaneously? Like in a wide open area where the flash may help give a little extra light but not enough for it to fully illuminate? or taking a photo of someone at a bit of a distance in the dark

@yasith
Copy link
Member Author

yasith commented Oct 10, 2024

just curious, is there ever a scenario where someone would want LLB and Flash to be on simultaneously? Like in a wide open area where the flash may help give a little extra light but not enough for it to fully illuminate? or taking a photo of someone at a bit of a distance in the dark

From my understanding LLB doesn't work well with Flash.
For all existing implementations of LLB and Night Capture, flash is turned off.

It's not something in the current scope.

Copy link
Collaborator

@Kimblebee Kimblebee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.
Noted that we may have to include a constraint for this new option in the default settings screen

Besides that, only small changes to be addressed and it should be good to go :-)

@yasith yasith requested a review from Kimblebee October 11, 2024 16:26
@Kimblebee
Copy link
Collaborator

Looks good, but you need to do one more thing.

Go into LocalSettingsRepository.kt

add this line: FlashModeProto.FLASH_MODE_LOW_LIGHT_BOOST -> FlashMode.LOW_LIGHT_BOOST

into

flashMode = when (it.flashModeStatus) {
                   FlashModeProto.FLASH_MODE_AUTO -> FlashMode.AUTO
                   FlashModeProto.FLASH_MODE_ON -> FlashMode.ON
                   FlashModeProto.FLASH_MODE_OFF -> FlashMode.OFF
                   FlashModeProto.FLASH_MODE_LOW_LIGHT_BOOST -> FlashMode.LOW_LIGHT_BOOST
                   else -> FlashMode.OFF
               }

The default setting will not set to LLB unless you include that line

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants