[Feature Suggestion] Ignore extra header lines about node-webvtt HOT 4 CLOSED

Hi @abstractvector, that sounds like a fine way to go about it and I'd be happy to merge a PR.

from node-webvtt.

@osk would you be happy if I go the route of exposing the fields as an extra top-level object called meta? In the case of my example above, parse() would return something like:

{
  "valid": true,
  "meta": {
    "Kind": "captions",
    "Language": "en"
  },
  "cues": [
    // ..
  ]
}

from node-webvtt.

Looks like this is something being added via the webm project:
http://wiki.webmproject.org/webm-metadata/temporal-metadata/webvtt-metadata
but that's just after a few minutes of research.

It should be fine if webvtt.parse(input); would get an options param so e.g. webvtt.parse(input, { meta: true }); would look for these meta fields and add them to the output. If the param is passed but no metadata found, it should return "meta": null or something to that effect.

from node-webvtt.

@osk Sounds great! I'm not an expert on this topic by any means, so I appreciate the steer! I'll work on a PR for it.

from node-webvtt.