Skip to content

Conversation

@MagicalMeghan
Copy link
Contributor

Updating Jetcaster to only use dark color theme even when device isn't in dark mode

@MagicalMeghan MagicalMeghan requested a review from a team as a code owner July 8, 2025 20:25
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Summary of Changes

Hello @MagicalMeghan, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request simplifies the theming implementation for the Jetcaster Android application. It modifies the Theme.kt file to enforce a dark color theme across the entire application, overriding the device's system-wide dark mode setting and removing support for dynamic theming and light color schemes.

Highlights

  • Forced Dark Theme: The Jetcaster application's theme logic has been updated to always apply the darkScheme, ensuring a consistent dark mode experience regardless of the device's system settings.
  • Theme Simplification: All definitions and imports related to light color schemes, dynamic theming, and various contrast variations (medium and high contrast for both light and dark themes) have been removed, streamlining the theme configuration.
  • Code Cleanup: The JetcasterTheme Composable function has been simplified by removing parameters for darkTheme and dynamicColor, as well as the conditional logic that previously selected between different color schemes.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist is currently in preview and may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request enforces a dark theme for the Jetcaster app by removing light theme support. The changes simplify the codebase by removing unused theme definitions. A critical issue regarding a breaking API change and medium-severity suggestions for further code cleanup have been identified.

