Re: [PATCH] Implement command to enable / disable input

Is there anything else that needs to be done to this patch? Is it likely to be integrated any time soon? Thanks, Anish > On Aug 7, 2014, at 1:52 PM, Anish Athalye wrote: > > Thanks for the quick feedback! I've attached an updated patch. > > –Anish > > <0001-Add-feature-to-enable-or-disable-k

Re: [PATCH] Implement command to enable / disable input

Applied to OpenBSD with some minor tweaks, thanks On Thu, Aug 07, 2014 at 01:52:21PM -0700, Anish Athalye wrote: > Thanks for the quick feedback! I've attached an updated patch. > > ???Anish > > > > > On Aug 7, 2014, at 1:01 PM, Nicholas Marriott > > wrote: > > > > Ok but in that case yo

Re: [PATCH] Implement command to enable / disable input

Thanks for the quick feedback! I've attached an updated patch. –Anish 0001-Add-feature-to-enable-or-disable-keyboard-input.patch Description: Binary data > On Aug 7, 2014, at 1:01 PM, Nicholas Marriott > wrote: > > Ok but in that case you also need to add it to the last-pane synopsis. > >

Re: [PATCH] Implement command to enable / disable input

Ok but in that case you also need to add it to the last-pane synopsis. Also I don't see a need for any more one-line helper functions, just set and clear the flag directly and do without server_enable/disable_input. This: + "[-ed] [-lDLRU] " CMD_TARGET_PANE_USAGE, Should be changed to -Dd

Re: [PATCH] Implement command to enable / disable input

The last-pane section is also the part that runs if you `select-pane -l`. As the patch is now, running `select-pane -d -l` will disable input for the last pane that you were in. I think that behavior is more intuitive than having that function not work (because something like `select-pane -d -U`

Re: [PATCH] Implement command to enable / disable input

Hi You don't need to add these to the last-pane section as well, just to select-pane. Also can you update the man page? Cheers On Thu, Aug 07, 2014 at 04:09:30PM +, Anish R Athalye wrote: > Oh, good idea. I think adding it to the select-pane command makes more sense. > > Please see attach

Re: [PATCH] Implement command to enable / disable input

Oh, good idea. I think adding it to the select-pane command makes more sense. Please see attached patch. –Anish 0001-Add-feature-to-enable-or-disable-keyboard-input.patch Description: Binary data On Aug 7, 2014, at 8:16 AM, Nicholas Marriott wrote: > Please make this a flag or a pair of

Re: [PATCH] Implement command to enable / disable input

Please make this a flag or a pair of flags on an existing command. Maybe select-pane or kill-pane. On Thu, Aug 07, 2014 at 07:25:27AM +, Anish R Athalye wrote: > Here?s a slightly modified version of the patch ? changed the interface > slightly (better names). > > ?Anish > > > > > On A

Re: [PATCH] Implement command to enable / disable input

Here’s a slightly modified version of the patch — changed the interface slightly (better names).–Anish 0001-Implement-command-to-enable-disable-input.patch Description: Binary data On Aug 6, 2014, at 11:58 PM, Anish R Athalye wrote:I think that it would be nice to have the abili

[PATCH] Implement command to enable / disable input

I think that it would be nice to have the ability to make specific panes read only (switching from read/write).I’ve wanted this feature to keep myself from accidentally sending input to certain processes that I’ve started in certain panes (that I’m just monitoring). This would be great for things l