Closed
Bug 60498
Opened 24 years ago
Closed 23 years ago
[rfe] Bookmark custom keywords should work in "open web location" dialog
Categories
(SeaMonkey :: UI Design, enhancement, P2)
SeaMonkey
UI Design
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.1
People
(Reporter: jruderman, Assigned: bugzilla)
References
Details
Attachments
(2 files)
25.62 KB,
patch
|
Details | Diff | Splinter Review | |
2.63 KB,
patch
|
Details | Diff | Splinter Review |
I have this in my bookmarks with the keyword "rfc": http://community.roxen.com/developers/idocs/rfc/rfc%s.html "rfc 1459" works in the location bar but not in the "open web location" dialog (ctrl-l). Combined with bug 19446 (need keyboard shortcut to focus location bar), this bug makes the keyword feature less useful than it could be. See also bug 38865, "[RFE] URL autocomplete in Open Web Location dialog".
Comment 1•24 years ago
|
||
According to the instructions here, it is not expected to work this way - http://home.netscape.com/bookmark/bestof/customize.html "A bookmark's keyword can be typed into the location field to go to the bookmarked site."
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → INVALID
Reporter | ||
Comment 2•24 years ago
|
||
I still think this is a bug, but I'll reopen it as an rfe since the docs aren't clear. It seems to me that typing into ctrl-L should have the same effect as typing into the location bar. Is there any reason keywords shouldn't work in the dialog?
Severity: normal → enhancement
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Summary: Bookmark keywords don't work in "open web location" dialog → [rfe] Bookmark keywords should work in "open web location" dialog
Assignee | ||
Comment 3•24 years ago
|
||
This is easy to fix, provided it's a good idea UI-wise. Sounds fine to me. Matthew?
Assignee: ben → blakeross
Status: REOPENED → NEW
Comment 4•24 years ago
|
||
Absolutely! As far as I can imagine, the address field in the Open Location dialog should offer *at least* the same features as the address field in the browser window. So if keywords work in the latter, they should work in the former as well.
OS: Windows 98 → All
Hardware: PC → All
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Priority: P3 → P2
Target Milestone: --- → mozilla0.9
nav triage team: reassinging component to xp apps, not a beta stopper. Should open location dialog do everything that URL bar does?
Component: Bookmarks → XP Apps
Keywords: nsbeta1-
Comment 6•24 years ago
|
||
removing keyword. nsbeta1- only applies to bugs assigned to ns engineers.
Keywords: nsbeta1-
Assignee | ||
Comment 8•24 years ago
|
||
Assignee | ||
Comment 9•24 years ago
|
||
This patch looks long for this bug; it's not. I just moved openLocation.xul/js/dtd/properties while I was touching some of them, since they don't belong in /browser, since they're accessed from other apps besides Navigator (this will also allow us to add the dialog to mailnews). It also persists the last "Open in window:" setting. I'll attach an openLocation.js diff that shows what changed in that file (very little), since you can't tell from the -uN diff.
Assignee | ||
Comment 10•24 years ago
|
||
Assignee | ||
Comment 11•24 years ago
|
||
I see some arbitrary spacing issues, yes, I'll fix them. cc'ing alec for sr.
Comment 12•24 years ago
|
||
you really don't need to do all that.. all you should have to do is add persist="value" to the widget that holds the url, right? let xul persistence do all the hard work.
Assignee | ||
Comment 13•24 years ago
|
||
I don't think persist works for menulists, which is what I added here (the url remembering stuff already existed). But in any case, does that persistence work across windows and sessions? (Or do we not want that?)
Comment 14•24 years ago
|
||
yep, it works across windows and sessions (which seems like the proper behavior anyway) but if you're right that persistence doesn't work for menulists, then sr=alecf on the existing patch
Assignee | ||
Comment 15•24 years ago
|
||
Cool. I'll go ahead and fix the url case, then, and ask ben or hyatt about the menulists.
Comment 16•24 years ago
|
||
r=timeless
Assignee | ||
Updated•23 years ago
|
Target Milestone: mozilla0.9 → mozilla0.9.1
Updated•23 years ago
|
Summary: [rfe] Bookmark keywords should work in "open web location" dialog → [rfe] Bookmark custom keywords should work in "open web location" dialog
Assignee | ||
Comment 17•23 years ago
|
||
This was fixed. It still doesn't work when the dialog is invoked from Composer, but we have a separate bug on that.
Status: ASSIGNED → RESOLVED
Closed: 24 years ago → 23 years ago
Resolution: --- → FIXED
Comment 18•23 years ago
|
||
VERIFIED Fixed with 2001053104 builds. (the persist works too)
Status: RESOLVED → VERIFIED
Comment 19•23 years ago
|
||
Fixed? with 2002011103 it isn't ... no sign in dialog that it ever was. Point being: keywords in location don't work and *from IRC chat* have remained unstable. 'Sup?
Reporter | ||
Comment 20•22 years ago
|
||
Ben: works fine for me using 2002 030604 on Windows 98.
Updated•20 years ago
|
Product: Core → Mozilla Application Suite
You need to log in
before you can comment on or make changes to this bug.
Description
•