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

YouTube Shell/Remixer Queries URL for default start/end times #142

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

anshukla
Copy link
Member

@anshukla anshukla commented Aug 8, 2013

Resolves issue #122 . The YouTube Remixer now sets clip selection according to the URL, if available. Supported time formats:

  • t=<n>
  • t=<n1>m<n2>s
  • ``start=`
  • start=<n1>m<n2>s
  • end=<n>
  • ``end=ms`

Currently, this functionality is limited to YouTube videos, but initializeDefaultClip can be moved up to LinkShell in general in order to handle other URL parameters (if relevant).

This pull request also introduces a new utility function acorn.util.fetchParameters. This function takes a URL and an array of parameters and returns an object with the parameters and their values, if available. If the parameters are not present, then they are not in the object. In the case that no parameters are supplied in the URL, the empty object is returned.

@jbenet
Copy link
Member

jbenet commented Aug 9, 2013

A few comments above. after addressing, looks good to merge

@jbenet
Copy link
Member

jbenet commented Aug 9, 2013

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants