Comment on attachment 8782846
patch v4

Review of attachment 8782846:
-----------------------------------------------------------------

Meh. r+ I guess? I'm not sure why this is "simpler", but obviously it
works, so whatever.

As far as the "window" case is concerned, you're explicitly regressing
bug 644186, but you reviewed that and it never had ui-review, so up to
you if you think you need it here.

::: browser/base/content/browser.js
@@ -7839,5 @@
>      }
>    }
>  };
>  
> -function BrowserOpenNewTabOrWindow(event) {

This method is referenced by a number of add-ons including but not
limited to tabmixplus. We probably need to list it in the compat notes.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/572074

Title:
  New tabs open to the right of all existing tabs instead of opening
  next to the current tab

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/572074/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to