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

Broken image in rewards panel #2259

Closed
btlechowski opened this issue Nov 27, 2018 · 9 comments · Fixed by brave/brave-core#970
Closed

Broken image in rewards panel #2259

btlechowski opened this issue Nov 27, 2018 · 9 comments · Fixed by brave/brave-core#970

Comments

@btlechowski
Copy link

btlechowski commented Nov 27, 2018

Steps to Reproduce

  1. Clean install
  2. Enable rewards through rewards panel

Actual result:

  • Token grant image is broken
  • weird text: MISSING: noActivity
  • rewards height is too big
    image

Expected result:

Rewards panel in Beta 0.57.7
image

Reproduces how often:

always

Brave version (brave://version info)

Brave 0.58.10 Chromium: 71.0.3578.53 (Official Build) dev (32-bit)
Revision 599fb72e740eab9d10609f1ed28a2ad669f82c1a-refs/branch-heads/3578@{#669}
OS Windows 7

Reproduced on MacOS, Brave 0.58.10 by @LaurenWags

Reproducible on current release:

Does not reproduce on Release(0.56.15) nor Beta(0.57.7) Channels

Not reproducible on

Brave 0.58.9 Chromium: 71.0.3578.53 (Official Build) dev (64-bit)
Revision 599fb72e740eab9d10609f1ed28a2ad669f82c1a-refs/branch-heads/3578@{#669}
OS Windows

This is a recent regression

cc @kjozwiak @srirambv @GeetaSarvadnya @NejcZdovc @ryanml @emerick

@ryanml
Copy link
Contributor

ryanml commented Nov 27, 2018

@btlechowski the first two problem (missing image and string) will for sure be fixed.

As for the new panel height, this was a design update made by @jenn-rhim in myself so there is not a flickering of height as you move between the panel and summary.

@kjozwiak
Copy link
Member

kjozwiak commented Nov 27, 2018

@btlechowski the first two problem (missing image and string) will for sure be fixed.

Thanks @ryanml, can we get this fixed in dev before 0.58.x moves into BETA next week? Couldn't reproduce with the latest beta so definitely a new regression:

Brave 0.57.7 Chromium: 71.0.3578.53 (Official Build) beta(64-bit)
Revision 599fb72e740eab9d10609f1ed28a2ad669f82c1a-refs/branch-heads/3578@{#669}
OS Mac OS X

@ryanml
Copy link
Contributor

ryanml commented Nov 27, 2018

@kjozwiak - definitely, I should be able to have this out EOD actually.

@petemill
Copy link
Member

This is two separate issues, we should split them up

@petemill
Copy link
Member

Issue for missing string - #2261

@petemill petemill changed the title Broken layout in rewards panel Broken image in rewards panel Nov 27, 2018
@LaurenWags
Copy link
Member

@ryanml @jenn-rhim re: #2259 (comment) and panel height, this is what you see when on a site. it looks odd to have the white space at the bottom. can this be adjusted?
screen shot 2018-11-28 at 11 55 19 am

@ryanml
Copy link
Contributor

ryanml commented Nov 28, 2018

@LaurenWags this will be fixed with: brave/brave-core#957

@LaurenWags
Copy link
Member

awesome, thanks @ryanml 😄

@LaurenWags
Copy link
Member

LaurenWags commented Dec 10, 2018

Verified passed with

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta(64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Mac OS X

screen shot 2018-12-10 at 6 18 16 pm

Verification Passed on

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Windows

image

Verification PASSED using Ubuntu 18.0.4 x64 with the following build:

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Linux

screen shot 2018-12-12 at 1 07 56 am

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