From 14c27e418a2e3fa2a551f067a55a18e3f22026ac Mon Sep 17 00:00:00 2001
From: Xavier Coulon <xcoulon@redhat.com>
Date: Sun, 20 Jan 2019 12:24:35 +0100
Subject: [PATCH] feat(renderer): support title on labeled lists

fixes #267

Signed-off-by: Xavier Coulon <xcoulon@redhat.com>
---
 pkg/parser/list_test.go                 | 48 +++++++++++++++++++++++++
 pkg/renderer/html5/labeled_list.go      |  3 +-
 pkg/renderer/html5/labeled_list_test.go | 21 +++++++++++
 3 files changed, 71 insertions(+), 1 deletion(-)

diff --git a/pkg/parser/list_test.go b/pkg/parser/list_test.go
index 207cc056..0217a29c 100644
--- a/pkg/parser/list_test.go
+++ b/pkg/parser/list_test.go
@@ -554,6 +554,54 @@ another fenced block
 			}
 			verify(GinkgoT(), expectedResult, actualContent, parser.Entrypoint("DocumentBlock"))
 		})
+
+		It("labeled list with title", func() {
+			actualContent := `.Labeled, single-line
+first term:: definition of the first term
+second term:: definition of the second term`
+			expectedResult := types.LabeledList{
+				Attributes: types.ElementAttributes{
+					types.AttrTitle: "Labeled, single-line",
+				},
+				Items: []types.LabeledListItem{
+					{
+						Attributes: types.ElementAttributes{},
+						Level:      1,
+						Term:       "first term",
+						Elements: []interface{}{
+							types.Paragraph{
+								Attributes: types.ElementAttributes{},
+								Lines: []types.InlineElements{
+									{
+										types.StringElement{
+											Content: "definition of the first term",
+										},
+									},
+								},
+							},
+						},
+					},
+					{
+						Attributes: types.ElementAttributes{},
+						Level:      1,
+						Term:       "second term",
+						Elements: []interface{}{
+							types.Paragraph{
+								Attributes: types.ElementAttributes{},
+								Lines: []types.InlineElements{
+									{
+										types.StringElement{
+											Content: "definition of the second term",
+										},
+									},
+								},
+							},
+						},
+					},
+				},
+			}
+			verify(GinkgoT(), expectedResult, actualContent, parser.Entrypoint("DocumentBlock"))
+		})
 	})
 
 	Context("unordered list", func() {
diff --git a/pkg/renderer/html5/labeled_list.go b/pkg/renderer/html5/labeled_list.go
index 2d5faa2f..1b2afa46 100644
--- a/pkg/renderer/html5/labeled_list.go
+++ b/pkg/renderer/html5/labeled_list.go
@@ -17,7 +17,8 @@ var horizontalLabeledListTmpl texttemplate.Template
 func init() {
 	defaultLabeledListTmpl = newTextTemplate("labeled list with default layout",
 		`{{ $ctx := .Context }}{{ with .Data }}<div{{ if .ID }} id="{{ .ID }}"{{ end }} class="dlist{{ if .Role }} {{ .Role }}{{ end }}">
-<dl>
+{{ if .Title }}<div class="title">{{ .Title }}</div>
+{{ end }}<dl>
 {{ $items := .Items }}{{ range $itemIndex, $item := $items }}<dt class="hdlist1">{{ $item.Term }}</dt>{{ if $item.Elements }}
 <dd>
 {{ renderElements $ctx $item.Elements | printf "%s" }}
diff --git a/pkg/renderer/html5/labeled_list_test.go b/pkg/renderer/html5/labeled_list_test.go
index 287c4d1b..899f2080 100644
--- a/pkg/renderer/html5/labeled_list_test.go
+++ b/pkg/renderer/html5/labeled_list_test.go
@@ -18,6 +18,7 @@ on 2 lines.
 item 3:: description 3
 on 2 lines, too.`
 			expectedResult := `<div id="listID" class="dlist myrole">
+<div class="title">mytitle</div>
 <dl>
 <dt class="hdlist1">item 1</dt>
 <dd>
@@ -144,6 +145,26 @@ item 2:: something simple`
 <p>something simple</p>
 </dd>
 </dl>
+</div>`
+			verify(GinkgoT(), expectedResult, actualContent)
+		})
+
+		It("labeled list with title", func() {
+			actualContent := `.Labeled, single-line
+first term:: definition of the first term
+second term:: definition of the second term`
+			expectedResult := `<div class="dlist">
+<div class="title">Labeled, single-line</div>
+<dl>
+<dt class="hdlist1">first term</dt>
+<dd>
+<p>definition of the first term</p>
+</dd>
+<dt class="hdlist1">second term</dt>
+<dd>
+<p>definition of the second term</p>
+</dd>
+</dl>
 </div>`
 			verify(GinkgoT(), expectedResult, actualContent)
 		})