Closed Bug 919030 Opened 11 years ago Closed 11 years ago

Status panel shifts to the right when moving the mouse over the bottom of the bookmarks or history sidebar

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
Firefox 27
Tracking Status
firefox24 --- unaffected
firefox25 - verified
firefox26 - verified
firefox27 - verified

People

(Reporter: alice0775, Assigned: dao)

References

Details

(Keywords: regression)

Attachments

(1 file)

This behavior was added by Bug 631302.
However, Bug 821687 is landed now on Aurora26.0a2 and Nightly27.0a1.
So, I think Bug 631302 should be backed out.

Step To Reproduce:
1. Open Sidebar Bookmarks or history(Ctrl+B or Ctrl+h)
2. Expand the tree in the sidebar to overflow (so that scrollbar appears)
3. Mouse over item at the bottom of sidebar

Actual Results:
Status panel shift to the right.

Expected Results:
No need to shift.
Attached patch patchSplinter Review
Assignee: nobody → dao
Status: NEW → ASSIGNED
Attachment #808018 - Flags: review?(mdeboer)
Keywords: regression
OS: Windows 7 → All
Hardware: x86_64 → All
Summary: Status panel shift to the right when mouse over item at the bottom of sidebar → Status panel shifts to the right when moving the mouse over the bottom of the bookmarks or history sidebar
Version: 26 Branch → Trunk
Not critical enough to track.
Comment on attachment 808018 [details] [diff] [review]
patch

Review of attachment 808018 [details] [diff] [review]:
-----------------------------------------------------------------

LGTM! Thanks for fixing this!
Attachment #808018 - Flags: review?(mdeboer) → review+
https://hg.mozilla.org/mozilla-central/rev/1ec9d4bee2ee
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
Comment on attachment 808018 [details] [diff] [review]
patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): bug 821687
User impact if declined: the status panel moves where it shouldn't in response to the user moving the mouse cursor over the bottom of the window
Testing completed (on m-c, etc.): landed on m-c
Risk to taking this patch (and alternatives if risky): low
String or IDL/UUID changes made by this patch: none
Attachment #808018 - Flags: approval-mozilla-beta?
Attachment #808018 - Flags: approval-mozilla-aurora?
Attachment #808018 - Flags: approval-mozilla-beta?
Attachment #808018 - Flags: approval-mozilla-beta+
Attachment #808018 - Flags: approval-mozilla-aurora?
Attachment #808018 - Flags: approval-mozilla-aurora+
Keywords: verifyme
Verified as fixed on Firefox 25 beta 3 (20130926170421), on Windows 7 64bit, Mac OSX 10.8.4 64bit and Ubuntu 13.04 32bit.
QA Contact: ioana.budnar
Mozilla/5.0 (Windows NT 6.2; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0
Mozilla/5.0 (Windows NT 5.1; rv:26.0) Gecko/20100101 Firefox/26.0
Mozilla/5.0 (X11; Linux i686; rv:26.0) Gecko/20100101 Firefox/26.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:26.0) Gecko/20100101 Firefox/26.0

Verified as fixed on latest Aurora 26.0a2 (buildID: 20130930004008) and latest Nightly 27.0a1 (buildID: 20131001030204).
Status: RESOLVED → VERIFIED
Keywords: verifyme
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: