Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

QR Code Popup from Add funds dialog scrolls independently from its parent #4043

Closed
bradleyrichter opened this issue Sep 16, 2016 · 12 comments
Closed

Comments

@bradleyrichter
Copy link
Contributor

It is one of the cutest bugs I have ever seen. But a bug, none the less.

(found on a Dell XPS with touchpad scrolling)

Brave 0.12.1 release version

@luixxiul
Copy link
Contributor

Is this intended?

ledger-popup

@luixxiul luixxiul reopened this Sep 25, 2016
@luixxiul luixxiul added needs-info Another team member needs information from the PR/issue opener. and removed QA/checked-Win32 labels Sep 25, 2016
@bbondy bbondy modified the milestones: 0.12.3dev, 0.12.2dev Sep 25, 2016
@bbondy
Copy link
Member

bbondy commented Sep 25, 2016

Assuming the current behavior is not horrible I'm moving it to 0.12.3, let me know if anyone disagrees.

@luixxiul
Copy link
Contributor

On a small window with a vertical scroll bar it still happens.

ledger-popup

wontfix?

@luixxiul luixxiul added the needs-info Another team member needs information from the PR/issue opener. label Sep 28, 2016
@bradleyrichter
Copy link
Contributor Author

needs more fixing

@jkup Any ideas? If it's only at this small size, we can push this out to 0.12.4.

@aekeus
Copy link
Member

aekeus commented Sep 28, 2016

Independent scroll on macOS

qr-scroll

@jkup
Copy link
Contributor

jkup commented Sep 29, 2016

I think I'm just not understanding what the desired behavior is here. Is the bug the fact that the background of the page like monthly budget, balance and statements is scrolling when you try to scroll the modal?

Because the modal itself should scroll right? So you can get to the bottom or top of the content?

@srirambv
Copy link
Collaborator

@jkup When the window size is reduced there is horizontal shift in the position of the QA Code popup after that it starts moving independently when the main window has vertical scroll bars.

qrcode

@bradleyrichter
Copy link
Contributor Author

@jkup

I think the best approach may be to restrict scrolling to the main parent window and child objects are relative to each parent.

If the child doesn't fit, then the parent scrolls and the child moved with it. (As if the entire viewable area was a big screenshot.)

What we have now feels more like nested iframes?

On Sep 28, 2016, at 10:11 PM, Jon Kuperman notifications@github.com wrote:

I think I'm just not understanding what the desired behavior is here. Is the bug the fact that the background of the page like monthly budget, balance and statements is scrolling when you try to scroll the modal?

Because the modal itself should scroll right? So you can get to the bottom or top of the content?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@jkup
Copy link
Contributor

jkup commented Sep 30, 2016

@bradleyrichter ok I'll try that! Modals on top of modals present a lot of challenges :)

@bradleyrichter
Copy link
Contributor Author

We may be able to use modal replacement instead of opening another. Other browsers have the same issue. I'll try to find an example.

On Sep 29, 2016, at 11:03 PM, Jon Kuperman notifications@github.com wrote:

@bradleyrichter ok I'll try that! Modals on top of modals present a lot of challenges :)


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@jkup
Copy link
Contributor

jkup commented Sep 30, 2016

@bradleyrichter ok I'm happy to go either way, also @srirambv I'm not seeing that horizontal shift on my mac, what OS are you on there?

@srirambv
Copy link
Collaborator

@jkup Its on a Windows 10 machine

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants