Ticket #1426 (closed feature: fixed)

Opened 22 months ago

Last modified 9 months ago

Related: Just in time RSS loading

Reported by: jeroen Owned by: sanil
Priority: Normal Milestone: Plugins
Component: general Keywords:
Cc: Forum thread:

Description (last modified by jeroen) (diff)

At present, the feed with related videos is loaded on JWPLAYER_PLAYLIST_ITEM. This results in:

  • Slower player initialization, since the related feed and thumbs are loaded together with the main assets.
  • Wasted bandwidth, since the related screen is watched by a subset of viewers.

This is fixed by delaying the RSS loading/parsing to the moment the related screen is first displayed.

Loading / Errors

This setup does imply a few changes to loading and error states in the player though:

  • If a related videos feed is being loaded, we should already fade in the related videos overlay, but not yet display any thumbs. Instead, the title should say Retrieving related videos...
  • If, for some reason, no videos were returned, the overlay can display a different message: No related videos found.

Note that the Related plugin currently hides the Related videos button and/or discards the Related videos onComplete fade if there's a feed error. This logic should get removed, since it's not possible to do anymore with just-in-time loading. The Error handling test page can be used to test this behavior.

Change History

comment:1 Changed 22 months ago by jeroen

  • Description modified (diff)

comment:2 Changed 18 months ago by jeroen

  • Priority set to Normal

comment:3 Changed 16 months ago by jeroen

  • Type changed from enhancement to feature

comment:4 Changed 10 months ago by jeroen

  • Summary changed from Related: just in time feed loading to Just in time related videos RSS loading
  • Milestone changed from Plugins to Player 6.0

comment:5 Changed 10 months ago by jeroen

  • Description modified (diff)
  • Summary changed from Just in time related videos RSS loading to Related: Just in time RSS loading

comment:6 Changed 10 months ago by jeroen

  • Milestone changed from Player 6.0 to Plugins

comment:7 Changed 9 months ago by jeroen

  • Description modified (diff)

comment:8 Changed 9 months ago by jeroen

  • Owner changed from jeroen to sanil

comment:9 Changed 9 months ago by sanil

  • Status changed from new to closed
  • Resolution set to fixed

comment:10 Changed 9 months ago by sanil

Committed the added features in [2379]

Note: See TracTickets for help on using tickets.