What changed, and why it matters
This commit adjusts the height of two buttons (Cancel and Confirm) in a settings dialog so that on mobile devices the buttons use a default height instead of a fixed large height. It is purely a user-interface layout fix with no security relevance.
No security action needed; treat as a normal UI fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change modifies lib/pages/shopinbit/shopinbit_settings_view.dart, setting buttonHeight for SecondaryButton and PrimaryButton inside _VerifyKeyDialogState to Util.isDesktop ? ButtonHeight.l : null. This makes the buttons responsive: desktop keeps the large fixed height, while mobile falls back to a default height. There is no functional, cryptographic, or security logic changed.
Changed components
lib/pages/shopinbit/shopinbit_settings_view.dartInspect captured patch +2 / −2
diff --git a/lib/pages/shopinbit/shopinbit_settings_view.dart b/lib/pages/shopinbit/shopinbit_settings_view.dart
index a4b36b2..8580f04 100644
--- a/lib/pages/shopinbit/shopinbit_settings_view.dart
+++ b/lib/pages/shopinbit/shopinbit_settings_view.dart
@@ -832,7 +832,7 @@ class _VerifyKeyDialogState extends State<_VerifyKeyDialog> {
Expanded(
child: SecondaryButton(
label: "Cancel",
- buttonHeight: ButtonHeight.l,
+ buttonHeight: Util.isDesktop ? ButtonHeight.l : null,
onPressed: () => Navigator.of(
context,
rootNavigator: Util.isDesktop,
@@ -845,7 +845,7 @@ class _VerifyKeyDialogState extends State<_VerifyKeyDialog> {
Expanded(
child: PrimaryButton(
label: "Confirm",
- buttonHeight: ButtonHeight.l,
+ buttonHeight: Util.isDesktop ? ButtonHeight.l : null,
enabled: _confirmEnabled,
onPressed: _confirmEnabled
? () => Navigator.of(
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.