Closed
Bug 476020
Opened 15 years ago
Closed 15 years ago
Pressing Esc while having the tag autocomplete open closes the bookmarks panel
Categories
(Firefox :: Bookmarks & History, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3.6a1
People
(Reporter: whimboo, Assigned: mak)
References
Details
(Keywords: dataloss, polish, verified1.9.1)
Attachments
(1 file, 1 obsolete file)
1.54 KB,
patch
|
beltzner
:
approval1.9.1+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20090127 Shiretoko/3.1b3pre Ubiquity/0.1.5 ID:20090127021057 This bug is similar to bug 462662 but handles the Esc key. Until know we still close the dialog even when the tag auto-completion menu is open. Steps: 1. Open a page and bookmark it with a tag 2. Open another page and bookmark it too (via Ctrl/Cmd+D) 3. Selecting the tag field and enter the first letter from the tag of step 1 4. Press Esc With step 4 we only should close the auto-completion menu but not the bookmarks panel.
Assignee | ||
Comment 1•15 years ago
|
||
mh, ESC is however an acronym to interrupt an operation, something to polish but probably less negative than saving a full dialog on enter through an autoselect.
Reporter | ||
Comment 2•15 years ago
|
||
Speaking in that way it's more a dataloss issue. When someone is hitting accidentally the Esc key while having already entered another name and other tags everything is lost. Added bookmarks can easily be removed.
Keywords: dataloss
Assignee | ||
Comment 3•15 years ago
|
||
this should be enough
Comment 4•15 years ago
|
||
Comment on attachment 359659 [details] [diff] [review] patch v1.0 r=me. please add a comment describing the cases this covers.
Attachment #359659 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 5•15 years ago
|
||
Attachment #359659 -
Attachment is obsolete: true
Assignee | ||
Comment 6•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/01748c44e5db
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.2a1
Assignee | ||
Comment 7•15 years ago
|
||
Comment on attachment 359734 [details] [diff] [review] patch v1.1 minor polish, low risk
Attachment #359734 -
Flags: approval1.9.1?
Reporter | ||
Comment 8•15 years ago
|
||
Verified with: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090131 Minefield/3.2a1pre Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090201 Minefield/3.2a1pre Aakash, can you include this into the litmus test, you already have created on bug 462662? But lets wait for the 3.1 branch fix.
Status: RESOLVED → VERIFIED
Flags: in-litmus?
Comment 9•15 years ago
|
||
Ok, the litmus test case, https://litmus.mozilla.org/show_test.cgi?id=7468 , was updated to regression test this bug.
Updated•15 years ago
|
Flags: in-litmus? → in-litmus+
Reporter | ||
Comment 10•15 years ago
|
||
Tracy, we left the question mark because this test cannot be enabled right now. We need the check-in for FF3.1 first. Do you see a better way to handle disabled tests or tests which were created before the original bug will be filed?
Flags: in-litmus+ → in-litmus?
Comment 11•15 years ago
|
||
Re-enabling the litmus test case as this has been verified FIXED
Flags: in-litmus? → in-litmus+
Reporter | ||
Comment 12•15 years ago
|
||
Aakash, have you read comment 10? There was no change yet. The patch isn't in 1.9.1.
Flags: in-litmus+ → in-litmus?
Assignee | ||
Updated•15 years ago
|
Flags: in-testsuite? → in-testsuite+
Reporter | ||
Comment 14•15 years ago
|
||
With the automated test we don't need a Litmus test anymore. Removing flag.
Flags: in-litmus?
Comment 15•15 years ago
|
||
Comment on attachment 359734 [details] [diff] [review] patch v1.1 a191=beltzner
Attachment #359734 -
Flags: approval1.9.1? → approval1.9.1+
Assignee | ||
Comment 16•15 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/263032bde4fa
Keywords: fixed1.9.1
Reporter | ||
Comment 17•15 years ago
|
||
Verified fixed on 1.9.1 with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b4pre) Gecko/20090416 Shiretoko/3.5b4pre ID:20090416030924 and an appropriate build on WinXP.
Flags: in-litmus-
Keywords: fixed1.9.1 → verified1.9.1
Comment 18•15 years ago
|
||
Bug 451915 - move Firefox/Places bugs to Firefox/Bookmarks and History. Remove all bugspam from this move by filtering for the string "places-to-b-and-h". In Thunderbird 3.0b, you do that as follows: Tools | Message Filters Make sure the correct account is selected. Click "New" Conditions: Body contains places-to-b-and-h Change the action to "Delete Message". Select "Manually Run" from the dropdown at the top. Click OK. Select the filter in the list, make sure "Inbox" is selected at the bottom, and click "Run Now". This should delete all the bugspam. You can then delete the filter. Gerv
Component: Places → Bookmarks & History
QA Contact: places → bookmarks
You need to log in
before you can comment on or make changes to this bug.
Description
•