Skip to content

Feature/add bitbox support#18

Merged
xlamn merged 12 commits into
trunkfrom
feature/add-bitbox-support
Dec 16, 2025
Merged

Feature/add bitbox support#18
xlamn merged 12 commits into
trunkfrom
feature/add-bitbox-support

Conversation

@konstantinullrich
Copy link
Copy Markdown
Collaborator

No description provided.

# Conflicts:
#	lib/packages/service/app_store.dart
#	lib/packages/wallet/wallet.dart
#	lib/packages/wallet/wallet_account.dart
@xlamn xlamn linked an issue Dec 7, 2025 that may be closed by this pull request
Comment thread lib/packages/wallet/wallet.dart Outdated
Comment thread lib/screens/hardware_connect_bitbox/bloc/connect_bitbox_cubit.dart Outdated
Comment thread lib/screens/hardware_connect_bitbox/bloc/connect_bitbox_cubit.dart Outdated
Comment thread lib/screens/hardware_connect_bitbox/bloc/connect_bitbox_cubit.dart
Comment thread lib/screens/hardware_connect_bitbox/connect_bitbox_page.dart Outdated
Comment thread lib/screens/hardware_connect_bitbox/connect_bitbox_view.dart Outdated
Comment thread lib/styles/styles.dart Outdated
Comment thread lib/styles/styles.dart Outdated
Comment thread lib/widgets/secondary_button.dart Outdated
Comment thread lib/packages/hardware_wallet/bitbox_credentials.dart
style: kBottonSheetTitleTextStyle,
),
SizedBox(
width: 330,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this does not seem responsive.

Comment thread lib/styles/styles.dart Outdated
Comment thread lib/styles/styles.dart Outdated
konstantinullrich and others added 5 commits December 10, 2025 14:59
Co-authored-by: Lam Nguyen <32935491+xlamn@users.noreply.github.com>
Co-authored-by: Lam Nguyen <32935491+xlamn@users.noreply.github.com>
# Conflicts:
#	assets/languages/strings_de.arb
#	assets/languages/strings_en.arb
#	lib/styles/colors.dart
description: S.of(context).hardware_wallet_subtitle,
trailing: SvgPicture.asset(
'assets/images/illustrations/bitbox.svg',
if (defaultTargetPlatform == TargetPlatform.android)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We could extend DeviceInfo with two other functions: One for is iOS and one for isAndroid so we have it bundled.

offset: state is BitboxNotConnected ? Offset.zero : const Offset(-1.2, 0),
child: ConnectContent(
title: S.of(context).connect_bitbox_title,
content: defaultTargetPlatform == TargetPlatform.iOS
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

see other comment.

@xlamn xlamn merged commit b9ce1c1 into trunk Dec 16, 2025
1 check passed
@xlamn xlamn deleted the feature/add-bitbox-support branch December 16, 2025 14:39
@xlamn xlamn added the feat A new feature label Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bitbox integration

2 participants