Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Windows resize in internet explorer #4206

Closed
rcubetrac opened this issue May 19, 2013 · 10 comments
Closed

Windows resize in internet explorer #4206

rcubetrac opened this issue May 19, 2013 · 10 comments

Comments

@rcubetrac
Copy link

Reported by dennis1993 on 19 May 2013 11:03 UTC as Trac ticket #1489114

Hello,

I've have found a bug, which is online in the Internet Explorer.

  1. set internet explorer in full screen
  2. Go to "Settings" => "Message Creation"
  3. Uncheck the first box "Compose messages in a new window"
  4. Go back to Mail an klick on "New Mail"

Now the window is reduced and does not remain in full-screen mode.

You have now 2 times click on full screen in the browser to him to have great again.

Thanks in advance.

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

@rcubetrac
Copy link
Author

Comment by @alecpl on 19 May 2013 13:35 UTC

This is by design. We define such windows' size, because we don't want them to be too big. However, maybe we should open compose window the same way we do with e.g. attachment preview. So, browser would be able to open the page in a tab. We'll decide before 1.0 release.

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 19 May 2013 13:35 UTC

later => 1.0-beta

@rcubetrac
Copy link
Author

Comment by @alecpl on 24 May 2013 06:44 UTC

I also observed that opened windows are often bigger than the current window. That is something I don't like and it can be resolved the same way.

@rcubetrac
Copy link
Author

Comment by @alecpl on 24 May 2013 07:46 UTC

Another thing. Message print page is opened in standard window (tab). So, for example, when you open a message in new window and the click Print, the page will be opened in browser tab (Firefox). That's why I think all popups should be opened without specifying size so they can be handled by browser in a standard way (e.g. by opening a new tab).

@rcubetrac
Copy link
Author

Comment by @thomascube on 30 May 2013 07:40 UTC

Replying to alec:

Another thing. Message print page is opened in standard window (tab). So, for example, when you open a message in new window and the click Print, the page will be opened in browser tab (Firefox). That's why I think all popups should be opened without specifying size so they can be handled by browser in a standard way (e.g. by opening a new tab).

Popups are more meant to be "modal" windows without the browser's toolbar and menubar.
For printing and viewing message source, we want to user to be able to choose to print or to save the contents and therefore a full browser window makes sense to me.

@rcubetrac
Copy link
Author

Comment by @thomascube on 30 May 2013 07:45 UTC

Replying to alec:

This is by design. We define such windows' size, because we don't want them to be too big. However, maybe we should open compose window the same way we do with e.g. attachment preview. So, browser would be able to open the page in a tab. We'll decide before 1.0 release.

I agree that attachment previews should open in the same way as compose and message view "in new windows". But that doesn't mean to not open them as popups. If somebody doesn't want compose or message view in popups she can simply uncheck these options and open them in a tab with ctrl-click.

For a better understanding, we should maybe replace "new window" with "popup window" in the labels of those options.

@rcubetrac
Copy link
Author

Comment by @alecpl on 30 May 2013 13:06 UTC

I've red the issue description once again and now I understand and can confirm the issue.

@rcubetrac
Copy link
Author

Comment by @alecpl on 30 May 2013 13:22 UTC

Fixed in 99e27c5.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 30 May 2013 13:22 UTC

new => closed

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 30 May 2013 13:22 UTC

1.0-beta => 0.9.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant