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

On Twitter is repeated twice on BR panel #4554

Closed
srirambv opened this issue May 23, 2019 · 9 comments · Fixed by brave/brave-core#2516
Closed

On Twitter is repeated twice on BR panel #4554

srirambv opened this issue May 23, 2019 · 9 comments · Fixed by brave/brave-core#2516

Comments

@srirambv
Copy link
Contributor

Description

On Twitter is repeated twice on BR panel

Steps to Reproduce

  1. Enable rewards on 0.67.51
  2. Upgrade to 0.67.55
  3. Visit a verified Twitter publisher
  4. on Twitter is repeated twice in the panel

Actual result:

Screenshot from 2019-05-23 12-20-28

Expected result:

Should not duplicate the platform name on the panel

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 0.67.55 Chromium: 75.0.3770.38 (Official Build)nightly (64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Linux

Version/Channel Information:

  • Can you reproduce this issue with the current release? N/A
  • Can you reproduce this issue with the beta channel? N/A
  • Can you reproduce this issue with the dev channel? N/A
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

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

Miscellaneous Information:

cc: @NejcZdovc

@LaurenWags
Copy link
Member

LaurenWags commented May 23, 2019

On a clean profile, the @ handle is in parenthesis, but no double 'on Twitter' observed:
Screen Shot 2019-05-23 at 12 42 37 PM

Brave 0.67.55 Chromium: 75.0.3770.38 (Official Build) nightly(64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Mac OS X

@NejcZdovc
Copy link
Contributor

I can't reproduce this one the latest nightly Version 0.67.62 Chromium: 75.0.3770.38 (Official Build) nightly (64-bit)

image

@NejcZdovc
Copy link
Contributor

@srirambv @LaurenWags can you please try again on a clean profile?

@NejcZdovc NejcZdovc added the needs-investigation A bug not 100% confirmed/fixed label May 27, 2019
@srirambv
Copy link
Contributor Author

I can reproduce on a clean profile of last nightly

Brave 0.67.62 Chromium: 75.0.3770.38 (Official Build)nightly (64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}

image
image

Two different behaviour on different profiles. Also notice how the image is shrinked when Twitter handle is included with the publisher name

@NejcZdovc
Copy link
Contributor

NejcZdovc commented May 27, 2019

@srirambv which OS is this? I am trying to find what is the difference

@srirambv
Copy link
Contributor Author

@NejcZdovc
image

@LaurenWags
Copy link
Member

@NejcZdovc my comment above reproduces on a clean profile with

Brave 0.67.62 Chromium: 75.0.3770.38 (Official Build) nightly(64-bit)
Revision 3860105745f2b12537da9e9f048f14c3f52ba970-refs/branch-heads/3770@{#618}
OS Mac OS X

Screen Shot 2019-05-28 at 9 41 22 AM

@LaurenWags
Copy link
Member

Had a call with @NejcZdovc to discuss this one - he noticed that if handle starts with a capital letter, we're not parsing correctly. Here are some examples:
https://twitter.com/kamiljozwiak
Screen Shot 2019-05-28 at 9 41 22 AM

https://twitter.com/bravesampson
Screen Shot 2019-05-28 at 10 39 30 AM

And when the handle starts with lower case letter:
https://twitter.com/bvsriram
Screen Shot 2019-05-28 at 10 39 45 AM

https://twitter.com/emerick
Screen Shot 2019-05-28 at 10 39 51 AM

https://twitter.com/bravelaurenwags
Screen Shot 2019-05-28 at 10 39 38 AM

@NejcZdovc NejcZdovc self-assigned this May 28, 2019
@NejcZdovc NejcZdovc removed the needs-investigation A bug not 100% confirmed/fixed label May 28, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue May 28, 2019
@NejcZdovc NejcZdovc added this to the 0.67.x - Nightly milestone May 28, 2019
@LaurenWags
Copy link
Member

LaurenWags commented Jul 10, 2019

Verified passed with

Brave 0.67.106 Chromium: 75.0.3770.100 (Official Build) beta(64-bit)
Revision cd0b15c8b6a4e70c44e27f35c37a4029bad3e3b0-refs/branch-heads/3770@{#1033}
OS Mac OS X

Verification passed on

Brave 0.67.106 Chromium: 75.0.3770.100 (Official Build) beta (64-bit)
Revision cd0b15c8b6a4e70c44e27f35c37a4029bad3e3b0-refs/branch-heads/3770@{#1033}
OS Windows 10 OS Version 1803 (Build 17134.523)

image

image

Verification passed on

Brave 0.67.110 Chromium: 75.0.3770.100 (Official Build) beta (64-bit)
Revision cd0b15c8b6a4e70c44e27f35c37a4029bad3e3b0-refs/branch-heads/3770@{#1033}
OS Linux

image

image

image

image

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