From 5ffe09e6165b23c2b8415313bc25ed84fa8745d5 Mon Sep 17 00:00:00 2001 From: smcv Date: Sun, 3 Feb 2019 14:39:51 -0400 Subject: [PATCH] respond --- doc/todo/support_multi-row_table_headers.mdwn | 30 +++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/doc/todo/support_multi-row_table_headers.mdwn b/doc/todo/support_multi-row_table_headers.mdwn index 1d418a41b..07198fd5c 100644 --- a/doc/todo/support_multi-row_table_headers.mdwn +++ b/doc/todo/support_multi-row_table_headers.mdwn @@ -96,3 +96,33 @@ and easily underneath. I'd appreciate your take on the above suggestions [[smcv]] before I roll my sleeves up. Thanks! — [[Jon]] (2018-09-24) + +> I continue to think that the `header` parameter shouldn't be sometimes a +> description of which parts of the table are header, and sometimes the header +> data itself; so if you want an inline header, it should indeed have a +> distinct name. +> +> If you can think of a good name for the new parameter, and can document it +> reasonably clearly, then I would be OK with having a separate parameter that +> is the externally-provided header. I don't know what the right name for that +> parameter would be: `headercontent` or `headerblock` is unwieldy but I can't +> think of anything better. +> +> It would maybe simplify things to make it mutually exclusive with `header`, +> but then you wouldn't be able to express things like "the first column of my +> CSV is a header, the first row is just an ordinary row, and please add +> this literal header row to the top". +> +> It might help to write the documentation and/or tests first, and then +> implement it afterwards, when you have an "API" you're happy with. +> +> Corner cases: +> +> How would it work if you want to add a literal header column on the left +> rather than adding a literal header row on the top? If you add both, what +> happens at the top left corner? +> +> Is it necessary to be able to add header columns on the right (for RTL +> languages?), or header rows (footer rows, I suppose) on the bottom? +> +> --[[smcv]] -- 2.39.2