Closed
Bug 786326
Opened 12 years ago
Closed 12 years ago
first popup of flyout panel doesn't show arrow
Categories
(Firefox Graveyard :: SocialAPI, defect)
Tracking
(firefox17 verified)
RESOLVED
FIXED
Firefox 18
Tracking | Status | |
---|---|---|
firefox17 | --- | verified |
People
(Reporter: mixedpuppy, Assigned: mixedpuppy)
Details
(Whiteboard: [Fx17])
Attachments
(1 file)
921 bytes,
patch
|
Gavin
:
review+
Gavin
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
images showing this are in bug 785952, and per comment #7 we should set the side attribute to fix this.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #656062 -
Flags: review?(gavin.sharp)
Comment 2•12 years ago
|
||
Why does this only affect the first popup?
Assignee | ||
Comment 3•12 years ago
|
||
Neil, (In reply to :Gavin Sharp (use gavin@gavinsharp.com for email) from comment #2) > Why does this only affect the first popup? TBH I'm not certain, but I think it has to do with panel.boxObject.show setting up display prior to css being set during popupshowing. I haven't traced everything through the boxobject class, there's a lot to digest. Maybe Neil can give a more informed answer?
Comment 4•12 years ago
|
||
The side attribute can be computed dynamically based on where the panel is opened, the position of which can be specified differently for each call to openPopup. However, the computations only work when the horizontal or vertical arrangement is known beforehand (defaulting to vertical like the bookmarks panel). This issue can also occur if you open a popup to the side and then open it vertically. Since changing the orientation isn't a normal operation (indeed, we haven't seen usage of side arrow panels yet), the arrow panels don't currently handle it.
Updated•12 years ago
|
Attachment #656062 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Comment 5•12 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=944991a1b7ab
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 656062 [details] [diff] [review] arrow patch [Approval Request Comment] Bug caused by (feature/regressing bug #): User impact if declined: initial open of social side panel fails to show correctly Testing completed (on m-c, etc.): on m-c Risk to taking this patch (and alternatives if risky): String or UUID changes made by this patch: none
Attachment #656062 -
Flags: approval-mozilla-aurora?
Updated•12 years ago
|
Attachment #656062 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Comment 7•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/6e9c83b3ba5b https://hg.mozilla.org/releases/mozilla-aurora/rev/dce109c23ef3
status-firefox17:
--- → fixed
status-firefox18:
--- → fixed
Flags: in-testsuite-
Keywords: checkin-needed
Updated•12 years ago
|
status-firefox18:
fixed → ---
Comment 8•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6e9c83b3ba5b
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
Comment 9•12 years ago
|
||
This issue is fixed on Firefox 17 Release Candidate, User Agent:Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/17.0 Firefox/17.0, Build ID: 20121116115405. (the arrow is visible) Marking this as verified.
Updated•5 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•