(In reply to Anthony Ramine [:nox] from comment #95)
> What should I do to get this fixed?

Your patch does:

>               } else if (this.AppConstants.platform == "macosx") {

and

>               this.AppConstants.platform == "macosx") {


It also hardcodes 'accel' to always mean 'ctrl'.

Instead, leave the accel handling as-is, don't check for the OS, but add
a preference line in all.js, e.g. for
"browser.urlbar.accelenter.newtab". Use #ifdefs for that pref and, on
Windows, set it to false, and on everything else, to true.

Then instead of checking the OS in your patch, check the preference, and
use that to decide what to do.

-- 
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to firefox in Ubuntu.
https://bugs.launchpad.net/bugs/66566

Title:
  Inconsistent shortcuts for new tab

Status in Mozilla Firefox:
  Fix Released
Status in firefox package in Ubuntu:
  Won't Fix
Status in firefox-3.0 package in Ubuntu:
  Won't Fix
Status in firefox-3.5 package in Ubuntu:
  Won't Fix

Bug description:
  There are currently too many shortcuts when opening a link in a new
  tab:

   - location bar: <ALT> + <ENTER>
   - GO button: <CTRL> + click
   - search bar: <CTRL> + <ENTER>
   - SEARCH button <ALT> + click
   - links: <SHIFT> + click
   - menu bar: <CTRL> + click
   - BACKWARD/FORWARD button: <CTRL> + click

  This is one of the reasons I'm keeping with Opera. There it is always
  the shift key and I have to remember only one key.

  Thanks for reading!

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

-- 
Mailing list: https://launchpad.net/~desktop-packages
Post to     : desktop-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~desktop-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to