Add Button Sizing and Haptic Feedback #59

balance-io / old-balance-ios

Balance in Swift

GNU General Public License v3.0

49 stars 16 forks source link

Add Button Sizing and Haptic Feedback #59 #105

Closed borwahs closed 5 years ago

borwahs commented 5 years ago

Prepare and call Haptic Feedback on Add Wallet action.
Run Haptic Feedback on Eth Wallet Add success.
Create new imageset Asset using Plus Icon.
Change Add Wallet button to use image and title.

Issue #59

CLAassistant commented 5 years ago

All committers have signed the CLA.

borwahs commented 5 years ago

Here is the Add Button I changed to:

Let me know if you like that for a bigger touch target.

borwahs commented 5 years ago

One thing I just thought of is that if support for pre iPhone 7 devices is needed, we will need to use the original taptic engine. Let me know your thoughts on going that route.

einsteinx2 commented 5 years ago

For now I think it's fine if this only works on iPhone 7 and up, as long as it doesn't crash or cause other problems on older devices.

borwahs commented 5 years ago

Playing with my change to the Add Button, I think I should remove the text. It looks a bit too bland. Maybe the grey is the issue?