Performance: changePage() searches the entire document for :focus elements

[jblas]

In changePage() I see this:

$( window.document.activeElement || "" ).add( "input:focus, textarea:focus, select:focus" ).blur();

This will cause JQM to search the entire document every time we change pages just to release keyboard focus. Instead, we should probably be using a live("focus") handler and tracking the currently focused element so we don't have to crawl the document.

[jblas]

I'll try to address this as part of the path navigation re-working I'm doing.

[jblas]

Related changePage() performance issue:

#1562

[StevenBlack]

Hey Kin, looking into this, unless I'm missing something window.document.activeElement is determinate. It's a reference to the page's h1 element typically. So this line does no document searching at all.

On the other hand, if you are actually referring to refocus(), which looks to be the other facet of this, then I understand :-)

[jblas]

But the add() searches the document doesn't it?

[StevenBlack]

Doh! Right you are. I was looking at the wrong alternate signatures of .add(). http://api.jquery.com/add/

Nevermind. :-)

[StevenBlack]

Something worth speed-testing: I bet $(":input:focus").blur() may be streets faster since, internally, the :input is regex-driven.
https://github.com/jquery/sizzle/blob/master/sizzle.js#L665-667

[staabm]

jsperf comparison?

[toddparker]

Is this still a concern? Already fixed?

[jblas]

We're still searching for :focus elements, so this should stay open.

[jaspermdegroot]

See also #1855

[jaspermdegroot]

Related to #4340

[ldeluca]

Hey @jaspermdegroot @jblas This issue is super old (has it really been 3 years since 2011?!?) Can you confirm whether this is still an issue or if it has been fixed with new releases of JQM? Thanks!

[Ruffio]

@ldeluca Even though this is super old, the issue/performance loss is still valid. Seaching jquery.mobile-1.4.5.js line 5573 reveals that this is still an issue:

// Kill the keyboard.
// XXX_jblas: We need to stop crawling the entire document to kill focus.
// Instead, we should be tracking focus with a delegate()
// handler so we already have the element in hand at this
// point.