Bug 1042910 - [gaia-components] Fix stopPropagation in attach.js

You want that feature?
On 29 Jul 2014 16:12, "justindarc" <notifications@github.com> wrote:

> In index.js:
>
> > -          if (out === false) { return e.stopPropagation(); }
> > +
> > +          // If the callback returns `false`,
> > +          // call `preventDefault` and
> > +          // `stopPropagation` on the event
> > +          // to preserve expected behavior.
> > +          if (fn.call(ctx || el, e, el) === false) {
> > +            e.preventDefault();
> > +            e.stopPropagation();
> > +            return;
> > +          }
> > +
> > +          // If one of the delegates called
> > +          // `stopPropagation` on the event,
> > +          // stop calling remaining delegates.
> > +          if (e.cancelBubble) { return; }
>
> I think we still need to check for return false because a delegate could
> technically still do that to stopPropagation and preventDefault at the same
> time.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gaia-components/attach/pull/1/files#r15529197>.
>