Comment on lines 104 to 105
fun JetcasterTheme(
darkTheme: Boolean = isSystemInDarkTheme(),
// Dynamic color is available on Android 12+
dynamicColor: Boolean = false,
content: @Composable () -> Unit,
Copy link
Contributor

Choose a reason for hiding this comment

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

critical

Removing the darkTheme and dynamicColor parameters from the JetcasterTheme composable is a breaking API change. Existing call sites providing arguments for these parameters will now cause compilation errors. Ensure all usages of JetcasterTheme are updated to the new signature.

Comment on lines 19 to 63
import androidx.compose.material3.ExperimentalMaterial3ExpressiveApi
import androidx.compose.material3.MaterialExpressiveTheme
import androidx.compose.material3.MotionScheme
import androidx.compose.material3.darkColorScheme
import androidx.compose.material3.dynamicDarkColorScheme
import androidx.compose.material3.dynamicLightColorScheme
import androidx.compose.material3.lightColorScheme
import androidx.compose.runtime.Composable
import androidx.compose.runtime.Immutable
import androidx.compose.ui.graphics.Color
import androidx.compose.ui.platform.LocalContext
import com.example.jetcaster.designsystem.theme.JetcasterShapes
import com.example.jetcaster.designsystem.theme.JetcasterTypography
import com.example.jetcaster.designsystem.theme.backgroundDark
import com.example.jetcaster.designsystem.theme.backgroundDarkHighContrast
import com.example.jetcaster.designsystem.theme.backgroundDarkMediumContrast
import com.example.jetcaster.designsystem.theme.backgroundLight
import com.example.jetcaster.designsystem.theme.backgroundLightHighContrast
import com.example.jetcaster.designsystem.theme.backgroundLightMediumContrast
import com.example.jetcaster.designsystem.theme.errorContainerDark
import com.example.jetcaster.designsystem.theme.errorContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.errorContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.errorContainerLight
import com.example.jetcaster.designsystem.theme.errorContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.errorContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.errorDark
import com.example.jetcaster.designsystem.theme.errorDarkHighContrast
import com.example.jetcaster.designsystem.theme.errorDarkMediumContrast
import com.example.jetcaster.designsystem.theme.errorLight
import com.example.jetcaster.designsystem.theme.errorLightHighContrast
import com.example.jetcaster.designsystem.theme.errorLightMediumContrast
import com.example.jetcaster.designsystem.theme.inverseOnSurfaceDark
import com.example.jetcaster.designsystem.theme.inverseOnSurfaceDarkHighContrast
import com.example.jetcaster.designsystem.theme.inverseOnSurfaceDarkMediumContrast
import com.example.jetcaster.designsystem.theme.inverseOnSurfaceLight
import com.example.jetcaster.designsystem.theme.inverseOnSurfaceLightHighContrast
import com.example.jetcaster.designsystem.theme.inverseOnSurfaceLightMediumContrast
import com.example.jetcaster.designsystem.theme.inversePrimaryDark
import com.example.jetcaster.designsystem.theme.inversePrimaryDarkHighContrast
import com.example.jetcaster.designsystem.theme.inversePrimaryDarkMediumContrast
import com.example.jetcaster.designsystem.theme.inversePrimaryLight
import com.example.jetcaster.designsystem.theme.inversePrimaryLightHighContrast
import com.example.jetcaster.designsystem.theme.inversePrimaryLightMediumContrast
import com.example.jetcaster.designsystem.theme.inverseSurfaceDark
import com.example.jetcaster.designsystem.theme.inverseSurfaceDarkHighContrast
import com.example.jetcaster.designsystem.theme.inverseSurfaceDarkMediumContrast
import com.example.jetcaster.designsystem.theme.inverseSurfaceLight
import com.example.jetcaster.designsystem.theme.inverseSurfaceLightHighContrast
import com.example.jetcaster.designsystem.theme.inverseSurfaceLightMediumContrast
import com.example.jetcaster.designsystem.theme.onBackgroundDark
import com.example.jetcaster.designsystem.theme.onBackgroundDarkHighContrast
import com.example.jetcaster.designsystem.theme.onBackgroundDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onBackgroundLight
import com.example.jetcaster.designsystem.theme.onBackgroundLightHighContrast
import com.example.jetcaster.designsystem.theme.onBackgroundLightMediumContrast
import com.example.jetcaster.designsystem.theme.onErrorContainerDark
import com.example.jetcaster.designsystem.theme.onErrorContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.onErrorContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onErrorContainerLight
import com.example.jetcaster.designsystem.theme.onErrorContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.onErrorContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.onErrorDark
import com.example.jetcaster.designsystem.theme.onErrorDarkHighContrast
import com.example.jetcaster.designsystem.theme.onErrorDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onErrorLight
import com.example.jetcaster.designsystem.theme.onErrorLightHighContrast
import com.example.jetcaster.designsystem.theme.onErrorLightMediumContrast
import com.example.jetcaster.designsystem.theme.onPrimaryContainerDark
import com.example.jetcaster.designsystem.theme.onPrimaryContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.onPrimaryContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onPrimaryContainerLight
import com.example.jetcaster.designsystem.theme.onPrimaryContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.onPrimaryContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.onPrimaryDark
import com.example.jetcaster.designsystem.theme.onPrimaryDarkHighContrast
import com.example.jetcaster.designsystem.theme.onPrimaryDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onPrimaryLight
import com.example.jetcaster.designsystem.theme.onPrimaryLightHighContrast
import com.example.jetcaster.designsystem.theme.onPrimaryLightMediumContrast
import com.example.jetcaster.designsystem.theme.onSecondaryContainerDark
import com.example.jetcaster.designsystem.theme.onSecondaryContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.onSecondaryContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onSecondaryContainerLight
import com.example.jetcaster.designsystem.theme.onSecondaryContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.onSecondaryContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.onSecondaryDark
import com.example.jetcaster.designsystem.theme.onSecondaryDarkHighContrast
import com.example.jetcaster.designsystem.theme.onSecondaryDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onSecondaryLight
import com.example.jetcaster.designsystem.theme.onSecondaryLightHighContrast
import com.example.jetcaster.designsystem.theme.onSecondaryLightMediumContrast
import com.example.jetcaster.designsystem.theme.onSurfaceDark
import com.example.jetcaster.designsystem.theme.onSurfaceDarkHighContrast
import com.example.jetcaster.designsystem.theme.onSurfaceDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onSurfaceLight
import com.example.jetcaster.designsystem.theme.onSurfaceLightHighContrast
import com.example.jetcaster.designsystem.theme.onSurfaceLightMediumContrast
import com.example.jetcaster.designsystem.theme.onSurfaceVariantDark
import com.example.jetcaster.designsystem.theme.onSurfaceVariantDarkHighContrast
import com.example.jetcaster.designsystem.theme.onSurfaceVariantDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onSurfaceVariantLight
import com.example.jetcaster.designsystem.theme.onSurfaceVariantLightHighContrast
import com.example.jetcaster.designsystem.theme.onSurfaceVariantLightMediumContrast
import com.example.jetcaster.designsystem.theme.onTertiaryContainerDark
import com.example.jetcaster.designsystem.theme.onTertiaryContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.onTertiaryContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onTertiaryContainerLight
import com.example.jetcaster.designsystem.theme.onTertiaryContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.onTertiaryContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.onTertiaryDark
import com.example.jetcaster.designsystem.theme.onTertiaryDarkHighContrast
import com.example.jetcaster.designsystem.theme.onTertiaryDarkMediumContrast
import com.example.jetcaster.designsystem.theme.onTertiaryLight
import com.example.jetcaster.designsystem.theme.onTertiaryLightHighContrast
import com.example.jetcaster.designsystem.theme.onTertiaryLightMediumContrast
import com.example.jetcaster.designsystem.theme.outlineDark
import com.example.jetcaster.designsystem.theme.outlineDarkHighContrast
import com.example.jetcaster.designsystem.theme.outlineDarkMediumContrast
import com.example.jetcaster.designsystem.theme.outlineLight
import com.example.jetcaster.designsystem.theme.outlineLightHighContrast
import com.example.jetcaster.designsystem.theme.outlineLightMediumContrast
import com.example.jetcaster.designsystem.theme.outlineVariantDark
import com.example.jetcaster.designsystem.theme.outlineVariantDarkHighContrast
import com.example.jetcaster.designsystem.theme.outlineVariantDarkMediumContrast
import com.example.jetcaster.designsystem.theme.outlineVariantLight
import com.example.jetcaster.designsystem.theme.outlineVariantLightHighContrast
import com.example.jetcaster.designsystem.theme.outlineVariantLightMediumContrast
import com.example.jetcaster.designsystem.theme.primaryContainerDark
import com.example.jetcaster.designsystem.theme.primaryContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.primaryContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.primaryContainerLight
import com.example.jetcaster.designsystem.theme.primaryContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.primaryContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.primaryDark
import com.example.jetcaster.designsystem.theme.primaryDarkHighContrast
import com.example.jetcaster.designsystem.theme.primaryDarkMediumContrast
import com.example.jetcaster.designsystem.theme.primaryLight
import com.example.jetcaster.designsystem.theme.primaryLightHighContrast
import com.example.jetcaster.designsystem.theme.primaryLightMediumContrast
import com.example.jetcaster.designsystem.theme.scrimDark
import com.example.jetcaster.designsystem.theme.scrimDarkHighContrast
import com.example.jetcaster.designsystem.theme.scrimDarkMediumContrast
import com.example.jetcaster.designsystem.theme.scrimLight
import com.example.jetcaster.designsystem.theme.scrimLightHighContrast
import com.example.jetcaster.designsystem.theme.scrimLightMediumContrast
import com.example.jetcaster.designsystem.theme.secondaryContainerDark
import com.example.jetcaster.designsystem.theme.secondaryContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.secondaryContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.secondaryContainerLight
import com.example.jetcaster.designsystem.theme.secondaryContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.secondaryContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.secondaryDark
import com.example.jetcaster.designsystem.theme.secondaryDarkHighContrast
import com.example.jetcaster.designsystem.theme.secondaryDarkMediumContrast
import com.example.jetcaster.designsystem.theme.secondaryLight
import com.example.jetcaster.designsystem.theme.secondaryLightHighContrast
import com.example.jetcaster.designsystem.theme.secondaryLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceBrightDark
import com.example.jetcaster.designsystem.theme.surfaceBrightDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceBrightDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceBrightLight
import com.example.jetcaster.designsystem.theme.surfaceBrightLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceBrightLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerDark
import com.example.jetcaster.designsystem.theme.surfaceContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerHighDark
import com.example.jetcaster.designsystem.theme.surfaceContainerHighDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerHighDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerHighLight
import com.example.jetcaster.designsystem.theme.surfaceContainerHighLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerHighLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerHighestDark
import com.example.jetcaster.designsystem.theme.surfaceContainerHighestDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerHighestDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerHighestLight
import com.example.jetcaster.designsystem.theme.surfaceContainerHighestLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerHighestLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLight
import com.example.jetcaster.designsystem.theme.surfaceContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLowDark
import com.example.jetcaster.designsystem.theme.surfaceContainerLowDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLowDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLowLight
import com.example.jetcaster.designsystem.theme.surfaceContainerLowLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLowLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLowestDark
import com.example.jetcaster.designsystem.theme.surfaceContainerLowestDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLowestDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLowestLight
import com.example.jetcaster.designsystem.theme.surfaceContainerLowestLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceContainerLowestLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceDark
import com.example.jetcaster.designsystem.theme.surfaceDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceDimDark
import com.example.jetcaster.designsystem.theme.surfaceDimDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceDimDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceDimLight
import com.example.jetcaster.designsystem.theme.surfaceDimLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceDimLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceLight
import com.example.jetcaster.designsystem.theme.surfaceLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceLightMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceVariantDark
import com.example.jetcaster.designsystem.theme.surfaceVariantDarkHighContrast
import com.example.jetcaster.designsystem.theme.surfaceVariantDarkMediumContrast
import com.example.jetcaster.designsystem.theme.surfaceVariantLight
import com.example.jetcaster.designsystem.theme.surfaceVariantLightHighContrast
import com.example.jetcaster.designsystem.theme.surfaceVariantLightMediumContrast
import com.example.jetcaster.designsystem.theme.tertiaryContainerDark
import com.example.jetcaster.designsystem.theme.tertiaryContainerDarkHighContrast
import com.example.jetcaster.designsystem.theme.tertiaryContainerDarkMediumContrast
import com.example.jetcaster.designsystem.theme.tertiaryContainerLight
import com.example.jetcaster.designsystem.theme.tertiaryContainerLightHighContrast
import com.example.jetcaster.designsystem.theme.tertiaryContainerLightMediumContrast
import com.example.jetcaster.designsystem.theme.tertiaryDark
import com.example.jetcaster.designsystem.theme.tertiaryDarkHighContrast
import com.example.jetcaster.designsystem.theme.tertiaryDarkMediumContrast
import com.example.jetcaster.designsystem.theme.tertiaryLight
import com.example.jetcaster.designsystem.theme.tertiaryLightHighContrast
import com.example.jetcaster.designsystem.theme.tertiaryLightMediumContrast

private val lightScheme = lightColorScheme(
primary = primaryLight,
onPrimary = onPrimaryLight,
primaryContainer = primaryContainerLight,
onPrimaryContainer = onPrimaryContainerLight,
secondary = secondaryLight,
onSecondary = onSecondaryLight,
secondaryContainer = secondaryContainerLight,
onSecondaryContainer = onSecondaryContainerLight,
tertiary = tertiaryLight,
onTertiary = onTertiaryLight,
tertiaryContainer = tertiaryContainerLight,
onTertiaryContainer = onTertiaryContainerLight,
error = errorLight,
onError = onErrorLight,
errorContainer = errorContainerLight,
onErrorContainer = onErrorContainerLight,
background = backgroundLight,
onBackground = onBackgroundLight,
surface = surfaceLight,
onSurface = onSurfaceLight,
surfaceVariant = surfaceVariantLight,
onSurfaceVariant = onSurfaceVariantLight,
outline = outlineLight,
outlineVariant = outlineVariantLight,
scrim = scrimLight,
inverseSurface = inverseSurfaceLight,
inverseOnSurface = inverseOnSurfaceLight,
inversePrimary = inversePrimaryLight,
surfaceDim = surfaceDimLight,
surfaceBright = surfaceBrightLight,
surfaceContainerLowest = surfaceContainerLowestLight,
surfaceContainerLow = surfaceContainerLowLight,
surfaceContainer = surfaceContainerLight,
surfaceContainerHigh = surfaceContainerHighLight,
surfaceContainerHighest = surfaceContainerHighestLight,
)

Copy link
Contributor

Choose a reason for hiding this comment

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

medium

Many of these imports are no longer used after the removal of light theme support. Removing unused imports keeps the code clean.

import androidx.compose.material3.ExperimentalMaterial3ExpressiveApi
import androidx.compose.material3.MaterialExpressiveTheme
import androidx.compose.material3.MotionScheme
import androidx.compose.material3.darkColorScheme
import androidx.compose.runtime.Composable
import com.example.jetcaster.designsystem.theme.JetcasterShapes
import com.example.jetcaster.designsystem.theme.JetcasterTypography
import com.example.jetcaster.designsystem.theme.backgroundDark
import com.example.jetcaster.designsystem.theme.errorContainerDark
import com.example.jetcaster.designsystem.theme.errorDark
import com.example.jetcaster.designsystem.theme.inverseOnSurfaceDark
import com.example.jetcaster.designsystem.theme.inversePrimaryDark
import com.example.jetcaster.designsystem.theme.inverseSurfaceDark
import com.example.jetcaster.designsystem.theme.onBackgroundDark
import com.example.jetcaster.designsystem.theme.onErrorContainerDark
import com.example.jetcaster.designsystem.theme.onErrorDark
import com.example.jetcaster.designsystem.theme.onPrimaryContainerDark
import com.example.jetcaster.designsystem.theme.onPrimaryDark
import com.example.jetcaster.designsystem.theme.onSecondaryContainerDark
import com.example.jetcaster.designsystem.theme.onSecondaryDark
import com.example.jetcaster.designsystem.theme.onSurfaceDark
import com.example.jetcaster.designsystem.theme.onSurfaceVariantDark
import com.example.jetcaster.designsystem.theme.onTertiaryContainerDark
import com.example.jetcaster.designsystem.theme.onTertiaryDark
import com.example.jetcaster.designsystem.theme.outlineDark
import com.example.jetcaster.designsystem.theme.outlineVariantDark
import com.example.jetcaster.designsystem.theme.primaryContainerDark
import com.example.jetcaster.designsystem.theme.primaryDark
import com.example.jetcaster.designsystem.theme.scrimDark
import com.example.jetcaster.designsystem.theme.secondaryContainerDark
import com.example.jetcaster.designsystem.theme.secondaryDark
import com.example.jetcaster.designsystem.theme.surfaceBrightDark
import com.example.jetcaster.designsystem.theme.surfaceContainerDark
import com.example.jetcaster.designsystem.theme.surfaceContainerHighDark
import com.example.jetcaster.designsystem.theme.surfaceContainerHighestDark
import com.example.jetcaster.designsystem.theme.surfaceContainerLowDark
import com.example.jetcaster.designsystem.theme.surfaceContainerLowestDark
import com.example.jetcaster.designsystem.theme.surfaceDark
import com.example.jetcaster.designsystem.theme.surfaceDimDark
import com.example.jetcaster.designsystem.theme.surfaceVariantDark
import com.example.jetcaster.designsystem.theme.tertiaryContainerDark
import com.example.jetcaster.designsystem.theme.tertiaryDark

darkTheme -> darkScheme
else -> lightScheme
}
val colorScheme = darkScheme
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

The colorScheme local variable is now redundant since the theme is always darkScheme. You can simplify the code by inlining darkScheme directly into the MaterialExpressiveTheme call.

    MaterialExpressiveTheme(
        colorScheme = darkScheme,
        motionScheme = MotionScheme.expressive(),
        shapes = JetcasterShapes,
        typography = JetcasterTypography,
        content = content,
    )

@MagicalMeghan MagicalMeghan merged commit 310a0ba into main Jul 9, 2025
2 checks passed
@MagicalMeghan MagicalMeghan deleted the jetcaster-force-dark branch July 9, 2025 14:54
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