New browser window is very small in Chrome

[rcubetrac]

Reported by @alecpl on 25 Jan 2013 09:04 UTC as Trac ticket #1488931

The new browser window opened for message preview or compose page is very small in Chrome/Chromium.

Migrated-From: http://trac.roundcube.net/ticket/1488931

[rcubetrac]

Comment by @alecpl on 25 Jan 2013 10:32 UTC

http://code.google.com/p/chromium/issues/detail?id=123109 There's no issue when we remove moveTo() call in open_window().

[rcubetrac]

Comment by @alecpl on 25 Jan 2013 10:50 UTC

Fixed in 01fc82d

[rcubetrac]

Status changed by @alecpl on 25 Jan 2013 10:50 UTC

new => closed

[rcubetrac]

Comment by yoni on 28 Apr 2013 05:53 UTC

This problem still exists in Chrome Version 26.0.1410.65. When composing a new window message, the copose window opens on the top left of the browser in a very narrow (read small) popup. It has to be manually resized every time to compose a message. Please see image below:

[[Image(http://i.imgur.com/iKHicZL.png)]]

[rcubetrac]

Status changed by yoni on 28 Apr 2013 05:53 UTC

closed => reopened

[rcubetrac]

Milestone changed by @alecpl on 28 Apr 2013 06:44 UTC

0.9-rc => 0.9.1

[rcubetrac]

Comment by @alecpl on 28 Apr 2013 07:10 UTC

I'm unable to reproduce in Chromium 25. This definitely is a broken Chrome. Please, find open_window() function in app.js file and try with bigger timeout at the end of this function. Let's see if changing 100 to 1000 will help. You can try also commenting out the whole window.setTimeout(....) line. If this is related to buggy moveTo() in Chrome, maybe we should just get rid of it (for Chrome only).

[rcubetrac]

Comment by yoni on 29 Apr 2013 03:29 UTC

Doubling the timeout at the end of the open_window() function in app.js did it (200).

[rcubetrac]

Comment by @alecpl on 29 Apr 2013 07:22 UTC

We can increase the value. However, I'm thinking that maybe we don't need this moveTo() thing at all. In my system opened windows are always centered without this line of code (tested with Firefox, Opera and Chrome).

[rcubetrac]

Comment by @thomascube on 29 Apr 2013 09:46 UTC

I can reproduce the bug on Chrome Mac and both, increasing the timeout as well as skipping the entire moveto command fix it. The window is placed correctly on the screen without the moveto(). I'll do some final testing with IE but I also tend to remove it if it's not really needed by some browser.

[rcubetrac]

Comment by @thomascube on 29 Apr 2013 09:53 UTC

Removed the moveTo() call in commit [backported in 5bc49fa(b620b1d],).

[rcubetrac]

Status changed by @thomascube on 29 Apr 2013 09:53 UTC

reopened => closed