Render task list item inside label for correct accessibility

Fixes #3303

diff --git a/helpers/content_renderer.go b/helpers/content_renderer.go
index dee8724a9b711350f4ada56af0db2da4e7c6eaa5..63be58104a1a44422f94dd577b0b51c1e8822a4d 100644 (file)
--- a/helpers/content_renderer.go
+++ b/helpers/content_renderer.go
@@ -50,10 +50,12 @@ func (r *HugoHTMLRenderer) ListItem(out *bytes.Buffer, text []byte, flags int) {
 
        switch {
        case bytes.HasPrefix(text, []byte("[ ] ")):
-               text = append([]byte(`<input type="checkbox" disabled class="task-list-item">`), text[3:]...)
+               text = append([]byte(`<label><input type="checkbox" disabled class="task-list-item">`), text[3:]...)
+               text = append(text, []byte(`</label>`)...)
 
        case bytes.HasPrefix(text, []byte("[x] ")) || bytes.HasPrefix(text, []byte("[X] ")):
-               text = append([]byte(`<input type="checkbox" checked disabled class="task-list-item">`), text[3:]...)
+               text = append([]byte(`<label><input type="checkbox" checked disabled class="task-list-item">`), text[3:]...)
+               text = append(text, []byte(`</label>`)...)
        }
 
        r.Renderer.ListItem(out, text, flags)

diff --git a/helpers/content_renderer_test.go b/helpers/content_renderer_test.go
index 63e681d97b9c733c3927a77652f007f68d612a7c..3bd038547d00f2bf236390083dd842d68ff84020 100644 (file)
--- a/helpers/content_renderer_test.go
+++ b/helpers/content_renderer_test.go
@@ -108,9 +108,9 @@ END
 `, true, `<p>TODO:</p>
 
 <ul class="task-list">
-<li><input type="checkbox" checked disabled class="task-list-item"> On1</li>
-<li><input type="checkbox" checked disabled class="task-list-item"> On2</li>
-<li><input type="checkbox" disabled class="task-list-item"> Off</li>
+<li><label><input type="checkbox" checked disabled class="task-list-item"> On1</label></li>
+<li><label><input type="checkbox" checked disabled class="task-list-item"> On2</label></li>
+<li><label><input type="checkbox" disabled class="task-list-item"> Off</label></li>
 </ul>
 
 <p>END</p>
@@ -122,7 +122,7 @@ END
                {`* [ ] Off
 
 END`, true, `<ul class="task-list">
-<li><input type="checkbox" disabled class="task-list-item"> Off</li>
+<li><label><input type="checkbox" disabled class="task-list-item"> Off</label></li>
 </ul>
 
 <p>END</p>