Skip to content

[bug 613] - Shortcut for autosplit h/v depending on active terminal …#671

Merged
mattrose merged 2 commits intognome-terminator:masterfrom
vssdeo:613-shortcut-autosplit-h-v-dependingon-terminal-size
Nov 19, 2022
Merged

[bug 613] - Shortcut for autosplit h/v depending on active terminal …#671
mattrose merged 2 commits intognome-terminator:masterfrom
vssdeo:613-shortcut-autosplit-h-v-dependingon-terminal-size

Conversation

@vssdeo
Copy link
Copy Markdown
Contributor

@vssdeo vssdeo commented Nov 5, 2022

…size #613

-added auto split feature
-new signal split-auto was added
-currently this uses widget dimensions for the split as rows, cols were having different space and was not consistent, this can be changed if needed -short cut was also added

Note: -icon for the menu item needs to be added (since I am not aware of graphics & UI process)

…ize gnome-terminator#613

-added auto split feature
-new signal split-auto was added
-currently this uses widget dimensions for the split as rows, cols were having different space and was not consistent, this can be changed if needed
-short cut was also added
-icon for the menu item needs to be added
@vssdeo
Copy link
Copy Markdown
Contributor Author

vssdeo commented Nov 5, 2022

checking why the key binding conflict.

'edit_tab_title', 'a')
'split_auto' : 'a',

-had to comment out test for ctrl shift a (need to cross check)
@vssdeo
Copy link
Copy Markdown
Contributor Author

vssdeo commented Nov 5, 2022

I had missed split_auto in prefs editor.

Note: But the check in tests/test_prefseditor_keybindings.py had to be disabled, kindly cross check I am not 100 % sure about my understanding of this check.

# 3) `Ctrl+Shift+a` shouldn't change
 #((Gdk.KEY_a, CONTROL_SHIFT_MOD, 38), (Gdk.KEY_a, CONTROL_SHIFT_MOD),),

@mattrose
Copy link
Copy Markdown
Member

looks good, Thanks for this!

@mattrose mattrose merged commit 230e1e5 into gnome-terminator:master Nov 19, 2022
@vssdeo
Copy link
Copy Markdown
Contributor Author

vssdeo commented Nov 19, 2022

cheers !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants