Skip to content
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

Identify the taken amount as a maker of an order with an amount range on small screens is clumsy #181

Closed
alaznem opened this issue Jul 2, 2022 · 4 comments
Labels
enhancement 🆙 New feature or request

Comments

@alaznem
Copy link

alaznem commented Jul 2, 2022

Describe the bug
Switching from the chat tab to contract tab to identify how much fiat the buyer should send is on small screens not possible, because the tabs on top move out of the visible canvas on small screens.

Knowing how much fiat to receive is crucial for the maker to evaluate before confirming fiat received.

To Reproduce
make or take an order and resize the browser window to a size of a mid/low range smartphone, when arriving at the chat step (5/5).

Expected behavior

  1. have access to all navigation elements, no matter what vertical display size is being used.
  2. Display the amount to receive next to the currency on top of the chat window in the text paragraph.

Additional context
To find out what the maker will receive is possible at one point in the workflow, while using small screens: when paying the lightning invoice for the trade amount and reading the description in the invoice.

@alaznem alaznem changed the title Identify the taken amount as a maker of an oder with an amount range on small screens is clumsy Identify the taken amount as a maker of an order with an amount range on small screens is clumsy Jul 2, 2022
@Reckless-Satoshi
Copy link
Collaborator

Indeed. The app does not display well at all on small screens.

We pushed a patch that fixed it on iOS devices only (allows scroll). It might be good to allow scroll on Android devices too. At least it would allow users to see the Order details tab.

Adding the fiat amount to the Chat component is also a very good idea.

@alaznem
Copy link
Author

alaznem commented Jul 28, 2022

It might be good to allow scroll on Android devices too. At least it would allow users to see the Order details tab.

What's the next action to get this done? Should I open a separate feature request with the title "Allow scroll on Tor Browser for Android"?

@Reckless-Satoshi
Copy link
Collaborator

"Allow scroll on Tor Browser for Android"?

@ozoono I believe you enabled the vertical scrolling on iOS. Do you think it will help improving the UX if we enable it in any device instead? I have not yet seen the scrolling in action.

@Reckless-Satoshi Reckless-Satoshi added the enhancement 🆙 New feature or request label Jul 29, 2022
@alaznem
Copy link
Author

alaznem commented Jul 29, 2022

TL;DR: To keep in mind: How to handle multiple scroll bars on the same page, currently only the case within the chat view. There is a scroll bar for the chat messages and a scroll bar for the complete page.

"Allow scroll on Tor Browser for Android"?

I have not yet seen the scrolling in action.

There are some proxy enabled browser apps on android which let the user scroll, but they are not the vanilla tor browser. On these apps scrolling on android is fine with the current robosats version, as long as one doesn't want to scroll inside of the chat window. Looks like multiple scroll bars on one page doesn't work well with these apps.

I suggest one should test enabling scroll bar on android against the tor browser.

❓ Follow up: Is android Tor browser able to inject html/css locally? If yes, I could give this test a try.

@alaznem alaznem closed this as completed Jul 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 🆙 New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants