-
Notifications
You must be signed in to change notification settings - Fork 29
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
strange design when wallet widget is minimized #29
Comments
I tried but still the same thing |
Ah, I bet it's because it's using the QR image cached from the previous version of the widget. When you receive funds it will generate a new one for the new address that I think is a little larger. |
Interesting train of thought. Okay, that would make sense in my case. So just a matter until someone wires some satoshis 😂 LOL |
In the interest of science... |
Yeah to science! Thanks for that! Here's what happened: QR code/address within app changed. I swiped to see the Widget, the old QR was shown for a second then replaced with the new one. - Same 'broken' layout though, and I did remove the widget, and pressed show less/more a couple of times. Should I reimburse you? |
Darn... so strange that I can't reproduce it on the same device and OS version. No you can keep the buck... don't spend it all in one place. |
I just took a quick look into the repo - Some weird constrains behaving mysteriously upon messing with self.preferredContentSize - I will try to tinker with it a little over the weekend. The minified widget has some room for improvement, right? |
yes, I quickly reworked it to get something that didn't look like ass with the new iOS 10 widgets, and maybe didn't succeed. |
If you're feeling ambitious and want to re-do it in swift 3, then it would likely make it into the swift 3 app rewrite we're doing. |
@voisine Where can we help contribute to the Swift version? |
Is this the intended behavior/design for the widget? See screenshot below:
Running on iPhone7/10.1 breadwallet v0.6.4-46
The text was updated successfully, but these errors were encountered: