LN-Zap / zap-android

Zap Wallet - Native android lightning wallet for node operators focused on user experience and ease of use ⚡️
MIT License
165 stars 49 forks source link

Complete refactoring of bottom sheet dialogs #296

Closed michaelWuensch closed 3 years ago

michaelWuensch commented 3 years ago

Description

This very big PR completely refactors how the bottom sheet dialogs behave and are animated. Animations do look slightly different now

Motivation and Context

The dialogs have been very messy with a lot of duplicated code. (saved 1700 lines of code with this PR) This makes code a lot easier to read / review or to contribute.

One further reason is that bottom sheet dialogs were very limited in their capabilities.

Now all Bottom sheet dialogs behave the following way:

Especially the scrolling behavior makes it much more useful as we are now no longer limited to the small space.

How Has This Been Tested?

On my S9, emulator

Types of changes

Checklist: