iron-fish / node-app

Electron-based Node App
https://ironfish.network/use/node-app
17 stars 8 forks source link

feat(ifl-1047): reset node and confirm modal #302

Closed jowparks closed 1 year ago

jowparks commented 1 year ago

Summary:

https://github.com/iron-fish/node-app/assets/26990067/02034166-59b1-4eba-a38f-6f29473c8589

jowparks commented 1 year ago
Screen Shot 2023-06-04 at 4 00 53 PM

I'd be weary of creating new button styling as it'll create inconsistencies within the app.

Checkout an account's settings and see the delete text link as an example of how we've implemented ui patterns in this vein. @dgca not sure if you want to handle the implementation or not?

I see what you mean, I will just make it a link rather than a button. Not sure what you mean by linking @dgca in here, hopefully he can sus it out.

skylarrichard123 commented 1 year ago

@jowparks sounds good. Was tagging dan because he's one of the ui kit owners and I wasn't sure if you had access to the existing design patterns from it, but the feedback is probably straight forward enough.

skylarrichard123 commented 1 year ago
Screen Shot 2023-06-07 at 2 05 26 PM
jowparks commented 1 year ago
Screen Shot 2023-06-07 at 2 05 26 PM

Ya I fixed this in the updated version

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
wallet-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 7, 2023 5:48pm