Comment author: gibson042

This is exactly analogous to #13353... I'm starting to think we should generalize and incorporate that  solution after all.

Comment author: dmethvin

Way too much code though. I was thinking that if the user passed data that we use the "standard" trigger path rather than native events. It passes the data at the expense of some non-standard event behavior, but I think people are probably misusing the event system most of the time when this happens.

Comment author: gibson042

That'd probably be acceptable for focus, but click handlers would see different checked states on checkbox and radio inputs when the event was provided trigger data—a definite no-go.

Comment author: dmethvin

So, to the OP, are you using this to initialize on page load? Could you use .triggerHandler() instead, or are you really trying to focus the element as well?

Comment author: mail@…

I was trying to distinguish between a) clicking or tabbing into the input field and b) using the .trigger('focus') function somewhere else. Nothing on page load.

Comment author: dmethvin

#13590 is a duplicate of this ticket.

Comment author: dmethvin

#13711 is a duplicate of this ticket.

Comment author: anonymous

triggerHandler is not really an acceptable workaround since it won't actually focus, blur, etc...

Comment author: dmethvin

It is a fine workaround. Just do the indicated DOM action afterwards.

var $x = $("#aninput");
$x.triggerHandler("focus");
$x[0].focus();

Comment author: anonymous

People from the core team certainly know this already, but this arises from:

focus: {
  // Fire native event if possible so blur/focus sequence is correct
  trigger: function() {
    if ( this !== safeActiveElement() && this.focus ) {
      this.focus();

and

blur: {
  trigger: function() {
  if ( this === safeActiveElement() && this.blur ) {
    this.blur();

Comment author: anonymous

Won't this trigger the handlers twice?

Replying to dmethvin:

It is a fine workaround. Just do the indicated DOM action afterwards.

var $x = $("#aninput");
$x.triggerHandler("focus");
$x[0].focus();

Comment author: osmestad

I also have this issue and it is still valid with latest 1.x and 2.x, here is a jsFiddle for what I'm doing:  http://jsfiddle.net/2dVMa/

My problem with using triggerHandler is that I am trying to avoid running the code in my 'focus' handler when setting focus programatically (with 'trigger'). (Only workarounds I can think of would be unbinding, triggering focus and then rebinding, or reverting to 1.8... )

Others with same issue:  http://forum.jquery.com/topic/extra-arguments-passed-to-trigger-doesn-t-reach-blur-event-handler-when-element-has-focus

Browser: Chrome 27, OS: OS X 10.8.4

Comment author: dmethvin

#14658 is a duplicate of this ticket.

Comment author: dmethvin

#14829 is a duplicate of this ticket.

For 'focus', 'blur' and 'click' events, it fires native event in certain condition and when
calling native event, currently there is no way to pass through trigger's data to event handler.

Well, is not the best solution but I though to use special event hook to handle that.
Need to modify trigger to hold the data, and add handle method to invoke event handler function
(obviously it can be generalized to cover other events too).

What do you think guys?

        focus: {
            // Fire native event if possible so blur/focus sequence is correct
            trigger: function() {
                if ( this !== safeActiveElement() && this.focus ) {

                    // Check if triggered with data
                    if( arguments.length > 1 ) {
                        jQuery.event.special.focus.triggerData = jQuery.makeArray(arguments).splice(1);
                    }

                    this.focus();
                    return false;
                }
            },
            delegateType: "focusin",
            handle: function( event ) {
                var param = [ event ],
                    data = jQuery.event.special.focus.triggerData;

                if( data ) {
                    param = param.concat( data );
                }

                event.handleObj.handler.apply( this, param );
                delete jQuery.event.special.focus.triggerData;
            }
        }

That solution is at risk for introducing strange behavior flips between the "no arguments" and "arguments" cases, and is not reentrant safe. I don't see us using such an approach.

Agreed. In retrospect, allowing users to pass data has caused a lot of issues and this is one of them. Focus management is highly flaky in browsers already, it doesn't take much to cause problems.

@timmywil Do we still want it for 3.0.0? There wasn't a lot of progress and we need to cut corners.

Moving to 3.1.0.

I'm dealing with the same situation as described in this comment.

...
My problem with using triggerHandler is that I am trying to avoid running the code in my 'focus' handler when setting focus programatically (with 'trigger').
...

Is there a proper way to solve this?

Welp, I just ran into this. Tried to handle a blur event only if triggered with a certain parameter, but I guess they just don't get passed. There's a use case, at least!

@mgol

There is an another way to show additional data when focus event triggered

Please check my fiddle - https://fiddle.jshell.net/426vdp05/1/

I am attaching screen shot for same

@PurvaA You're not triggering the focus event by yourself in your fiddle and you don't pass any data so this doesn't seem relevant to this issue.

This is still broken. Is this issue being tracked in one of the future milestones?

Yes, the ticket is still open and the milestone is 4.0.0, scroll to the top of this ticket and it should say that. If you'd like to submit a PR we can review.

This might be an acceptable workaround to prevent firing the handler twice as mentioned in #1741 (comment):

var $x = $("#aninput");
$x.triggerHandler("focus", ['extra', 'data']);
$x[0].focus();

$x.on("focus", function (e, extra, data) {
    if (e.originalEvent) return;
    console.log(extra, data);
});

Changing the milestone, we'll fix that in 3.4.0.