← Watch feed
fix styling on new wallet type selection (#3392)
Public commit record
53/100 · Thin
What the developer wrote
Authored by malik1004x
fix styling on new wallet type selection (#3392)
✓ Descriptive subject✓ Names a concrete action or component✓ Links an issue, advisory, or supporting reference! No meaningful explanatory body
What changed, and why it matters
This commit is a purely cosmetic UI fix for the wallet type selection screen in the Cake Wallet app. It wraps a list container in a rounded clipping widget, slightly increases row height from 48 to 50 pixels, and adjusts indentation. There is no security-relevant change.
Recommended action
No security action needed. Treat as routine UI/styling maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies lib/src/screens/new_wallet/new_wallet_type_page.dart only. It wraps the existing Container with a ClipRRect(borderRadius: BorderRadius.circular(18)) to clip child content to rounded corners, changes SizedBox height from 48 to 50, and fixes whitespace/indentation. No logic, input handling, network, crypto, or permission code is touched.
Changed components
lib/src/screens/new_wallet/new_wallet_type_page.dartInspect captured patch +61 / −58
diff --git a/lib/src/screens/new_wallet/new_wallet_type_page.dart b/lib/src/screens/new_wallet/new_wallet_type_page.dart
index 2c44370e..076ff48f 100644
--- a/lib/src/screens/new_wallet/new_wallet_type_page.dart
+++ b/lib/src/screens/new_wallet/new_wallet_type_page.dart
@@ -139,68 +139,71 @@ class WalletTypeFormState extends State<WalletTypeForm> {
child: SingleChildScrollView(
child: Padding(
padding: const EdgeInsets.only(left: 18, right: 18, bottom: 12),
- child: Container(
- decoration: BoxDecoration(
- color: Theme.of(context).colorScheme.surfaceContainer,
- borderRadius: BorderRadius.circular(18),
- ),
- child: ListView.separated(
- key: const ValueKey('new_wallet_type_scrollable_key'),
- itemCount: filteredTypes.length,
- shrinkWrap: true,
- physics: NeverScrollableScrollPhysics(),
- padding: EdgeInsets.zero,
- itemBuilder: (context, index) {
- final item = filteredTypes[index];
-
- return GestureDetector(
- key: ValueKey('new_wallet_type_${item.name}_button_key'),
- behavior: HitTestBehavior.opaque,
- onTap: () {
- onTypeSelected(item);
- },
- child: SizedBox(
- height: 48,
- child: Padding(
- padding: const EdgeInsets.symmetric(horizontal: 12.0),
- child: Row(
- mainAxisAlignment: MainAxisAlignment.spaceBetween,
- children: [
- Row(
- children: [
- CakeImageWidget(
- height: 24,
- width: 24,
- imageUrl: getCryptoCurrencyIconForWalletListItem(item),
- ),
- const SizedBox(width: 12),
- Text(walletTypeToDisplayName(item)),
- const SizedBox(width: 4),
- Text(
- walletTypeToDisplayTicker(item),
- style: TextStyle(
- color: Theme.of(context).colorScheme.onSurfaceVariant,
+ child: ClipRRect(
+ borderRadius: BorderRadius.circular(18),
+ child: Container(
+ decoration: BoxDecoration(
+ color: Theme.of(context).colorScheme.surfaceContainer,
+ borderRadius: BorderRadius.circular(18),
+ ),
+ child: ListView.separated(
+ key: const ValueKey('new_wallet_type_scrollable_key'),
+ itemCount: filteredTypes.length,
+ shrinkWrap: true,
+ physics: NeverScrollableScrollPhysics(),
+ padding: EdgeInsets.zero,
+ itemBuilder: (context, index) {
+ final item = filteredTypes[index];
+
+ return GestureDetector(
+ key: ValueKey('new_wallet_type_${item.name}_button_key'),
+ behavior: HitTestBehavior.opaque,
+ onTap: () {
+ onTypeSelected(item);
+ },
+ child: SizedBox(
+ height: 50,
+ child: Padding(
+ padding: const EdgeInsets.symmetric(horizontal: 12.0),
+ child: Row(
+ mainAxisAlignment: MainAxisAlignment.spaceBetween,
+ children: [
+ Row(
+ children: [
+ CakeImageWidget(
+ height: 24,
+ width: 24,
+ imageUrl: getCryptoCurrencyIconForWalletListItem(item),
),
- )
- ],
- ),
- CakeImageWidget(
- imageUrl: "assets/new-ui/arrow_forward.svg",
- height: 16,
- colorFilter: ColorFilter.mode(
- Theme.of(context).colorScheme.onSurfaceVariant,
- BlendMode.srcIn,
+ const SizedBox(width: 12),
+ Text(walletTypeToDisplayName(item)),
+ const SizedBox(width: 4),
+ Text(
+ walletTypeToDisplayTicker(item),
+ style: TextStyle(
+ color: Theme.of(context).colorScheme.onSurfaceVariant,
+ ),
+ )
+ ],
),
- )
- ],
+ CakeImageWidget(
+ imageUrl: "assets/new-ui/arrow_forward.svg",
+ height: 16,
+ colorFilter: ColorFilter.mode(
+ Theme.of(context).colorScheme.onSurfaceVariant,
+ BlendMode.srcIn,
+ ),
+ )
+ ],
+ ),
),
),
- ),
- );
- },
- separatorBuilder: (context, index) => Container(
- height: 1,
- color: Theme.of(context).colorScheme.surfaceContainerHigh,
+ );
+ },
+ separatorBuilder: (context, index) => Container(
+ height: 1,
+ color: Theme.of(context).colorScheme.surfaceContainerHigh,
+ ),
),
),
),
Risk score
Our methodology →Why this scored 15/100
Human-validated context
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
No validated notes yet.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.