-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Settings page #199
Settings page #199
Conversation
This has to be in sync with bisq2 PR @ bisq-network/bisq2#3188 |
… animation); TextField updated to handle decimals
e21922d
to
4b842fa
Compare
rebased to latest main changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great @nostrbuddha
ACK (all 3 apps)
The language settings only will apply to that very seem settings screen, we can leave the generalized application for #150 . Hopefully. we'll make it before the MVP release, otherwise we can decide either to do it in full and delay or hide the language settings cause it would be confusing for the user that its there but language doesn't actually change.
@@ -54,6 +56,7 @@ fun TopBar( | |||
val tabNavController: NavHostController = presenter.getRootTabNavController() | |||
|
|||
val interactionEnabled = presenter.isInteractive.collectAsState().value | |||
val showAnimation = presenter.showAnimation.collectAsState().value |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:)
For #148