Unequal and sub-optimal splits

Bug #1089162 reported by Ben Slusky
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Terminator
Fix Released
Undecided
Stephen Boddy

Bug Description

A concrete example: On my PC at work, Terminator in full-screen mode gives me a terminal window with a width of 131 characters. When I split that horizontally, the child windows are 64 and 63 characters in width. However, I can nudge the separator a bit and make both windows 64 characters in width. Why couldn't Terminator just do that on its own?

This happens on other displays as well as my work PC, with vertical splits as well as horizontal, with every release from 0.11 (possibly earlier) up to 0.96. It should be trivially reproducible.

Tags: split

Related branches

Revision history for this message
Stephen Boddy (stephen-j-boddy) wrote :

Yeah some ratio code was added in a while ago, and I can see where they missed out accounting for the size of the handle/seperator.

I have a trivial fix for the trivially reproducable problem.

Changed in terminator:
assignee: nobody → Stephen Boddy (stephen-j-boddy)
status: New → Confirmed
status: Confirmed → In Progress
Changed in terminator:
status: In Progress → Fix Committed
Chris Jones (cmsj)
Changed in terminator:
status: Fix Committed → Fix Released
Revision history for this message
Stephen Boddy (stephen-j-boddy) wrote :

Not quite sure why this got set to fix released. It is now committed as revision 1579

Changed in terminator:
status: Fix Released → Fix Committed
Revision history for this message
Stephen Boddy (stephen-j-boddy) wrote :

I should also mention: If you have saved layouts the splitters will have shifted a few pixels. When you originally saved them they used the ratio of the edge of the splitter. With this fix they saved the middle point. i.e. before your ratio might have been 0.49957663637 for a perfect 50:50 split. That would need to be changed to 0.5 now. Or you could re-equalize the splits, and use the new "Save" button to update the current layout.

Changed in terminator:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.