Re: [dev] [dmenu] add a space when tab is pressed [patch]

2011-08-26 Thread hiro
On 27.08.2011, Andrew Hills  wrote:
> I don't want to add an argument when I press Tab. On what system is
> this the default operation, such that it is "obvious"?

The shell completion feature



Re: [dev] [dmenu] add a space when tab is pressed [patch]

2011-08-26 Thread Jacob Todd
Then don't apply the patch.
On Aug 26, 2011 6:56 PM, "Andrew Hills"  wrote:
> I don't want to add an argument when I press Tab. On what system is
> this the default operation, such that it is "obvious"?
>
> --Andrew Hills
>
>
>
> On Fri, Aug 26, 2011 at 1:23 PM,  wrote:
>> Hello,
>> when tab is pressed, one wants obviously to add an argument
>> and has to press space before that.
>> The attached patch does it for you.
>> Have a nice week-end.
>>
>


Re: [dev] [dmenu] add a space when tab is pressed [patch]

2011-08-26 Thread Andrew Hills
I don't want to add an argument when I press Tab. On what system is
this the default operation, such that it is "obvious"?

--Andrew Hills



On Fri, Aug 26, 2011 at 1:23 PM,   wrote:
> Hello,
> when tab is pressed, one wants obviously to add an argument
> and has to press space before that.
> The attached patch does it for you.
> Have a nice week-end.
>



[dev] [dmenu] add a space when tab is pressed [patch]

2011-08-26 Thread skls
Hello,
when tab is pressed, one wants obviously to add an argument
and has to press space before that.
The attached patch does it for you.
Have a nice week-end.
diff -up a/dmenu.c b/dmenu.c
--- a/dmenu.c   Fri Aug 26 19:13:17 2011
+++ b/dmenu.c   Fri Aug 26 19:10:56 2011
@@ -353,6 +353,7 @@ keypress(XKeyEvent *ev) {
case XK_Tab:
if(!sel)
return;
+   strcat(sel->text, " ");
strncpy(text, sel->text, sizeof text);
cursor = strlen(text);
match(True);