Closed Bug 623768 Opened 14 years ago Closed 14 years ago

Pressing Backspace in Panorama causes last focused tab to navigate backwards

Categories

(Firefox Graveyard :: Panorama, defect, P3)

Product:
Firefox Graveyard
Component:
Panorama
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: krazi.bazil, Assigned: raymondlee)

References

Details

Attachments

(1 file, 1 obsolete file)

[checked-in] Patch for check-in
3.97 KB, patch
Details | Diff | Splinter Review
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b8) Gecko/20100101 Firefox/4.0b8 Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b8) Gecko/20100101 Firefox/4.0b8 After entering Panorama, pressing backspace causes the tab I entered Panorama from to navigate back a page. This became an issue when I was using the search feature and pressed backspace one to many times. Reproducible: Always Steps to Reproduce: 1. Open a tab in which you are able to navigate backwards. 2. From said tab, open Panorama (Ctrl+E or button - same result with either). 3. Press backspace. Actual Results: Tab from which Panorama was opened navigates back a page. Expected Results: Nothing should happen.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 7 → All
Hardware: x86_64 → All
Assignee: nobody → raymond
Other shortcuts work as well, for example Ctrl-+
This is a specific case of bug 587276; marking as duplicate.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → DUPLICATE
The patch for bug 587276 does not cover this case (it white lists commands, not key strokes, see comment 6 on bug 636178). Re-opening. Also, that patch has a- for Fx4. Perhaps we can make a trivial patch for this one?
Blocks: 585689
Status: RESOLVED → REOPENED
Priority: -- → P3
Resolution: DUPLICATE → ---
Attached patch v1 (obsolete) — — Splinter Review
Attachment #515024 - Flags: review?(ian)
Attachment #515024 - Flags: feedback?(mitcho)
Comment on attachment 515024 [details] [diff] [review] v1 Looks great. Definitely worth doing for RC, imho.
Attachment #515024 - Flags: feedback?(mitcho) → feedback+
Attachment #515024 - Flags: review?(ian) → review+
Attachment #515024 - Flags: approval2.0?
Comment on attachment 515024 [details] [diff] [review] v1 a=beltzner
Attachment #515024 - Flags: approval2.0? → approval2.0+
Attachment #515024 - Attachment is obsolete: true
Keywords: checkin-needed
Status: REOPENED → ASSIGNED
Comment on attachment 515800 [details] [diff] [review] [checked-in] Patch for check-in http://hg.mozilla.org/mozilla-central/rev/270aec271dc8
Attachment #515800 - Attachment description: Patch for check-in → [checked-in] Patch for check-in
Status: ASSIGNED → RESOLVED
Closed: 14 years ago → 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
I guess that Ctrl+/- case is not fixed by this patch - will it be fixed by bug 587276 or should I file a separate bug?
(In reply to comment #11) > I guess that Ctrl+/- case is not fixed by this patch - will it be fixed by bug > 587276 or should I file a separate bug? It will be fixed by bug 587276 but it would happen post Fx4
Blocks: 637840
No longer blocks: 637840
Verified with: Mozilla/5.0 (X11; Linux i686; rv:2.0) Gecko/20100101 Firefox/4.0
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: