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 ads appear underneath MacOS dock/menu bar and Windows taskbar #13065

Closed
stshank opened this issue Dec 7, 2020 · 6 comments · Fixed by brave/brave-core#7363
Closed

Brave ads appear underneath MacOS dock/menu bar and Windows taskbar #13065

stshank opened this issue Dec 7, 2020 · 6 comments · Fixed by brave/brave-core#7363

Comments

@stshank
Copy link

stshank commented Dec 7, 2020

Description

Ads appear underneath the dock on MacOS Catalina and Big Sur when the dock is positioned on the right edge of the screen.
Brave ads appear to low if the menubar is hidden.

Ads appear underneath the taskbar on Windows when the taskbar is positioned on the right edge of the screen.

Steps to Reproduce

  1. Move dock to right edge of screen
  2. Wait for ad to appear

Actual result:

image
Ad appears underneath the dock icons.

Expected result:

Ad should appear on top of dock (or maybe flush with right edge of browser window?)

Reproduces how often:

Always

Brave version (brave://version info)

Brave 1.19.48 Chromium: 87.0.4280.88 (Official Build) dev (x86_64)
Revision 89e2380a3e36c3464b5dd1302349b1382549290d-refs/branch-heads/4280@{#1761}
OS macOS Version 11.0.1 (Build 20B29)
JavaScript V8 8.7.220.29
Flash (Disabled)
User Agent Mozilla/5.0 (Macintosh; Intel Mac OS X 11_0_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Safari/537.36
Command Line /Applications/Brave Browser Dev.app/Contents/MacOS/Brave Browser Dev --enable-dom-distiller --disable-domain-reliability --no-pings --extension-content-verification=enforce_strict --extensions-install-verification=enforce --origin-trial-public-key=bYUKPJoPnCxeNvu72j4EmPuK7tr1PAC7SHh8ld9Mw3E=,fMS4mpO6buLQ/QMd+zJmxzty/VQ6B1EUZqoCU04zoRU= --sync-url=https://sync-v2.brave.com/v2 --lso-url=https://no-thanks.invalid --variations-server-url=https://variations.brave.com/seed --enable-features=AutoupgradeMixedContent,LegacyTLSEnforced,WebUIDarkMode,ReducedReferrerGranularity,PasswordImport,PrefetchPrivacyChanges --disable-features=PrivacySettingsRedesign,AutofillServerCommunication,AutofillEnableAccountWalletStorage,TextFragmentAnchor,SmsReceiver,NotificationTriggers,PasswordCheck,NetworkTimeServiceQuerying,TabHoverCards,SignedExchangeSubresourcePrefetch,SafeBrowsingEnhancedProtection,AllowPopupsDuringPageUnload,VideoPlaybackQuality,IdleDetection --flag-switches-begin --flag-switches-end
-- --
@tmancey tmancey added priority/P3 The next thing for us to work on. It'll ride the trains. release-notes/include QA/Yes labels Dec 7, 2020
@tmancey tmancey changed the title Brave ads appear underneath right-side MacOS dock Brave ads appear underneath right-side MacOS dock/menu bar and Windows taskbar/system tray Dec 7, 2020
@tmancey tmancey changed the title Brave ads appear underneath right-side MacOS dock/menu bar and Windows taskbar/system tray Brave ads appear underneath right-side MacOS dock/menu bar and Windows taskbar Dec 7, 2020
@tmancey tmancey changed the title Brave ads appear underneath right-side MacOS dock/menu bar and Windows taskbar Brave ads appear underneath MacOS dock/menu bar and Windows taskbar Mar 7, 2021
@tmancey tmancey added this to the 1.23.x - Nightly milestone Mar 11, 2021
@btlechowski
Copy link

Based on the test plan from brave/brave-core#7363, added QA/Test-All-Platforms

@stephendonner
Copy link

Hi @stshank! If you wouldn't mind, could you let us know 1) screen resolution settings (default/scaled, etc.) 2) # of items in your dock, when you saw this issue? And, anything else which might be pertinent in reproducing this issue? I've tried on Big Sur 11.2.3 with various builds, but so far haven't triggered your specific scenario, and I'd like to be able to do so before confidently verifying the fix; thanks!

@tmancey
Copy link
Contributor

tmancey commented Mar 24, 2021

@stephendonner If you could send a calendar event, or let me know when online and I can demonstrate the issue. The issue was fixed in 1.23.33 and will only occur in nightly as custom ad notifications are glued to nightly builds at this time. Thanks, Terry

@stshank
Copy link
Author

stshank commented Mar 25, 2021 via email

@stephendonner
Copy link

@tmancey @stshank thanks, both of you; easily reproduced this bug using nightly version 1.23.33 on macOS 11.2.3

Screen Shot 2021-03-26 at 9 06 48 AM

@stephendonner stephendonner added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Mar 26, 2021
@stephendonner
Copy link

stephendonner commented Mar 26, 2021

Verified FIXED on macOS Big Sur (11.2.3) using beta build

Brave 1.23.53 Chromium: 89.0.4389.105 (Official Build) beta (x86_64)
Revision 14f44e21a9d539cd49c72468a29bfca4fa43f710-refs/branch-heads/4389_90@{#7}
OS macOS Version 11.2.3 (Build 20D91)

Steps:

  1. clean profile
  2. launched Brave using --args --enable-logging=stderr --vmodule="*/bat-native-ledger/*"=6,"*/brave_rewards/*"=6,"*/bat-native-ads/*"=6,"*/bat-native-confirmations/*"=6,"*/brave_ads/*"=6,"*/brave_user_model/*"=6 --brave-ads-staging --rewards=staging=true
  3. set Position on Screen for the dock to {Bottom (default), Left, and Right}
  4. opened a few new-tab pages
  5. clicked on Start using Rewards
  6. waited a bit
  7. confirmed I received a Brave ad notification:
  • at the appropriate location and position for each corresponding dock placement
  • ads were not clipped behind any of the docks
Bottom Left Right
Screen Shot 2021-03-26 at 9 52 03 AM Screen Shot 2021-03-26 at 9 47 15 AM Screen Shot 2021-03-26 at 10 43 03 AM

Verification passed on


Brave | 1.23.56 Chromium: 89.0.4389.105 (Official Build) dev (64-bit)
-- | --
Revision | 14f44e21a9d539cd49c72468a29bfca4fa43f710-refs/branch-heads/4389_90@{#7}
OS | Windows 10 OS Version 2004 (Build 19041.867)


  • Ensured that Ad notification isn't overlapped on the taskbar

Taskbar is set to Bottom
image

Taskbar is set to Top
image

Taskbar is set to Left
image

Taskbar is set to Right
image


Verification PASSED on Ubuntu 20.04 x64 using the following build:

Brave | 1.23.68 Chromium: 90.0.4430.51 (Official Build) (64-bit)
--- | ---
Revision | 32e5fa33a31641bded70a90e60121060691e7125-refs/branch-heads/4430@{#927}
OS | Linux

This doesn't really affect Linux as ad notifications are displayed in the middle at the top of the screen. You can only place the dock on either the right/left or bottom of the screen. Either way, switched the dock into the various positions and ensured that the ad notifications are appearing correctly under Linux.

Right Example Left Example Bottom Example
Screen Shot 2021-04-07 at 7 12 59 PM Screen Shot 2021-04-07 at 7 15 18 PM Screen Shot 2021-04-07 at 7 17 32 PM

@stephendonner stephendonner added QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Mar 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment