Closed
Bug 348472
Opened 18 years ago
Closed 17 years ago
Command-Option-F should select the web search box.
Categories
(Firefox :: Keyboard Navigation, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: web, Assigned: dev)
References
(Depends on 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
1.45 KB,
patch
|
asaf
:
review+
beltzner
:
ui-review+
damons
:
approval1.9+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1b1) Gecko/20060710 Firefox/2.0b1
Build Identifier: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US; rv:1.8.1b1) Gecko/20060710 Firefox/2.0b1
Command-Option-F is a standard shortcut key for selecting the search bar in many Mac applications (e.g. Safari, iTunes, etc.). It should be supported in Firefox on the Mac in addition to Command-K.
Reproducible: Always
Assignee | ||
Comment 2•17 years ago
|
||
The patch adds Option+Command+F as a secondary shortcut key for the web search bar.
Attachment #294547 -
Flags: ui-review?(beltzner)
Comment 3•17 years ago
|
||
Comment on attachment 294547 [details] [diff] [review]
patch
Mike, if you want this to land now, you'd have to use the entity used for Accel+F, strings are frozen at this point.
Assignee | ||
Comment 4•17 years ago
|
||
Ace: Thanks for the review! Patch update coming shortly...
Assignee | ||
Comment 5•17 years ago
|
||
The patch *should* work, but isn't - probably due to some new coca keyboard shortcut bugs?
Oddly enough: if you hit CMD+F to open the find bar, the patch works perfectly. Without the find bar open, it doesn't.
Attachment #294547 -
Attachment is obsolete: true
Attachment #302555 -
Flags: ui-review?(beltzner)
Attachment #302555 -
Flags: review?(mano)
Attachment #294547 -
Flags: ui-review?(beltzner)
Comment 6•17 years ago
|
||
Comment on attachment 302555 [details] [diff] [review]
v1.2
ui-r=beltzner, supporting the user's memory for keyboard shortcuts on the OS seems right to me
Attachment #302555 -
Flags: ui-review?(beltzner) → ui-review+
Comment 7•17 years ago
|
||
Comment on attachment 302555 [details] [diff] [review]
v1.2
r=mano.
Attachment #302555 -
Flags: review?(mano)
Attachment #302555 -
Flags: review+
Attachment #302555 -
Flags: approval1.9?
Comment 8•17 years ago
|
||
Comment on attachment 302555 [details] [diff] [review]
v1.2
a1.9+=damons
Attachment #302555 -
Flags: approval1.9? → approval1.9+
Assignee | ||
Updated•17 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Updated•17 years ago
|
Assignee: nobody → dev
Status: ASSIGNED → NEW
Comment 9•17 years ago
|
||
mozilla/browser/base/content/browser-sets.inc 1.112
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 10•17 years ago
|
||
Verified with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9b4pre) Gecko/2008022504 Minefield/3.0b4pre ID:2008022504
Status: RESOLVED → VERIFIED
Hardware: Macintosh → All
Version: unspecified → Trunk
Comment 11•17 years ago
|
||
It stopped working with the latest nightly.
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9pre) Gecko/2008041504 Minefield/3.0pre
Comment 12•17 years ago
|
||
I've tested several builds from http://hourly-archive.localgho.st/mac.html
and it stopped working between build 20080414_2047 and 20080414_2130
http://bonsai.mozilla.org/cvsquery.cgi?module=PhoenixTinderbox&date=explicit&mindate=1208231220&maxdate=1208233799
looks like a regression by bug 359638
Comment 13•17 years ago
|
||
Please file a new bug and add bug 359638 to the blocking list. It has nothing to do with this bug. Thanks.
Comment 14•17 years ago
|
||
(In reply to comment #13)
> Please file a new bug and add bug 359638 to the blocking list. It has nothing
> to do with this bug. Thanks.
>
Filed Bug 429160. Sorry for the bug spam.
You need to log in
before you can comment on or make changes to this bug.
Description
•