lxterminal icon indicating copy to clipboard operation
lxterminal copied to clipboard

Making tabs reorderable

Open SauronfromMordor opened this issue 6 years ago • 8 comments

SauronfromMordor avatar Mar 02 '18 00:03 SauronfromMordor

I tried this patch and it seems to be working. This is a nice feature and I will be glad if this patch is merged :-)

mtasaka avatar May 04 '18 09:05 mtasaka

In "lxterminal 0.3.1", I can drag-and-drop tabs to re-order them. Would this change affect anything?

davesp avatar Aug 10 '18 22:08 davesp

Well, I cannot reorder tabs on 0.3.1 (compiled with GTK2) without this patch. I guess the patch author actually cannot reorder, either.

mtasaka avatar Aug 12 '18 10:08 mtasaka

My earlier test seems to have been faulty; after re-testing with the baseline, I can re-order tabs with shortcut keys but not with the mouse. With this patch, I can re-order tabs with the mouse.

davesp avatar Aug 13 '18 08:08 davesp

After reordering, the terminal->terms doesn't properly reindexed (see terminal_move_tab_execute). I am wondering if we shouldn't make such array keep tracking the terminals but extend the underlying GtkBox to accommodate term structure.

medicalwei avatar Aug 18 '18 00:08 medicalwei

After reordering, the terminal->terms doesn't properly reindexed

Just a thought: there is "page-reordered" signal, so perhaps we can do what is written in terminal_move_tab_execute when this signal is emitted (but some modification will be needed)

mtasaka avatar Aug 27 '18 07:08 mtasaka

After reordering, the terminal->terms doesn't properly reindexed (see terminal_move_tab_execute). I am wondering if we shouldn't make such array keep tracking the terminals but extend the underlying GtkBox to accommodate term structure.

I had actually started working on this a while back. If I recall, my approach was to get rid of our list of term objects and instead manage it through the gtk objects themselves (Which would re-order automatically). It was a lot of re-writing and testing though and sadly fell on the wayside. It is definitely something I planned to implement however.

FinboySlick avatar Mar 11 '20 16:03 FinboySlick

Fixes #73

itsdonnix avatar Jan 31 '21 12:01 itsdonnix