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

Brave Rewards panel can have a mismatch for icon and URL #2073

Closed
bbondy opened this issue Nov 9, 2018 · 3 comments · Fixed by brave/brave-core#1030
Closed

Brave Rewards panel can have a mismatch for icon and URL #2073

bbondy opened this issue Nov 9, 2018 · 3 comments · Fixed by brave/brave-core#1030

Comments

@bbondy
Copy link
Member

bbondy commented Nov 9, 2018

Description

Sometimes there's a mismatch for icon and name in the rewards panel.
screen shot 2018-11-09 at 11 22 11 am

Steps to Reproduce

Unsure of the exact steps, but probably something like go to calendar.google.com first, then google.com.

Actual result:

google.com is displayed as the name
chrome://favicon/size/48@1x/https://calendar.google.com/ is displayed as the icon.

Both going to google.com and calendar.google.com displays the same way as per the screenshot above.

Expected result:

It displays consistent results.

Reproduces how often:

Always on my open window.

Brave version (brave://version info)

0.56.10

Reproducible on current release:

Yes,

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? Yes.
  • Is the issue reproducible on the latest version of Chrome? N/a.
@bbondy bbondy self-assigned this Nov 9, 2018
@NejcZdovc NejcZdovc added this to the 1.x Backlog milestone Nov 9, 2018
@bbondy bbondy removed their assignment Nov 9, 2018
@kjozwiak kjozwiak added the QA/Yes label Nov 9, 2018
@NejcZdovc NejcZdovc self-assigned this Nov 19, 2018
@NejcZdovc NejcZdovc added the priority/P4 Planned work. We expect to get to it "soon". label Nov 19, 2018
NejcZdovc added a commit to brave/brave-core that referenced this issue Dec 6, 2018
NejcZdovc added a commit to brave/brave-core that referenced this issue Dec 17, 2018
@NejcZdovc NejcZdovc modified the milestones: 1.x Backlog, 0.60.x - Dev Dec 18, 2018
@LaurenWags
Copy link
Member

@NejcZdovc two questions:

  1. test plan from Fixes subdomain logging for rewards brave-core#1030 works, but I have to refresh brave://rewards to see the favicon for brave.com at the end of the test plan. Is this expected?
  2. stats.brave.com doesn't have a favicon
    screen shot 2019-02-06 at 11 00 46 am
    so when I substitute a different site that does have a favicon, such as technology.slashdot.org, i see the same result (no favicon in Rewards until I visit slashdot.org). Is this expected?

@NejcZdovc
Copy link
Contributor

NejcZdovc commented Feb 6, 2019

@LaurenWags re 1 and 2: yes expected. For 2 we use chromium favicon system and because you didn't visit root domain yet we don't have favicon for it

@LaurenWags
Copy link
Member

LaurenWags commented Feb 6, 2019

Verified passed with

Brave 0.60.19 Chromium: 72.0.3626.81 (Official Build) beta(64-bit)
Revision ac8b982e05014492d1bd7d317628a4f22a97ffa0-refs/branch-heads/3626@{#796}
OS Mac OS X

Verification passed on

Brave 0.60.25 Chromium: 72.0.3626.96 (Official Build) beta (64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Windows 10

Verification passed on

Brave 0.60.26 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Linux

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

Successfully merging a pull request may close this issue.

6 participants