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

Captcha target not fully shown if the target is below the visible BR panel #2956

Closed
srirambv opened this issue Jan 15, 2019 · 2 comments
Closed

Comments

@srirambv
Copy link
Contributor

Description

Captcha target not fully shown if the target is below the visible BR panel

Steps to Reproduce

  1. Set browser zoom to 125%
  2. Enable rewards on BR Panel and claim grant
  3. Ensure one of the target is below the standard view of the BR Panel
  4. Scroll down the panel won't see the full target (See screentshot)

Actual result:

screenshot from 2019-01-15 10-26-35

Expected result:

Browser zoom level shouldn't affect the CATPCHA shown on BR panel

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 0.60.13 Chromium: 72.0.3626.53 (Official Build) dev(64-bit)
Revision 98434e6cd182d68ce396daa92e9c6310422e6763-refs/branch-heads/3626@{#620}
OS Linux

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    Yes on Dev

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
  • Is the issue reproducible on the latest version of Chrome?

Additional Information

@brave/legacy_qa to verify on other platforms, @NejcZdovc @ryanml @rebron

@btlechowski
Copy link

Reproduced on Windows 7 by changing global page zoom setting in chrome://settings/?search=page+zoom

image

Brave 0.60.13 Chromium: 72.0.3626.53 (Official Build) dev (64-bit)
Revision 98434e6cd182d68ce396daa92e9c6310422e6763-refs/branch-heads/3626@{#620}
OS Windows

@NejcZdovc NejcZdovc added the priority/P5 Not scheduled. Don't anticipate work on this any time soon. label Jan 16, 2019
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@NejcZdovc
Copy link
Contributor

closing this one as it's not a problem anymore

@NejcZdovc NejcZdovc added closed/not-actionable and removed QA/Test-Plan-Specified QA/Yes bug priority/P5 Not scheduled. Don't anticipate work on this any time soon. release-notes/include labels Nov 18, 2019
@NejcZdovc NejcZdovc added this to the Dupe / Invalid / Not actionable milestone Mar 5, 2020
@bbondy bbondy removed this from the Dupe / Invalid / Not actionable milestone May 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants