RoundCube steals focus in app.js

[rcubetrac]

Reported by David Adam zanchey on 9 Oct 2006 15:51 UTC as Trac ticket #1484069

Some users have complained that RoundCube steals focus from other applications when an operation completes.

This is due to line 178 of app.js (in revision 358):

177         // focus this window
178         window.focus();

There is a school of thought which says that acquiring focus for anything except system-critical events is inelegant UI. I would like to suggest that RoundCube avoids grabbing focus unless absolutely essential, by removing lines 177-178.

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

[rcubetrac]

Comment by seansan on 16 Feb 2008 22:20 UTC

Decide in 1.1 (fix/close)

[rcubetrac]

Milestone changed by seansan on 16 Feb 2008 22:20 UTC

=> 0.1.1

[rcubetrac]

Comment by @till on 22 Mar 2008 13:19 UTC

Hehehe... We are so evil. :P

[rcubetrac]

Comment by @till on 31 Mar 2008 19:40 UTC

Unless someone supplies a patch, I'll postpone it!

[rcubetrac]

Milestone changed by @till on 31 Mar 2008 19:40 UTC

0.1.1 => later

[rcubetrac]

Milestone changed by @alecpl on 2 Sep 2008 06:42 UTC

later => 0.2-beta

[rcubetrac]

Comment by tensor on 16 Sep 2008 08:28 UTC

Replying to till:

Unless someone supplies a patch, I'll postpone it!

There are several places where RC grabs focus. Shall the patch just remove them?

[rcubetrac]

Comment by robin on 17 Sep 2008 09:23 UTC

RoundCube only grabs focus in one place (app.js), but there are 14 occurances of window.focus() in tiny_mce. I think we can keep them.

Index: program/js/app.js
===================================================================
--- program/js/app.js   (revision 1811)
+++ program/js/app.js   (working copy)
@@ -214,12 +214,6 @@

         this.set_page_buttons();

-        // focus main window
-        if (this.env.framed && window.parent)
-          window.parent.focus();
-        else
-          window.focus();
-
         // init message compose form
         if (this.env.action=='compose')
           this.init_messageform();

[rcubetrac]

Comment by robin on 18 Sep 2008 11:47 UTC

Fixed in cd6422ba.

[rcubetrac]

Status changed by robin on 18 Sep 2008 11:47 UTC

new => closed