Update Capture Button visuals

[temcguir]

This PR updates the appearance and behavior of the CaptureButton component to better match design mocks and improve preview visibility.

Key Changes

• Visual Updates:
  • Added support for multiple background styles via LocalShutterBackgroundStyle (WHITE_20 and BLACK_60).
  • Adjusted the size of the capture button, ring stroke width, and nucleus to align with mocks.
  • Fixed alignment of buttons in previews to ensure they are not cut off.

• Accessibility and Localization:

  • Made the CaptureButton focusable and added appropriate roles and content descriptions.
  • Moved content descriptions to strings.xml for proper localization support.
  • Enabled automated accessibility checks in tests.

• Refactoring and Testing:

  • Refactored pressed state previews to use CompositionLocalProvider(LocalInitialPressedState provides true), avoiding manual construction of complex UI states and ensuring accurate rendering in previews.
  • Added screenshot tests for various states (Idle, Pressed, Disabled, Recording) across different capture modes and background styles.
  • Consolidated previews in CaptureButtonScreenshotTest.kt for better maintainability.

[temcguir]

/gemini summary

[gemini-code-assist]

Code Review

This pull request does a great job of updating the CaptureButton component's appearance and behavior. The changes align well with the design mocks, and the fixes for VIDEO_ONLY mode and volume button capture are significant improvements. The refactoring of the animation logic using updateTransition and the introduction of InteractionSource are excellent choices that follow Compose best practices. The simplification of the preview composables is also a welcome change that improves maintainability.

I've left a couple of comments: one to address a logic regression that could cause onRelease to be called twice, and another with a suggestion to simplify a boolean check for better readability. Overall, this is a high-quality contribution.

[gemini-code-assist]

Summary of Changes

This pull request refines the CaptureButton component by implementing several visual enhancements and behavioral corrections. The changes aim to align the button's appearance and interaction more closely with design mocks, improving user experience and visual clarity. Key updates include a new semi-transparent background, precise sizing adjustments, and fixes for recording state indicators and capture animations, alongside a streamlined preview setup.

Highlights

• Visual Updates: A 50% semi-transparent black background has been added to the capture button ring for improved contrast. The capture button's overall size, ring stroke width, and nucleus dimensions have been adjusted to match design specifications. Compose Previews now utilize a gradient background (Gray to Dark Gray) and force a White content color, ensuring the semi-transparent background and button details are clearly visible in preview modes.
• Behavioral Fixes: In VIDEO_ONLY mode, the nucleus now correctly appears white when idle and only turns red when pressed or actively recording, addressing a previous issue where it was always red. The capture animation has been fixed to trigger reliably when capturing via volume buttons. The pressed state color for IMAGE_ONLY mode has also been corrected.
• Refactoring: The preview logic has been simplified and updated to directly use the real CaptureButton implementation, improving consistency and reducing redundancy in preview definitions.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Changelog

• ui/components/capture/src/main/java/com/google/jetpackcamera/ui/components/capture/CaptureButtonComponents.kt
  • Added new animation-related imports: animateColor, animateDp, snap, updateTransition.
  • Introduced MutableInteractionSource, PressInteraction, and collectIsPressedAsState for enhanced interaction handling.
  • Imported MaterialTheme, darkColorScheme, and CompositionLocalProvider for theme and content color management in previews.
  • Added Brush import for gradient backgrounds.
  • Decreased DEFAULT_CAPTURE_BUTTON_SIZE from 80f to 76f.
  • Modified the CaptureButton composable to accept an interactionSource parameter.
  • Implemented PressInteraction emission for key events to provide visual feedback.
  • Removed redundant Log.d calls.
  • Updated the private CaptureButton composable to use the new interactionSource and isPressedInteraction.
  • Adjusted the animatedColor animation specification based on the isVisuallyDisabled state.
  • Refactored onPress and awaitRelease logic to emit PressInteraction events.
  • Passed isVisuallyDisabled and isPressed parameters to LockSwitchCaptureButtonNucleus and CaptureButtonNucleus.
  • Reduced CaptureButtonRing borderWidth from 4f to 3f.
  • Added a 50% semi-transparent black background to the CaptureButtonRing.
  • Introduced NucleusState enum (Disabled, Idle, Pressed) for clearer state management.
  • Adjusted idleImageCaptureScale to .74f and idleVideoCaptureScale to .26f.
  • Added isVisuallyDisabled parameter to CaptureButtonNucleus.
  • Renamed centerShapeSize to standardShapeSize and implemented a new centerShapeSize driven by pressTransition for image capture mode.
  • Reworked the nucleus color animation using updateTransition and NucleusState to provide more granular control over color changes based on the button's state and capture mode.
  • Removed the explicit alpha modification logic from CaptureButtonNucleus as it is now handled by the new color transitions.
  • Refactored all @Preview composables to use a new PreviewCaptureButton wrapper, which configures a MaterialTheme with darkColorScheme, LocalContentColor as Color.White, and a gradient background for improved preview visibility.
  • Updated LockSwitchCaptureButtonNucleus previews to correctly pass isVisuallyDisabled and isPressed.

Activity

• The pull request author, temcguir, requested a summary of the changes.
• A bot, gemini-code-assist[bot], provided a review comment suggesting to simplify the isPressableImageMode logic using a when expression for better readability, citing a style guide rule.

[Kimblebee]

some duplicate comments regarding readibility, otherwise I think the PR looks great