more DD53

1 files changed, 74 insertions, 0 deletions

diff --git a/design-documents/053-wallet-ui.rst b/design-documents/053-wallet-ui.rst
index 9d11a78c..6b7abdc6 100644
--- a/design-documents/053-wallet-ui.rst
+++ b/design-documents/053-wallet-ui.rst
@@ -823,6 +823,27 @@ Actions
 .. attention::
   User should be able to play with the amount, not possible in the current design
 
+Issues
+^^^^^^
+
+* WebEx(screenshot): need new section in this document for 'ctx-manage-account'.
+* WebEx(minor): should probably separate out target account selection and amount entry, as done on iOS
+* WebEx(text): "amount" is a bad label, "brut amount" might be good, then use "net deposit"
+* WebEx(text): might be good to have "net deposit" amount also editable, not just computed (and then update "brut amount").
+* Android(screenshot): lacks screenshots for dialogs where we enter/edit accounts and amounts
+* Android(minor): if we do it iOS/WebEx style, we can probalby skip the 'pure' confirmation dialog and have the one where the brut/net amounts are entered be the final one in the sequence before the transaction happens
+* all(screenshot): we need a new dialog showing the transaction in-flight/done as reachable via transaction history
+
+
+Adoption
+^^^^^^^^
+
+* iOS: I think it makes sense to split this section up into account selection / management and amount entry & confirmation; we should only have one major screen per section! Please split other UIs similarly and let's introduce a new section here.
+
+
+
+
+
 Screenshots
 ^^^^^^^^^^^
 
@@ -876,6 +897,17 @@ Actions
   Is the invoice creation always free of charge or does the exchange have a mechanism
   to impose a fee to pay on creation?
 
+Issues
+^^^^^^
+
+* WebEx(text): 20 days is odd, Android has 30 days instead
+* Android(text): Webex uses "1 Week" instead of "7 days", let's use "week".
+* iOS(text): 3 min/ 1h are inconsistent; other wallets have 1 day, 7 days, 30 days. We should be consistent.
+
+
+Adoption
+^^^^^^^^
+
 Screenshots
 ^^^^^^^^^^^
 
@@ -943,6 +975,19 @@ Actions
 * **confirm operation**: on success will be redirected to the ``transaction-details`` screen where the detail of the current transaction will be displayed
 * **cancel**: user will be redirected to ``balance``
 
+Issues
+^^^^^^
+
+* WebEx(minor): forces user to scroll to the bottom. Android nicely shows the accept button always; that seems nicer;
+* WebEx(text): has text about "Digital cash withdrawal" above. Would be more consistent if we also *only* showed the Terms of service.
+* Android(text): uses "Exchange", should say "PSP's Terms of Service"
+* iOS(screenshot): lacks actual ToS, not ideal to compare!
+
+
+Adoption
+^^^^^^^^
+
+
 Screenshots
 ^^^^^^^^^^^
 
@@ -983,6 +1028,20 @@ Actions
 * **review and confirm ToS**: if the current selected exchange has a version of ToS that the user didn't yet accepted, use the :ref:`cta-accept-tos-ref` screen
 * **cancel**: user will be redirected to ``balance``
 
+Issues
+^^^^^^
+
+* All(screenshot): we have no screenshots!
+
+
+Adoption
+^^^^^^^^
+
+
+Screenshots
+^^^^^^^^^^^
+
+
 .. _operation-table-details-ref:
 
 operation-table-details
@@ -1035,6 +1094,15 @@ Actions
 * **accept tos**: will mark this version as accepted in wallet core and redirect the user to the screen from where it was invoked
 * **save/print tos**: will save the ToS outside of the wallet
 
+Issues
+^^^^^^
+
+
+
+Adoption
+^^^^^^^^
+
+
 Screenshots
 ^^^^^^^^^^^
 
@@ -1068,6 +1136,9 @@ Actions
 Issues
 ^^^^^^
 
+Adoption
+^^^^^^^^
+
 Screenshots
 ^^^^^^^^^^^
 
@@ -1089,6 +1160,9 @@ Actions
 Issues
 ^^^^^^
 
+Adoption
+^^^^^^^^
+
 Screenshots
 ^^^^^^^^^^^