From 721c913fdebdf5216b86a2c87ec73729624be2b2 Mon Sep 17 00:00:00 2001
From: "Evan R. Murphy" <evan@answerdash.com>
Date: Thu, 11 Aug 2016 16:13:02 -0700
Subject: [PATCH 1/6] Convert <video> tags to links

Don't strip out `<video>` tags completely, rather convert
them to bracketed links like is done for images.

Relates to https://github.com/werk85/node-html-to-text/issues/103
---
 lib/formatter.js    | 20 ++++++++++++++++++++
 lib/html-to-text.js |  3 +++
 2 files changed, 23 insertions(+)

diff --git a/lib/formatter.js b/lib/formatter.js
index 126d32e..f06bbe5 100644
--- a/lib/formatter.js
+++ b/lib/formatter.js
@@ -33,6 +33,25 @@ function formatImage(elem, options) {
 	return (result);
 }
 
+function formatVideo(elem, options) {
+  // Uncomment once we have an ignoreVideo option
+  //if (options.ignoreVideo) {
+  //  return '';
+  //}
+
+	var result = '', attribs = elem.attribs || {};
+	if (attribs.alt) {
+		result += he.decode(attribs.alt, options.decodeOptions);
+		if (attribs.src) {
+			result += ' ';
+		}
+	}
+	if (attribs.src) {
+		result += '[' + attribs.src + ']';
+	}
+	return (result);
+}
+
 function formatLineBreak(elem, fn, options) {
 	return '\n' + fn(elem.children, options);
 }
@@ -219,6 +238,7 @@ function formatTable(elem, fn, options) {
 
 exports.text = formatText;
 exports.image = formatImage;
+exports.video = formatVideo;
 exports.lineBreak = formatLineBreak;
 exports.paragraph = formatParagraph;
 exports.anchor = formatAnchor;
diff --git a/lib/html-to-text.js b/lib/html-to-text.js
index 08e7ae0..9f20d3f 100644
--- a/lib/html-to-text.js
+++ b/lib/html-to-text.js
@@ -112,6 +112,9 @@ function walk(dom, options, result) {
 					case 'img':
 						result += format.image(elem, options);
 						break;
+					case 'video':
+						result += format.video(elem, options);
+						break;
 					case 'a':
 						// Inline element needs its leading space to be trimmed if `result`
 						// currently ends with whitespace

From c18914ccfce229355f3847dbfcbffb965437f5b4 Mon Sep 17 00:00:00 2001
From: "Evan R. Murphy" <evan@answerdash.com>
Date: Thu, 11 Aug 2016 16:36:32 -0700
Subject: [PATCH 2/6] Add ignoreVideo option

Also correct spacing (tabs, not spaces) in `formatVideo` function.
---
 README.md        | 1 +
 bin/cli.js       | 4 +++-
 lib/formatter.js | 7 +++----
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/README.md b/README.md
index 7adf55d..12f960c 100644
--- a/README.md
+++ b/README.md
@@ -60,6 +60,7 @@ You can configure the behaviour of html-to-text with the following options:
  * `hideLinkHrefIfSameAsText` by default links are translated the following `<a href='link'>text</a>` => becomes => `text [link]`. If this option is set to true and `link` and `text` are the same, `[link]` will be hidden and only `text` visible.
  * `ignoreHref` ignore all document links if `true`.
  * `ignoreImage` ignore all document images if `true`.
+ * `ignoreVideo` ignore all document videos if `true`.
  * `preserveNewlines` by default, any newlines `\n` in a block of text will be removed. If `true`, these newlines will not be removed.
  * `decodeOptions` defines the text decoding options given to `he.decode`. For more informations see the [he](https://github.com/mathiasbynens/he) module.
  * `uppercaseHeadings` by default, headings (`<h1>`, `<h2>`, etc) are uppercased. Set to `false` to leave headings as they are.
diff --git a/bin/cli.js b/bin/cli.js
index 0372178..231fc36 100755
--- a/bin/cli.js
+++ b/bin/cli.js
@@ -8,6 +8,7 @@ var argv = optimist
 	.default('wordwrap', 80)
 	.default('ignore-href', false)
 	.default('ignore-image', false)
+	.default('ignore-video', false)
 	.argv;
 
 var text = '';
@@ -25,7 +26,8 @@ process.stdin.on('end', function end() {
 		tables: interpretTables(argv.tables),
 		wordwrap: argv.wordwrap,
 		ignoreHref: argv['ignore-href'],
-		ignoreImage: argv['ignore-image']
+		ignoreImage: argv['ignore-image'],
+		ignoreVideo: argv['ignore-video']
 	});
 	process.stdout.write(text + '\n', 'utf-8');
 });
diff --git a/lib/formatter.js b/lib/formatter.js
index f06bbe5..f1cc286 100644
--- a/lib/formatter.js
+++ b/lib/formatter.js
@@ -34,10 +34,9 @@ function formatImage(elem, options) {
 }
 
 function formatVideo(elem, options) {
-  // Uncomment once we have an ignoreVideo option
-  //if (options.ignoreVideo) {
-  //  return '';
-  //}
+	if (options.ignoreVideo) {
+	  return '';
+	}
 
 	var result = '', attribs = elem.attribs || {};
 	if (attribs.alt) {

From 4507139cf158b3e08455d5c6451b93f79500940b Mon Sep 17 00:00:00 2001
From: "Evan R. Murphy" <evan@answerdash.com>
Date: Thu, 11 Aug 2016 17:21:05 -0700
Subject: [PATCH 3/6] Don't check for alt attribute on video tags

`video` tags don't have `alt` attributes.
---
 lib/formatter.js | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/lib/formatter.js b/lib/formatter.js
index f1cc286..f83387b 100644
--- a/lib/formatter.js
+++ b/lib/formatter.js
@@ -39,12 +39,6 @@ function formatVideo(elem, options) {
 	}
 
 	var result = '', attribs = elem.attribs || {};
-	if (attribs.alt) {
-		result += he.decode(attribs.alt, options.decodeOptions);
-		if (attribs.src) {
-			result += ' ';
-		}
-	}
 	if (attribs.src) {
 		result += '[' + attribs.src + ']';
 	}

From 978df3cd478273d01134721ea231d51deeb1f239 Mon Sep 17 00:00:00 2001
From: "Evan R. Murphy" <evan@answerdash.com>
Date: Thu, 11 Aug 2016 17:46:30 -0700
Subject: [PATCH 4/6] Recognize videos with source tag

Before only recognized `video` tags with a `src` attribute. Now
if no `src` attribute is found, check for a `source` child element.
---
 lib/formatter.js | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/lib/formatter.js b/lib/formatter.js
index f83387b..57edced 100644
--- a/lib/formatter.js
+++ b/lib/formatter.js
@@ -39,9 +39,18 @@ function formatVideo(elem, options) {
 	}
 
 	var result = '', attribs = elem.attribs || {};
+
 	if (attribs.src) {
 		result += '[' + attribs.src + ']';
+	} else if (elem.children.length > 0) {
+		_.each(elem.children, function(el) {
+			var elAttribs = el.attribs || {};
+			if (el.type === 'tag' && el.name.toLowerCase() === 'source') {
+				result += '[' + el.attribs.src + ']';
+			}
+		});
 	}
+
 	return (result);
 }
 

From 52f174879d9b8b1dc83976147c6588e6eebe4ad2 Mon Sep 17 00:00:00 2001
From: "Evan R. Murphy" <evan@answerdash.com>
Date: Thu, 11 Aug 2016 17:53:26 -0700
Subject: [PATCH 5/6] Update examples/test.html with image a video examples

---
 example/test.html | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/example/test.html b/example/test.html
index 4e8706e..7af0b67 100644
--- a/example/test.html
+++ b/example/test.html
@@ -142,5 +142,19 @@ <h2>Pretty printed Source Code</h2>
 	console.log(text);
 });
 		</pre>
