pre roll ads fall out #6477

Closed
opened 2021-07-14 18:05:17 +02:00 by tzarebczan · 6 comments
tzarebczan commented 2021-07-14 18:05:17 +02:00 (Migrated from github.com)
- [ ] some ads can be clicked, open in new tab, and ad pauses - there's no way to resume or skip (see slack) - [ ] do not load any ad stuff when signed in (https://tag.targeting.unrulymedia.com/rmp/216276/0/vast2?vastfw=vpaid&w=300&h=500&url=https://odysee.com/@Kona_and_Suba_Guinea_Pig_Adventures:c/this-is-peanut-the-guinea-pig:1 is called) - [ ] cut off at the bottom a bit and have a top black bar (https://lbryians.slack.com/archives/C81FGKR51/p1625678103453600?thread_ts=1625646978.442100&cid=C81FGKR51) - [ ] ads must have a skip button (Josh mentioned this)
infinite-persistence commented 2021-07-17 14:41:19 +02:00 (Migrated from github.com)

Regression:

  • The issue of videos not switching correctly (previously fixed by restoring the double src call) is back again.
  • The "Retry" button is gone, so we are back to having to do a full reload or re-enter the page.
  • Quality Selector not showing up in Brave Incognito. Works in Chrome Incognito.
  • Localization for some buttons (e.g. Play/Pause) seems to no longer work. Just need an update on the hierarchy, or update the code to be hierarchy-independant.
  • Pre-roll ads broke Theather Mode 6908

Nitpick:

  • The word "Advertisement" often lingers 1-2s when the video is already running, then goes away. Should go away sooner. If it ended up having no ads, the word shouldn't appear in the first place?
  • Things are not centered in Chrome Incognito. Looks ok when logged in. Maybe the same root-cause as "cut off at the bottom a bit" that Tom mentioned above.
  • Minor: It doesn't make sense to keep spinning when it already failed (makes user think whether to wait or not). This was previously fixed.
Regression: - [ ] The issue of videos not switching correctly (previously fixed by restoring the double `src` call) is back again. - [ ] The "Retry" button is gone, so we are back to having to do a full reload or re-enter the page. - [ ] Quality Selector not showing up in Brave Incognito. Works in Chrome Incognito. - [ ] Localization for some buttons (e.g. Play/Pause) seems to no longer work. Just need an update on the hierarchy, or update the code to be hierarchy-independant. - [ ] Pre-roll ads broke Theather Mode 6908 Nitpick: - [ ] The word "Advertisement" often lingers 1-2s when the video is already running, then goes away. Should go away sooner. If it ended up having no ads, the word shouldn't appear in the first place? - [ ] Things are not centered in Chrome Incognito. Looks ok when logged in. Maybe the same root-cause as "_cut off at the bottom a bit_" that Tom mentioned above. - <img src="https://user-images.githubusercontent.com/64950861/126041685-fa348909-61cb-4a3c-802b-3628775524cf.png" width="300"> - [ ] Minor: It doesn't make sense to keep spinning when it already failed (makes user think whether to wait or not). This was previously fixed. - <img src="https://user-images.githubusercontent.com/64950861/126037059-9ab18149-4b26-47c0-893d-fb0901a938a2.png" width="300">
tzarebczan commented 2021-07-19 17:12:11 +02:00 (Migrated from github.com)

We are getting more reports of adblockers blocking various calls/media on the page after we pushed this change, even when logged in. It may be related to the ad calls still getting made when logged in, but not sure. Reached out to adblocker.

We are getting more reports of adblockers blocking various calls/media on the page after we pushed this change, even when logged in. It may be related to the ad calls still getting made when logged in, but not sure. Reached out to adblocker.
tzarebczan commented 2021-07-20 16:34:25 +02:00 (Migrated from github.com)

Confirmed that my findings are causing issues for signed in users with pop up blockers due to the call still being made while signed in. This prevents the video from playing (probably just from this error, and potentially not from our domain being blacklisted)
image

Confirmed that my findings are causing issues for signed in users with pop up blockers due to the call still being made while signed in. This prevents the video from playing (probably just from this error, and potentially not from our domain being blacklisted) ![image](https://user-images.githubusercontent.com/8120721/126342744-95bb2e95-f802-4732-a838-4394e3e39641.png)
tzarebczan commented 2021-07-20 17:02:29 +02:00 (Migrated from github.com)

Some others

Some others - [ ] don't load for signed in users: https://imasdk.googleapis.com/js/sdkloader/ima3.js - [ ] double head request causes double master playlist call: ![image](https://user-images.githubusercontent.com/8120721/126347574-a180d0b4-5494-4b98-9366-6d5d0aeb7519.png)
kauffj commented 2021-09-15 18:08:53 +02:00 (Migrated from github.com)

@mayeaux this is an older ads issue, if it's not useful, please close it

@mayeaux this is an older ads issue, if it's not useful, please close it
tzarebczan commented 2021-10-08 22:33:37 +02:00 (Migrated from github.com)
Issue moved to [OdyseeTeam/odysee-frontend #18](https://github.com/OdyseeTeam/odysee-frontend/issues/18) via [**ZenHub**](https://www.zenhub.com/)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#6477
No description provided.