Skip to main content

Bug Tracker

Side navigation

#10809 closed bug (fixed)

Opened November 17, 2011 12:30AM UTC

Closed May 28, 2012 04:58PM UTC

Last modified May 29, 2012 12:07AM UTC

incorrect test using ".activeElement" in the :focus pseudo-class resolver

Reported by: diego.perini@gmail.com Owned by: timmywil
Priority: low Milestone: 1.8
Component: selector Version: 1.6.4
Keywords: Cc:
Blocked by: Blocking:
Description

The "document.activeElement" does not completely reflect a "focussed" element in every situation/environment.

Newer browser have a "document.hasFocus()" method to make sure a given document contains the currently "focussed" element.

There can only be one focussed element in the browser. So the "focus" is bound to the current editing view.

So if you are typing in a text box in one window, that element is both "active" and "focussed". As soon as you switch window and you type in a different text box the previous element is still "active" but not "focussed".

Use the ".activeElement" property to provide support for the CSS2 ":active" pseudo-class which is currently missing. Just rename current "focus" method to "active" and add a new "focus" method that considers newer browsers implementing the ".hasFocus()" method.

Diego

Attachments (0)
Change History (4)

Changed November 17, 2011 01:02AM UTC by timmywil comment:1

component: unfiledselector
milestone: None1.8
owner: → timmywil
priority: undecidedlow
status: newassigned

I probably should have filed a ticket for this myself. I have this change on my list for the Sizzle revamp in 1.8. Thanks for filing Diego.

If it's a major concern, we can push this ahead to 1.7.x, but imo, I don't think that's necessary.

Changed May 28, 2012 04:58PM UTC by timmywil comment:3

resolution: → fixed
status: assignedclosed

Changed May 29, 2012 12:07AM UTC by timmywil comment:4

#10569 is a duplicate of this ticket.