Open
Conversation
This comment has been minimized.
This comment has been minimized.
| currentTabIndex = tabs.indexOf(selectedTab), | ||
| selectedColor = Colors.White, | ||
| onTabChange = { selectedTab = it }, | ||
| modifier = Modifier.padding(horizontal = 16.dp), |
Contributor
There was a problem hiding this comment.
CLAUDE.md violation: trailing comma on modifier = at a call site.
The project rule (CLAUDE.md) states:
NEVER add a trailing comma to
modifier = ...at call sites
Suggested change
| modifier = Modifier.padding(horizontal = 16.dp), | |
| CustomTabRowWithSpacing( | |
| tabs = tabs, | |
| currentTabIndex = tabs.indexOf(selectedTab), | |
| selectedColor = Colors.White, | |
| onTabChange = { selectedTab = it }, | |
| modifier = Modifier.padding(horizontal = 16.dp) | |
| ) |
This comment has been minimized.
This comment has been minimized.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR ports four design-review fixes from bitkit-ios
fix/design-reviewto keep Android parity with iOS:Description
textPrimary.belldrawables across density buckets from the new iOS 768×768 source PNG so the bell illustration fills more of the sheet area. Stored as PNG during the port; can be re-encoded to WebP later if APK size becomes a concern.#151515background (darker than the surrounding cards) instead of the translucentWhite06. The flex spacer moved to above the button and a fixed 62dp gap sits between the button and the swipe-to-confirm control — matching iOSSpacer(minLength: 16)+.padding(.bottom, 62).SIZE_LATESTbumped from 3 to 4 inActivityListViewModel, andHomeScreenreplaces its fixed small/large activity counts with a slot-budget calculation:slotCapacity = isSmallScreen ? 3 : 4, minus 1 per non-item slot (transfer banner, widgets onboarding hint). With nothing else showing the home list fits 4 items (3 on small screens); when the banner or widgets hint is visible the count shrinks so the overall rhythm stays within the cap.Note: iOS also added a pressed-state gradient swap to
PrimaryButtonViewso custom-background buttons still show press feedback. Android'sPrimaryButtonalready has analphaFeedbackmodifier providing equivalent press feedback, so this change is not mirrored on Android.Preview
QA Notes
shop.webm
qr-code.webm
Screen_recording_20260424_103644.webm
Screen_recording_20260424_104619.webm
small-device.webm
medium.device.webm
./gradlew compileDevDebugKotlinclean./gradlew testDevDebugUnitTestclean./gradlew detektclean