peercoin / peercoin_flutter

Light Peercoin wallet written in Flutter, deployable on Android, iOS and Web.
https://wallet.peercoin.net
GNU Affero General Public License v3.0
26 stars 14 forks source link

New setup screens #78

Closed lorenz10 closed 2 years ago

lorenz10 commented 2 years ago

Update 2

peerchemist commented 2 years ago

That's quite good.

lorenz10 commented 2 years ago

I considered your suggestions and updated the images, so please check the text. I think there should be somthing dividing Create Wallet and Import Wallet buttons but I don't like the divider, I'll think about it...

lorenz10 commented 2 years ago

For me that's it @bananenwilly

willyfromtheblock commented 2 years ago

grafik I'm not a fan of the numbered list. I like the formatting it gives, but I dislike the leading numbers. Have you seen this anywhere else?

Also, would be nice to have kept the double tap to copy.

grafik Furthermore, 24 words overflow

willyfromtheblock commented 2 years ago

btw, can we handle this at the same time? https://github.com/peercoin/peercoin_flutter/projects/1#card-68213985

would be awesome

belovachap commented 2 years ago

The images are pretty cool, who drew them?

lorenz10 commented 2 years ago

grafik I'm not a fan of the numbered list. I like the formatting it gives, but I dislike the leading numbers. Have you seen this anywhere else?

Also, would be nice to have kept the double tap to copy.

grafik Furthermore, 24 words overflow

Check the overflow. Also check if the suggestion text is too clear or is readable ...

lorenz10 commented 2 years ago

The images are pretty cool, who drew them?

https://designs.ai/graphicmaker

willyfromtheblock commented 2 years ago
Run flutter analyze
Analyzing peercoin_flutter...                                   

   info • 'accentColor' is deprecated and shouldn't be used. Use colorScheme.secondary instead. For more information, consult the migration guide at https://flutter.dev/docs/release/breaking-changes/theme-data-accent-properties#migration-guide. This feature was deprecated after v2.3.0-0.1.pre. • lib/screens/setup/setup_import_seed.dart:158:81 • deprecated_member_use

1 issue found. (ran in 32.7s)
Error: Process completed with exit code 1.
willyfromtheblock commented 2 years ago

Screenshot_1631885328 dark mode is pretty low contrast

willyfromtheblock commented 2 years ago

Screenshot_1631887122

cool effect, hehe