0143b63c74
- Instead of 2 ways to display ads (DOM injection + React method) and having both of them clash, just do it the predictable React way.
- Augment the existing React version to support tile layout + ability to place in last visible slot.
- Consolidate styling code to scss ... DOM manipulations were making it even harder to maintain.
- Removed the need to check for ad-blockers for now. It was being executed every time an ad is displayed, and now that we are displaying ads in more places, the gains doesn't justify the performance loss. Also, it wasn't being done for Recommended ads anyway, so the inconsistency probably means it's not needed in the first place.
Other known issues fixed:
- double ad injection when changing language via nag.
- additional "total-blocking-time" due to ads at startup removed.
- fixed ads not appearing in mobile homepage until navigated away and back to homepage.
- enable ads in channel page.
- support for both List and Tile layout.
|
||
|---|---|---|
| .. | ||
| ads | ||
| browserNotificationBanner | ||
| browserNotificationHints | ||
| browserNotificationSettings | ||
| fileViewerEmbeddedEnded | ||
| openInAppLink | ||
| youtubeReferralWelcome | ||
| footer.jsx | ||
| meme.jsx | ||
| nag-data-collection.jsx | ||
| nag-degraded-performance.jsx | ||
| nag-no-user.jsx | ||
| nag-sunset.jsx | ||