* [PATCH] column doc: rewrite documentation for column.ui
@ 2013-06-25 15:54 Ramkumar Ramachandra
2013-06-25 22:18 ` Junio C Hamano
0 siblings, 1 reply; 3+ messages in thread
From: Ramkumar Ramachandra @ 2013-06-25 15:54 UTC (permalink / raw)
To: Git List; +Cc: Junio C Hamano
The configuration option column.ui is very poorly documented, and it is
unclear what the defaults are, and what option can be combined with
what. Rewrite it by splitting up the options into three sections
clearly showing how COL_ENABLED, COL_LAYOUT_MASK, and COL_DENSE work.
Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
---
Documentation/config.txt | 20 +++++++++++++++++---
1 file changed, 17 insertions(+), 3 deletions(-)
diff --git a/Documentation/config.txt b/Documentation/config.txt
index 1153585..5a10169 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -934,6 +934,9 @@ column.ui::
This variable consists of a list of tokens separated by spaces
or commas:
+
+These three options control when the feature should be enabled
+(defaults to 'never'):
++
--
`always`;;
always show in columns
@@ -941,19 +944,30 @@ column.ui::
never show in columns
`auto`;;
show in columns if the output is to the terminal
+--
++
+These three options control layout (defaults to 'column'). Setting
+layout implies 'always' if none of 'always', 'never', or 'auto' are
+specified.
++
+--
`column`;;
- fill columns before rows (default)
+ fill columns before rows
`row`;;
fill rows before columns
`plain`;;
show in one column
+--
++
+Finally, these options can be combined with a layout option (defaults
+to 'nodense'):
++
+--
`dense`;;
make unequal size columns to utilize more space
`nodense`;;
make equal size columns
--
-+
-This option defaults to 'never'.
column.branch::
Specify whether to output branch listing in `git branch` in columns.
--
1.8.3.1.585.g9832cb9
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] column doc: rewrite documentation for column.ui
2013-06-25 15:54 [PATCH] column doc: rewrite documentation for column.ui Ramkumar Ramachandra
@ 2013-06-25 22:18 ` Junio C Hamano
2013-06-26 6:37 ` Ramkumar Ramachandra
0 siblings, 1 reply; 3+ messages in thread
From: Junio C Hamano @ 2013-06-25 22:18 UTC (permalink / raw)
To: Ramkumar Ramachandra; +Cc: Git List
Ramkumar Ramachandra <artagnon@gmail.com> writes:
> The configuration option column.ui is very poorly documented, and it is
> unclear what the defaults are, and what option can be combined with
> what. Rewrite it by splitting up the options into three sections
> clearly showing how COL_ENABLED, COL_LAYOUT_MASK, and COL_DENSE work.
>
> Signed-off-by: Ramkumar Ramachandra <artagnon@gmail.com>
> ---
Thanks.
One micronit (read: if you think the suggested change is a good
idea, I could just locally amend it) is that it would read better
and also easier to maintain to say
These options control
without "three". Both 'man' and 'html' output looked good.
> Documentation/config.txt | 20 +++++++++++++++++---
> 1 file changed, 17 insertions(+), 3 deletions(-)
>
> diff --git a/Documentation/config.txt b/Documentation/config.txt
> index 1153585..5a10169 100644
> --- a/Documentation/config.txt
> +++ b/Documentation/config.txt
> @@ -934,6 +934,9 @@ column.ui::
> This variable consists of a list of tokens separated by spaces
> or commas:
> +
> +These three options control when the feature should be enabled
> +(defaults to 'never'):
> ++
> --
> `always`;;
> always show in columns
> @@ -941,19 +944,30 @@ column.ui::
> never show in columns
> `auto`;;
> show in columns if the output is to the terminal
> +--
> ++
> +These three options control layout (defaults to 'column'). Setting
> +layout implies 'always' if none of 'always', 'never', or 'auto' are
> +specified.
> ++
> +--
> `column`;;
> - fill columns before rows (default)
> + fill columns before rows
> `row`;;
> fill rows before columns
> `plain`;;
> show in one column
> +--
> ++
> +Finally, these options can be combined with a layout option (defaults
> +to 'nodense'):
> ++
> +--
> `dense`;;
> make unequal size columns to utilize more space
> `nodense`;;
> make equal size columns
> --
> -+
> -This option defaults to 'never'.
>
> column.branch::
> Specify whether to output branch listing in `git branch` in columns.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] column doc: rewrite documentation for column.ui
2013-06-25 22:18 ` Junio C Hamano
@ 2013-06-26 6:37 ` Ramkumar Ramachandra
0 siblings, 0 replies; 3+ messages in thread
From: Ramkumar Ramachandra @ 2013-06-26 6:37 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Git List
Junio C Hamano wrote:
> One micronit (read: if you think the suggested change is a good
> idea, I could just locally amend it) is that it would read better
> and also easier to maintain to say
>
> These options control
>
> without "three". Both 'man' and 'html' output looked good.
Sure, amend it.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-26 6:38 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-25 15:54 [PATCH] column doc: rewrite documentation for column.ui Ramkumar Ramachandra
2013-06-25 22:18 ` Junio C Hamano
2013-06-26 6:37 ` Ramkumar Ramachandra
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).