position from --geometry param not respected when combined with hidden param

Bug #1035617 reported by Lars Kreisz
4
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Terminator
Fix Released
Medium
Unassigned

Bug Description

When i start terminator with the -H switch this way:

terminator -H --geometry 1280x720+320+24

the height and width are set correctly, but the position is at (0, 24) (directly under the main panel).

Taking away the hidden switch:

terminator --geometry 1280x720+320+24

anything is fine.

I tried this with the terminator version that comes from UBuntu 12.04 packages (0.96) under Unity (pretty fresh system).

Related branches

Chris Jones (cmsj)
Changed in terminator:
milestone: none → 1.0
Changed in terminator:
importance: Undecided → Medium
Revision history for this message
Stephen Boddy (stephen-j-boddy) wrote :

Lars, I have a fix that works for me. Any chance you could check it? I causes a quick ugly flash of the window on start, but after that it all works fine. I don't think the flash can be gotten rid of, as even getting the position and storing in a temporary variable causes the flash.

=== modified file 'terminatorlib/window.py'
--- terminatorlib/window.py 2013-02-10 15:58:06 +0000
+++ terminatorlib/window.py 2013-09-03 13:06:06 +0000
@@ -385,6 +385,7 @@

         #Window must be shown, then hidden for the hotkeys to be registered
         if (self.ignore_startup_show and startup == True):
+ self.position = self.get_position()
             self.hide()

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

Ugh! Ugly formatting! The added line should be at the same indentation as the self.hide() statement.

Changed in terminator:
status: New → In Progress
Changed in terminator:
status: In Progress → Fix Committed
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.