Skip to content

Dragging Rewards aCAPTCHA prompt grows to fill up screen #18891

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

Closed
stephendonner opened this issue Oct 20, 2021 · 4 comments · Fixed by brave/brave-core#10626
Closed

Dragging Rewards aCAPTCHA prompt grows to fill up screen #18891

stephendonner opened this issue Oct 20, 2021 · 4 comments · Fixed by brave/brave-core#10626

Comments

@stephendonner
Copy link

stephendonner commented Oct 20, 2021

Description

Dragging Rewards aCAPTCHA prompt grows to fill up screen

Steps to Reproduce

  1. new profile
  2. launch Brave
  3. enable Rewards
  4. schedule an aCAPTCHA
  5. when it appears, drag it and "snap" it to a screen corner (bottom, top, left, right)
  6. now, try to move the notification by dragging it

Actual result:

The aCAPTCHA grows to fill up the screen

acaptcha

Expected result:

Clicking to drag should just drag, not arbitrarily resize

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.33.19 Chromium: 95.0.4638.54 (Official Build) nightly (64-bit)
Revision d31a821ec901f68d0d34ccdbaea45b4c86ce543e-refs/branch-heads/4638@{#871}
OS Windows 10 Version 20H2 (Build 19042.1288)

Additional info:

display-settings

@stephendonner
Copy link
Author

Verified PASSED using

Brave 1.33.29 Chromium: 95.0.4638.54 (Official Build) nightly (64-bit)
Revision d31a821ec901f68d0d34ccdbaea45b4c86ce543e-refs/branch-heads/4638@{#871}
OS Windows 10 Version 20H2 (Build 19042.1288)

Confirmed I was able to reposition the modal dialog after snapping it to the cardinal screen positions, without it growing in size.

example example
modal-fixed-1 modal-fixed-2

@LaurenWags
Copy link
Member

@stephendonner did this reproduce on macOS for you? if not, we may want to add OS/Windows label.

@stephendonner
Copy link
Author

@stephendonner did this reproduce on macOS for you? if not, we may want to add OS/Windows label.

No, I don't think so.

@LaurenWags
Copy link
Member

updated to release-notes/exclude per discussion with @rebron

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment