X-Git-Url: http://git.vanrenterghem.biz/git.ikiwiki.info.git/blobdiff_plain/82cc6ff4dadefa57c920c77ceb6a481afe6f0be9..refs/tags/3.14159:/doc/bugs/map_fails_to_close_ul_element_for_empty_list.mdwn
diff --git a/doc/bugs/map_fails_to_close_ul_element_for_empty_list.mdwn b/doc/bugs/map_fails_to_close_ul_element_for_empty_list.mdwn
index 28960b9d7..0edba438c 100644
--- a/doc/bugs/map_fails_to_close_ul_element_for_empty_list.mdwn
+++ b/doc/bugs/map_fails_to_close_ul_element_for_empty_list.mdwn
@@ -6,10 +6,10 @@ input:
Presuming that the pagespec does not match, output:
- <p>before.
- <div class="map">
- <ul>
- </div></p>
+
before.
+
The UL element is not closed.
@@ -33,3 +33,15 @@ Patch[[!tag patch]]:
-- [[Jon]]
+
+> Strictly speaking, a `` with no `- `s isn't valid HTML either...
+> could `map` instead delay emitting the first `
` until it determines that
+> it will have at least one item? Perhaps refactoring that function into
+> something easier to regression-test would be useful. --[[smcv]]
+
+>> You are right (just checked 4.01 DTD to confirm). I suspect refactoring
+>> the function would be wise. From my brief look at it to formulate the
+>> above I thought it was a bit icky. I'm not a good judge of what would
+>> be regression-test friendly but I might have a go at reworking it. With
+>> this variety of problem I have a strong inclination to use HOFs like map,
+>> grep. - [[Jon]]