Skip to content

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

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

mattrose merged 2 commits into
gnome-terminator:masterfrom
vssdeo:613-shortcut-autosplit-h-v-dependingon-terminal-size

Conversation

@vssdeo

@vssdeo vssdeo commented Nov 5, 2022

Copy link
Copy Markdown
Contributor

…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

vssdeo commented Nov 5, 2022

Copy link
Copy Markdown
Contributor Author

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

vssdeo commented Nov 5, 2022

Copy link
Copy Markdown
Contributor Author

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

vssdeo commented Nov 19, 2022

Copy link
Copy Markdown
Contributor Author

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