Brave-browser: incorrect tip banner text for tipping an image/video

Created on 23 May 2019  路  5Comments  路  Source: brave/brave-browser

Description

Per spec, when tipping a tweet without any extractable text, you should see something like the image below, however, that's not what's occurring.
Screen Shot 2019-05-23 at 4 54 02 PM

Steps to Reproduce

  1. Locate a tweet with just an image (can use this one if needed, should work for both staging and prod: https://twitter.com/bravelaurenwags/status/1131664248856354816)
  2. Click on Tip button

Actual result:

Screen Shot 2019-05-23 at 4 52 44 PM

Expected result:

Expecting to see text that said 'Tip @bravelaurenwags for their tweet!' instead of empty tweet box.

Reproduces how often:

easily

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 | Mac OS X

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 @brave/legacy_qa @mandar-brave @alexwykoff @jenn-rhim

https://twitter.com/bravelaurenwags/status/1131664248856354816 - there's also a reply tweet on this thread which contains a gif that acts the same as the image.

QA Pass-Linux QA Pass-Win64 QA Pass-macOS QTest-Plan-Specified QYes bug featurrewards release-noteexclude

Most helpful comment

This is what we it would look like now:

image

All 5 comments

This is what we it would look like now:

image

@NejcZdovc to confirm, even if the twitter user you're trying to tip has a custom banner, in this case you still want it to show the default text? Here's what I'm doing:
(using staging)

  1. Navigate to https://twitter.com/bravelaurenwags/status/1131664248856354816
  2. Click on in-line tip button
  3. See this banner - note, it shows default text in the body (text from #4616)
    Screen Shot 2019-07-10 at 6 14 00 PM
  4. Close banner, navigate to just twitter.com/bravelaurenwags
  5. Click on panel, click on Send a Tip
  6. Here, the entire custom banner is shown, even custom text in the body of the banner:
    Screen Shot 2019-07-10 at 6 14 17 PM

Just want to confirm we are expecting different body text in the two places when tipping an image/gif.

title is different yes if you tip from inline tip or from panel tip

ok, thanks!

Verification passed on

Brave | 0.67.108 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)

In-line tip:
image

Tip from BR panel:

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 | Ubuntu 18.04 LTS

image

image

Was this page helpful?
0 / 5 - 0 ratings