Tickets :: [#12573] Tweak new window resize logic

6.0.0-beta1

7/4/25

Summary	Tweak new window resize logic
Queue	IMP
Queue Version	Git master
Type	Enhancement
State	No Feedback
Priority	1. Low
Owners
Requester	thomas.jarosch (at) intra2net (dot) com
Created	08/14/2013 (4342 days ago)
Due
Updated	09/02/2013 (4323 days ago)
Assigned
Resolved	09/02/2013 (4323 days ago)
Milestone
Patch	Yes

09/02/2013 07:36:04 PM	Michael Slusarz	Comment #6 State ⇒ No Feedback	Reply to this comment
Will mark as "No Feedback" for now.

09/02/2013 01:01:05 PM	Thomas Jarosch	Comment #5	Reply to this comment
I'll try to output some debug logging with alert() or console.log() tomorrow. So much for that... anyway I'll probably won't have time for this before my vacation. Feel free to close the ticket and I'll reopen it if ever needed in October.

08/26/2013 10:30:05 PM	Thomas Jarosch	Comment #4	Reply to this comment
Ping? As mentioned, 100 pixel offset is way too much. So this needs to be worked around from a browser property value perspective if possible (i.e. detect browser that has broken window.availHeight. I'll try to output some debug logging with alert() or console.log() tomorrow. (I must admin I haven't done much Javascript development yet...) to look at the actual values of my browser / window manager.

08/26/2013 09:37:15 PM	Michael Slusarz	Comment #3	Reply to this comment
Ping? As mentioned, 100 pixel offset is way too much. So this needs to be worked around from a browser property value perspective if possible (i.e. detect browser that has broken window.availHeight.

08/14/2013 09:00:01 PM	Michael Slusarz	Comment #2 State ⇒ Feedback	Reply to this comment
This isn't correct. screen.availHeight is defined as the available screen height taking into account any OS-specific UI elements. It works fine for me: on FF/Win 7 screen.availHeight is the height between the top of the screen and the bottom taskbar. The 50 pixels are just in there because it is difficult to move the window if it takes up the full screen space. But I'd rather not be throwing in random pixel offsets to account for broken browser behavior.

08/14/2013 11:02:40 AM	Thomas Jarosch	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ Tweak new window resize logic Queue ⇒ IMP Milestone ⇒ Patch ⇒ Yes New Attachment: 0001-Adjust-compose-window-resize-logic.patch State ⇒ New	Reply to this comment
Hi, I really like the new window resize logic of imp. On my desktop, the current code resizes the window so big that the lower border of it directly sits on the taskbar. That's makes grabbing the lower border harder than it needs to be. I've changed the threshold value from -50 to -100 and now it looks nice. That's also the same threshold value used in compose.js (found while grepping for "- 100"). Cheers, Thomas