Closed
Bug 310651
Opened 19 years ago
Closed 18 years ago
Full URL not displayed after cmd/ctrl/shift-enter and switching tabs
Categories
(Firefox :: Address Bar, defect)
Firefox
Address Bar
Tracking
()
RESOLVED
FIXED
People
(Reporter: Mardak, Assigned: Mardak)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Seems similar to bug 287996. Also regressed with the checkin for bug 272002 on
20050215. Tested on Firefox 1.4 20050930 mac.
Steps:
Type in the domain to complete with www and .com
Press cmd/ctrl-enter and switch to a new tab and back quickly
Location bar shows the domain without www and .com
Comment 1•19 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20050930
Firefox/1.6a1 ID:2005093016
Works for me. The URL is updated once the page loads.
Assignee | ||
Comment 2•19 years ago
|
||
I guess I should have put more emphasis on *quickly* indicating that it would
display the correct url when the page loads.
You can try it with "agadak" shift-enter.. it should always take a couple
seconds to load (sleeps 1 second before sending the page).
Assignee | ||
Comment 3•19 years ago
|
||
A patch for this issue would probably be similar to the one that fixed bug 287996?
From patch comment.. "Completing a result should simulate the user typing the
result, so fire an input event."
Flags: blocking1.8rc1?
Comment 4•19 years ago
|
||
can you all figure out when this regressed? thanks.
Flags: blocking1.8rc1? → blocking1.8rc1-
Assignee | ||
Comment 5•19 years ago
|
||
This regressed with the checkin for bug 272002 on 20050215. (Build on 20050215
still works while the new build on 20050216 fails.)
Comment 6•19 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051012
Firefox/1.6a1 ID:2005101212
I still can't reproduce this. Are you testing this in a clean profile? Could be
Mac-only, too...
Keywords: qawanted
Assignee | ||
Comment 7•19 years ago
|
||
I can reproduce this with a 20051013 windows build as well as mac on clean profiles.
Perhaps it has something to do with directly setting the autocomplete's value
property directly?
http://lxr.mozilla.org/mozilla1.8/source/browser/base/content/browser.js#2155
Brian recently added a setTextValue (private?) method to the autocomplete that
triggers a UI event to help fix bug 287996. Perhaps instead of doing something
like..
gURLBar.value = getShortcutOrURI(url, aPostDataRef);
it should do something similar to a setTextValue.
Updated•19 years ago
|
Keywords: qawanted
Summary: full URL not displayed after cmd/ctrl/shift-enter and switching tabs → Full URL not displayed after cmd/ctrl/shift-enter and switching tabs
Comment 8•19 years ago
|
||
==> firefoz
Component: Location Bar → Location Bar and Autocomplete
Product: Core → Firefox
Comment 9•19 years ago
|
||
*** Bug 315075 has been marked as a duplicate of this bug. ***
Updated•19 years ago
|
Assignee: bryner → nobody
QA Contact: location.bar
Assignee | ||
Comment 10•18 years ago
|
||
One possibility is to update gBrowser.userTypedValue, but I suppose the bigger issue is why doesn't it get updated in this situation?
Another way to make this bug go away is to trigger an input UIEvent on gURLBar.
Attachment #240052 -
Flags: review?(gavin.sharp)
Comment 11•18 years ago
|
||
*** Bug 361538 has been marked as a duplicate of this bug. ***
Comment 12•18 years ago
|
||
Comment on attachment 240052 [details] [diff] [review]
Update gBrowser's userTypedValue
Sorry for taking so long to review this, Edward. One little nit: add a newline before the added lines. I can do that and check this in for you.
Attachment #240052 -
Flags: review?(gavin.sharp) → review+
Updated•18 years ago
|
Whiteboard: [checkin needed]
Updated•18 years ago
|
Assignee: nobody → edilee
Comment 13•18 years ago
|
||
Attachment #240052 -
Attachment is obsolete: true
Comment 14•18 years ago
|
||
mozilla/browser/base/content/browser.js 1.773
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
You need to log in
before you can comment on or make changes to this bug.
Description
•