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.

Sign up for GitHub

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

Jump to bottom

in opera the keypress event does not fire #4011

Closed
rcubetrac opened this issue Nov 20, 2012 · 5 comments
Closed

in opera the keypress event does not fire #4011

rcubetrac opened this issue Nov 20, 2012 · 5 comments
Labels
bug C: Client Scripts
Milestone
0.9-beta

Comments

@rcubetrac
Copy link

Reported by bes internal on 20 Nov 2012 09:38 UTC as Trac ticket #1488823

In the opera, delete not delete messages at msglist

Opera generate keydown,keyup event instead keydown,keypress,keyup for delete and other function keys

In js code a lot of hacks on this. Need to review again

Migrated-From: http://trac.roundcube.net/ticket/1488823
@rcubetrac
Copy link
Author

Comment by @alecpl on 20 Nov 2012 11:17 UTC

I made a small refactoring, see attached patch. It looks that everything is ok in Firefox and Opera. Need to check in other browsers.

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 20 Nov 2012 11:17 UTC

later => 0.9-beta

@rcubetrac
Copy link
Author

Comment by @alecpl on 20 Nov 2012 19:54 UTC

Tested with IE. Patch applied in 17a8fb3.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 20 Nov 2012 19:54 UTC

new => closed

@rcubetrac
Copy link
Author

Comment by bes internal on 21 Nov 2012 08:49 UTC

Thanks. At 0.8.4 works too

@rcubetrac rcubetrac added this to the 0.9-beta milestone Mar 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug C: Client Scripts
Projects
None yet
Milestone
0.9-beta
Development

No branches or pull requests
1 participant
@rcubetrac