+
+		<hr />
+		<h2>Img tag</h2>
+		<img src="http://example.com/image" />
+
+		<hr />
+		<h2>Video tag (src attribute)</h2>
+		<video src="http://example.com/video-with-src-attribute"></video>
+
+		<hr />
+		<h2>Video tag (source tag)</h2>
+		<video>
+			<source src="http://example.com/video-with-source-tag">
+		</video>
 	</body>
-</html>
\ No newline at end of file
+</html>

From 32167991d3ca8ea95fd78f1ddb6c516e71d6008a Mon Sep 17 00:00:00 2001
From: "Evan R. Murphy" <evan@answerdash.com>
Date: Thu, 11 Aug 2016 18:09:28 -0700
Subject: [PATCH 6/6] Use file extensions in image and video examples

---
 example/test.html | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/example/test.html b/example/test.html
index 7af0b67..ab14ea9 100644
--- a/example/test.html
+++ b/example/test.html
@@ -145,16 +145,16 @@ <h2>Pretty printed Source Code</h2>
 
 		<hr />
 		<h2>Img tag</h2>
-		<img src="http://example.com/image" />
+		<img src="http://example.com/image.png" />
 
 		<hr />
 		<h2>Video tag (src attribute)</h2>
-		<video src="http://example.com/video-with-src-attribute"></video>
+		<video src="http://example.com/video-with-src-attribute.mp4"></video>
 
 		<hr />
 		<h2>Video tag (source tag)</h2>
 		<video>
-			<source src="http://example.com/video-with-source-tag">
+			<source src="http://example.com/video-with-source-tag.mp4">
 		</video>
 	</body>
 </html>