Skip to content

Fix background color and add APK build action#14

Merged
HereLiesAz merged 2 commits into
masterfrom
jules-fix-bg-color-and-add-apk-action-5856042109503101069
May 13, 2026
Merged

Fix background color and add APK build action#14
HereLiesAz merged 2 commits into
masterfrom
jules-fix-bg-color-and-add-apk-action-5856042109503101069

Conversation

@HereLiesAz

Copy link
Copy Markdown
Owner
  • Changed Material3 default background and surface color in Theme.kt to #FFFFFF.
  • Added GitHub action .github/workflows/build-apk.yml to automatically build and publish a pre-release debug APK on push.
  • Used secrets.GH_TOKEN for GitHub release creation to resolve permission issues.
  • Removed broken jules-pr-lifecycle and assign-jules actions.

- Changed Material3 default background and surface color in `Theme.kt` to #FFFFFF.
- Added GitHub action `.github/workflows/build-apk.yml` to automatically build and publish a pre-release debug APK on push.
- Used `secrets.GH_TOKEN` for GitHub release creation to resolve permission issues.
- Removed broken `jules-pr-lifecycle` and `assign-jules` actions.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sorry @HereLiesAz, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@HereLiesAz HereLiesAz merged commit 9ffef30 into master May 13, 2026
2 of 4 checks passed
@HereLiesAz HereLiesAz deleted the jules-fix-bg-color-and-add-apk-action-5856042109503101069 branch May 13, 2026 16:53

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

Copy link
Copy Markdown

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 updates the LightColorScheme in Theme.kt by explicitly defining background and surface colors. The reviewer suggests using the more idiomatic Color.White instead of hex literals and recommends centralizing these color definitions in Color.kt to improve maintainability and consistency.

Comment on lines +17 to +18
background = Color(0xFFFFFFFF),
surface = Color(0xFFFFFFFF)

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Using Color.White is more idiomatic and readable in Jetpack Compose than the hex literal Color(0xFFFFFFFF). Furthermore, to maintain consistency with the project's established structure, consider updating the existing color constants in Color.kt (such as ScreenBackground or CardBackground) and referencing them here instead of hardcoding hex values directly in the theme definition.

Suggested change
background = Color(0xFFFFFFFF),
surface = Color(0xFFFFFFFF)
background = Color.White,
surface = Color.White

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.

1 participant