Opened 10 months ago

Closed 10 months ago

Last modified 9 months ago

#1488620 closed Bugs (fixed)

Focus issues on IE8 after fix 3c7346a for ticket 1488600

Reported by: OHosNL Owned by:
Priority: 2 Milestone: 0.8.2
Component: User Interface Version: 0.8.1
Severity: major Keywords: focus list.js IE8
Cc: agueijo@…

Description

Using 0.8.1 with IE8, IE the IE8 window will drop to the background if any other windows are open. Unlike previously stated on the forum, the window does not actually lose focus, so you have to click whatever window that came to the front and then the IE-window to get back to it.

Reverting the fix that was applied with commit 3c7346a solves the issue.
That commit was in response to 1488600

Change History (13)

comment:1 Changed 10 months ago by alec

  • Milestone changed from later to 0.9-beta

comment:2 follow-up: Changed 10 months ago by alec

I'm unable to reproduce using IE8 on Windows7. What system do you have?

comment:3 Changed 10 months ago by LonelyPixel

Same here. It makes Roundcube pretty unusable on IE8/XP. I'm now going to try the indicated fix above.

comment:4 in reply to: ↑ 2 Changed 10 months ago by OHosNL

Replying to alec:

I'm unable to reproduce using IE8 on Windows7. What system do you have?

My apologies, I thought noone would use IE8 unless they're on Windows XP. I've confirmed the problem on two instances of Windows XP, an old laptop (where I stumbled upon this problem by chance) and in a virtualbox.

comment:5 Changed 10 months ago by lumix1991

Same problem here.

I am using Windows 7 Ultimate x64 with Internet Explorer 9 (latest updated version). Even when I use the Browsermode: IE8 in internet explorer the problem exists.

This should be fixed FAST because you can't use Roundcube now to read mail with Internet Explorer.

Last edited 10 months ago by lumix1991 (previous) (diff)

comment:6 Changed 10 months ago by myoreo5

  • Priority changed from 5 to 2
  • Severity changed from normal to major

Same problem here.

Win7 IE 9.

I have noticed that if you minimize the window that comes up the problem goes away.

comment:7 Changed 10 months ago by agueijo

  • Cc agueijo@… added

comment:8 Changed 10 months ago by gulenler

Same problem also in IE 9 with safe mode on.

Win 7 Professional version 6.1.7601 Service Pack 1 Build 7601
IE version 9.0.8112.16421

comment:9 follow-ups: Changed 10 months ago by jeffreydev

Suggested a fix: https://github.com/roundcube/roundcubemail/pull/23

This can also be fixed using the following:

cd /var/www/html/roundcube/program/js/
cp list.js list.js.bak
sed -i 's/$(document.activeElement).blur();/ if(document.activeElement) { if(document.activeElement.nodeName.toLowerCase() != "body") { document.activeElement.blur(); } } else { $("*:focus", window).blur(); } /g' list.js

comment:10 in reply to: ↑ 9 Changed 10 months ago by agueijo

This works for my.

Thanks, Agu

Replying to jeffreydev:

Suggested a fix: https://github.com/roundcube/roundcubemail/pull/23

This can also be fixed using the following:

cd /var/www/html/roundcube/program/js/
cp list.js list.js.bak
sed -i 's/$(document.activeElement).blur();/ if(document.activeElement) { if(document.activeElement.nodeName.toLowerCase() != "body") { document.activeElement.blur(); } } else { $("*:focus", window).blur(); } /g' list.js

comment:11 in reply to: ↑ 9 Changed 10 months ago by lumix1991

Replying to jeffreydev:

Suggested a fix: https://github.com/roundcube/roundcubemail/pull/23

This can also be fixed using the following:

cd /var/www/html/roundcube/program/js/
cp list.js list.js.bak
sed -i 's/$(document.activeElement).blur();/ if(document.activeElement) { if(document.activeElement.nodeName.toLowerCase() != "body") { document.activeElement.blur(); } } else { $("*:focus", window).blur(); } /g' list.js

Worked here also! I hope that they will fix this in the next Roundcube Release.

comment:12 Changed 10 months ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [e263994a]

comment:13 Changed 9 months ago by alec

  • Milestone changed from 0.9-beta to 0.8.2
Note: See TracTickets for help on using tickets.