(In reply to :Gijs Kruitbosch (PTO recovery mode) from comment #70)
> (In reply to Dão Gottwald [:dao] from comment #69)
> > >+ let sourceNotKeyEvent = !event.sourceEvent || event.sourceEvent.target.localName != "key";
> > >+ let relatedToCurrent = (where == "tab" || where == "tabshifted") && sourceNotKeyEvent;
> > >+ openUILinkIn(BROWSER_NEW_TAB_URL, "tab", {relatedToCurrent,
> > >+ inBackground: where == "tabshifted"});
> >
> > I'm somewhat confused by this. Why are you not passing through "tabshifted"
> > as openUILinkIn's 'where' argument?
>
> I literally just used the suggestion you gave in comment #65. I can't just
> pass "tab" because in some cases where == "current".
(In reply to :Gijs Kruitbosch (PTO recovery mode) from comment #70) nt.target. localName != "key"; BROWSER_ NEW_TAB_ URL, "tab", {relatedToCurrent,
> (In reply to Dão Gottwald [:dao] from comment #69)
> > >+ let sourceNotKeyEvent = !event.sourceEvent || event.sourceEve
> > >+ let relatedToCurrent = (where == "tab" || where == "tabshifted") && sourceNotKeyEvent;
> > >+ openUILinkIn(
> > >+ inBackground: where == "tabshifted"});
> >
> > I'm somewhat confused by this. Why are you not passing through "tabshifted"
> > as openUILinkIn's 'where' argument?
>
> I literally just used the suggestion you gave in comment #65. I can't just
> pass "tab" because in some cases where == "current".
I meant, I can't just pass |where|.