Skip to content

fix: various fixes#508

Merged
pwltr merged 4 commits intomasterfrom
fix/various-fixes
Mar 27, 2026
Merged

fix: various fixes#508
pwltr merged 4 commits intomasterfrom
fix/various-fixes

Conversation

@pwltr
Copy link
Copy Markdown
Contributor

@pwltr pwltr commented Mar 25, 2026

Some minor design updates and fixes.

Description

  • fix(widgets): add keyboard avoidance
  • fix(wallet): update savings/spending wallet screen UI
  • fix(ui): add secondary button custom blur
  • fix(widgets): update widgets list order
  • fix(shop): minor UI fixes

Linked Issues/Tasks

#493

Screenshot / Video

Simulator.Screen.Recording.-.iPhone.17.-.2026-03-25.at.14.16.39.mov

@pwltr pwltr linked an issue Mar 25, 2026 that may be closed by this pull request
@pwltr pwltr self-assigned this Mar 25, 2026
@pwltr pwltr added this to the 2.2.0 milestone Mar 25, 2026
@pwltr pwltr force-pushed the fix/various-fixes branch from f7a45fe to 5f9d06d Compare March 26, 2026 09:26
@pwltr pwltr requested review from jvsena42 and piotr-iohk March 26, 2026 10:04
@piotr-iohk
Copy link
Copy Markdown
Collaborator

Mostly LGTM, however the calculator widget keyboard behaves a bit strange:

  • on appear there's sometimes a black space covering part of the widget
  • maybe also it would be good to be able to close the keyboard by tapping somewhere outside, currently cannot close it until I swipe to main wallet screen.
Screen.Recording.2026-03-26.at.13.25.08.mov

@pwltr
Copy link
Copy Markdown
Contributor Author

pwltr commented Mar 26, 2026

Mostly LGTM, however the calculator widget keyboard behaves a bit strange:

  • on appear there's sometimes a black space covering part of the widget
  • maybe also it would be good to be able to close the keyboard by tapping somewhere outside, currently cannot close it until I swipe to main wallet screen.

Screen.Recording.2026-03-26.at.13.25.08.mov

Right, this was hard to get working because there is a ScrollView nested in another one. I thought I got it to an acceptable state but retesting now it is very broken. Let's remove that commit from the PR and I'll give it another try separately.

@pwltr pwltr force-pushed the fix/various-fixes branch from 5f9d06d to 8ca48de Compare March 26, 2026 17:21
@claude
Copy link
Copy Markdown

claude bot commented Mar 26, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

Copy link
Copy Markdown
Collaborator

@piotr-iohk piotr-iohk left a comment

Choose a reason for hiding this comment

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

LGTM

@pwltr pwltr merged commit 7d7f035 into master Mar 27, 2026
17 of 19 checks passed
@pwltr pwltr deleted the fix/various-fixes branch March 27, 2026 09:21
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.

[Bug]: Order of the widgets in Add widget screen is incorrect in iOS

2 participants