Merge pull request #2 from TheBookPeople/unordered-lists

Add new line for unordered lists

Author: k3a

html2text.go

@@ -161,8 +161,12 @@ func HTML2Text(html string) string {
 			shouldOutput = true
 			tagName := strings.ToLower(html[tagStart:i])
 
-			if headersRE.MatchString(tagName) {
-				if canPrintNewline {
+			if tagName == "/ul" {
+				outBuf.WriteString("\r\n")
+			} else if tagName == "li" || tagName == "li/" {
+				outBuf.WriteString("\r\n")
+      } else if headersRE.MatchString(tagName) {
+			  if canPrintNewline {
 					outBuf.WriteString("\r\n\r\n")
 				}
 				canPrintNewline = false

html2text_test.go

@@ -1,7 +1,10 @@
 package html2text
 
-import "testing"
-import . "github.com/smartystreets/goconvey/convey"
+import (
+	"testing"
+
+	. "github.com/smartystreets/goconvey/convey"
+)
 
 func TestHTML2Text(t *testing.T) {
 	Convey("HTML2Text should work", t, func() {
@@ -49,6 +52,7 @@ func TestHTML2Text(t *testing.T) {
 				ShouldEqual, "would you pay in ¢, £, ¥ or €?")
 			So(HTML2Text(`Tom & Jerry is not an entity`), ShouldEqual, "Tom & Jerry is not an entity")
 			So(HTML2Text(`this &neither; as you see`), ShouldEqual, "this &neither; as you see")
+			So(HTML2Text(`list of items<ul><li>One</li><li>Two</li><li>Three</li></ul>`), ShouldEqual, "list of items\r\nOne\r\nTwo\r\nThree\r\n")
 			So(HTML2Text(`fish &amp; chips`), ShouldEqual, "fish & chips")
 			So(HTML2Text(`&quot;I'm sorry, Dave. I'm afraid I can't do that.&quot; – HAL, 2001: A Space Odyssey`), ShouldEqual, "\"I'm sorry, Dave. I'm afraid I can't do that.\" – HAL, 2001: A Space Odyssey")
 			So(HTML2Text(`Google &reg;`), ShouldEqual, "Google ®")