WordPress.org

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#12424 closed defect (bug) (fixed)

Opera + Tabbing in Editors does not live happily

Reported by: dd32 Owned by: azaozz
Milestone: 3.0 Priority: normal
Severity: normal Version: 3.0
Component: JavaScript Keywords: has-patch needs-testing
Focuses: Cc:

Description

Opera behaves differently from most modern browsers, It seems to ignore tab-canceling requests.

The result is that a Tab is inserted, and Opera moves focus to the next element.

One solution to this is to add a timeout to set the focus back to the textarea in a few milliseconds.

See Patch

Attachments (2)

12424.diff (14.3 KB) - added by dd32 8 years ago.
opera-tab.12424.diff (1.1 KB) - added by filosofo 8 years ago.

Download all attachments as: .zip

Change History (6)

@dd32
8 years ago

#1 @dd32
8 years ago

  • Keywords has-patch needs-testing added

This is potentially buggy though, and has been suggested this could be a bug in Opera. (But has been around awhile)

Should probably be tied specifically to Opera if commited..

#2 @filosofo
8 years ago

Try my patch. It's a less hacky approach.

Also, I took out the unnecessary "textarea" part of the query string. If you're querying for an element by ID, it only slows things down to add additional info for jQuery to parse.

And I added checks for two of the event methods not used by IE. Even if jQuery is doing something to catch the error (I don't know), better for performance not to throw it in the first place.

#3 @dd32
8 years ago

Also, I took out the unnecessary "textarea" part of the query string.

I was thinking the same thing, But wasnt sure if it had been written that way for some reason.

The onblur handler should work, that was the other suggested alternative to the timeout setfocus method.

I'll test that out this evening.

#4 @dd32
8 years ago

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

(In [13548]) Tabbing in Editors: Prevent certain browsers from Tabbing out of the field. Props filosofo. Fixes #12424

Note: See TracTickets for help on using tickets.