Merge pull request #12964 from ryanml/empty-yahoo-ad-container-fix

Fixes issue #12918, hides empty ad container on www.yahoo.com
brave · Feb 12, 2018 · 28c7db0 · 28c7db0
2 parents f8f0eb7 + 9677a6b
commit 28c7db0
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/app/extensions.js b/app/extensions.js
@@ -53,7 +53,12 @@ let generateBraveManifest = () => {
       {
         run_at: 'document_start',
         all_frames: true,
-        matches: ['https://www.washingtonpost.com/*', 'https://www.youtube.com/*', 'https://coinmarketcap.com/*'],
+        matches: [
+          'https://www.washingtonpost.com/*',
+          'https://www.youtube.com/*',
+          'https://coinmarketcap.com/*',
+          'https://www.yahoo.com/*'
+        ],
         css: [
           'content/styles/removeEmptyElements.css'
         ]

diff --git a/app/extensions/brave/content/styles/removeEmptyElements.css b/app/extensions/brave/content/styles/removeEmptyElements.css
@@ -8,7 +8,8 @@
   .ytp-ad-progress-list,
   .ad-container,
   #video-masthead,
-  #watch-channel-brand-div {
+  #watch-channel-brand-div,
+  #Billboard-ad {
     display: none !important;
 }