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

twitter tip button doesn't show on selected tweet when viewing thread #6762

Closed
LaurenWags opened this issue Nov 4, 2019 · 5 comments · Fixed by brave/brave-core#6671
Closed

Comments

@LaurenWags
Copy link
Member

Description

When you click on a tweet to view it/the associated thread, the selected tweet does not have the Tip button, but other tweets in the thread do.

Note - this works with the old twitter layout (when you're not logged in) but does not work with the newer twitter layout.

Steps to Reproduce

  1. Clean profile
  2. Enable Rewards
  3. Navigate to twitter.com and login (logging in is important so you get the new layout)
  4. Navigate to twitter.com/brave - note tweets have the Tip button.
  5. Click on a tweet
  6. Note tweet does not contain the tip button, but replies/tweets in the thread do.
  7. Click on one of the replies
  8. Note now this tweet does not contain the tip button, but other tweets in the thread do.

Actual result:

Step 6:
Screen Shot 2019-11-04 at 3 17 51 PM

Step 8:
Screen Shot 2019-11-04 at 3 35 55 PM

Expected result:

All tweets would have the tip button.

Reproduces how often:

easily

Brave version (brave://version info)

Brave 0.71.111 Chromium: 78.0.3904.87 (Official Build) (64-bit)
Revision 20c21f4010010f32462ea8e1d6af30cef66d48c8-refs/branch-heads/3904@{#840}
OS macOS Version 10.13.6 (Build 17G5019)

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the dev channel? yes
  • 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:

This is an existing issue in release version.
@kjozwiak also encountered this while testing

@NejcZdovc NejcZdovc added the priority/P4 Planned work. We expect to get to it "soon". label Nov 18, 2019
@NejcZdovc
Copy link
Contributor

@LaurenWags is this still a problem on the latest nightly?

@LaurenWags
Copy link
Member Author

yes, reproduces with

Brave 1.9.15 Chromium: 81.0.4044.92 (Official Build) nightly (64-bit)
Revision 32921c79b6f01a0fb2deef0e1d45b42f96581051-refs/branch-heads/4044@{#883}
OS macOS Version 10.14.6 (Build 18G3020)

@emerick
Copy link
Contributor

emerick commented Apr 17, 2020

It appears that this no longer works:

let tweets = document.querySelectorAll('[data-testid="tweet"], [data-testid="tweetDetail"]')

It used to find tweets in the regular stream (data-testid="tweet") and tweets in the detail view (data-testid="tweetDetail"). We would need to adjust the Twitter content script to account for the new HTML, possibly using the article role.

@jonathansampson
Copy link
Contributor

Recently received a +1 on Twitter regarding this: https://twitter.com/brianoflondon/status/1261577925943611392

@LaurenWags
Copy link
Member Author

LaurenWags commented Oct 21, 2020

Verified passed with

Brave | 1.17.45 Chromium: 86.0.4240.99 (Official Build) dev (x86_64)
-- | --
Revision | 002668237e13d38aabd6d11c2d216dd22b736ff2-refs/branch-heads/4240@{#1229}
OS | macOS Version 10.14.6 (Build 18G3020)

Verified STR from description. Confirmed main tweet shows Tip button.

Screen Shot 2020-10-21 at 2 20 50 PM

Confirmed clicking on this button shows the appropriate twitter publisher and selected tweet on the tip banner:
Screen Shot 2020-10-21 at 2 21 57 PM

Encountered and logged #12264


Verification passed on

Brave | 1.18.2 Chromium: 86.0.4240.111 (Official Build) nightly (64-bit)
-- | --
Revision | b8c36128a06ebad76af51591bfec980224db5522-refs/branch-heads/4240@{#1290}
OS | Windows 10 OS Version 1903 (Build 18362.1139)

  • Verified STR from the description. Confirmed main tweet shows Tip button.
    image

  • Confirmed clicking on this button shows the appropriate twitter publisher and selected tweet on the tip banner:
    image

Encountered #12264


Verification passed on

Brave 1.17.50 Chromium: 86.0.4240.111 (Official Build) dev (64-bit)
Revision b8c36128a06ebad76af51591bfec980224db5522-refs/branch-heads/4240@{#1290}
OS Ubuntu 18.04 LTS
  • Verified STR from the description.

  • Confirmed main tweet shows Tip button.
    image

  • Confirmed clicking on this button shows the appropriate twitter publisher and selected tweet on the tip banner:
    image

  • Verified replies have Tip button
    image

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