search

acudovs / terminator-split

Wrapper script for splitting Terminator terminal emulator
MIT License
25 stars 11 forks source link

Documentation required #2

Open reaperzn opened 7 years ago

reaperzn commented 7 years ago

Firstly, Thanks for addressing #1 This is a useful tool for me and I hope that others can make use of it too.

There is some clarification required for the -h message. I have made some minor changes and added some explanations for some of the options I was able to figure out. I'll send a PR soon.

Clarification required: 1) Confirmation that the -g option is the terminator config file to be used 2) Unsure what the -t, --terminator option is used for. 3) The -e, --command option needs a bit of detail as to what it is used for. I tried running terminator-split -e hostname 3 but the command was not run and the command in the debug was still resulting in the default "bash -l":

args:Namespace(command='bash -l', config='/home/reaper/.config/terminator/config', hostname=[0, 1, 2], level='DEBUG', shell='bash -l', terminator='/usr/bin/terminator')
unknown_args:[]
config:/tmp/tmppP9iRM:'{'global_config': {'handle_size': '5', 'inactive_color_offset': '1.0'}, 'keybindings': {'broadcast_all': '<Primary><Shift>a', 'broadcast_group': '<Primary><Shift>q', 'broadcast_off': '<Primary><Shift>z', 'close_window': '<Primary><Shift><Alt>k'}, 'layouts': {'default': {'child0': {'type': 'Window', 'parent': ''}, 'child1': {'type': 'HPaned', 'parent': 'child0'}, 'child2': {'type': 'VPaned', 'parent': 'child1'}, 'child3': {'type': 'Terminal', 'command': 'bash -l', 'parent': 'child2'}, 'child4': {'type': 'Terminal', 'command': 'bash -l', 'parent': 'child2'}, 'child5': {'type': 'Terminal', 'command': 'bash -l', 'parent': 'child1'}}}, 'plugins': {}, 'profiles': {'default': {'background_darkness': '0.92', 'background_image': 'None', 'background_type': 'transparent', 'palette': '#000000:#aa0000:#00aa00:#aa5500:#0000aa:#aa00aa:#00aaaa:#aaaaaa:#555555:#ff5555:#55ff55:#ffff55:#5555ff:#ff55ff:#55ffff:#ffffff', 'scrollback_infinite': 'True'}}}'
acudovs commented 7 years ago

I'll think to make the help more understandable.