Brave-browser: ad notifications no longer have a View button

Created on 4 Dec 2019  路  6Comments  路  Source: brave/brave-browser

Description

On C79 versions the ad notifications no longer have a View button, just one large Close button.

For reference, I'm using High Sierra on macOS.

Steps to Reproduce

  1. Enable Rewards
  2. Browse until an ad is served.
  3. 3.

Actual result:

Screen Shot 2019-12-04 at 4 21 00 PM

Expected result:

Screen Shot 2019-12-04 at 4 26 31 PM

Reproduces how often:

easily

Brave version (brave://version info)

1.1.18

Version/Channel Information:

  • Can you reproduce this issue with the current release? not on 1.0.1
  • Can you reproduce this issue with the beta channel? unsure
  • Can you reproduce this issue with the dev channel? unsure
  • Can you reproduce this issue with the nightly channel? unsure

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 @bsclifton @mkarolin @rebron @brave/legacy_qa

closewontfix

Most helpful comment

Confirming the above

All 6 comments

@LaurenWags Could be specific to macOS and Linux (not sure about Linux) - from the beginning, there is no View option in ads notification in Windows OS.

Adding 1.1.x milestone as this is likely a C79 regression we may want resolved before releasing.

This actually improves consistency across desktop operating systems.

Linux (Fedora/Gnome): No view button.
Win10: No view button.

Closing as wontfix after sharing with team and getting verbal approval from @jsecretan

Confirming the above

Not reproducible on Linux. Adding OS/macOS label.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

bbondy picture bbondy  路  3Comments

Sondro picture Sondro  路  3Comments

jonathansampson picture jonathansampson  路  3Comments

kjozwiak picture kjozwiak  路  3Comments

lansana picture lansana  路  3Comments