Closed Bug 597499 Opened 9 years ago Closed 9 years ago
Expand stack icon moves incorrectly and isn't centered
When you create a stacked group, resize the group such that the aspect ratio changes, but the group remains stacked. Note that the "expand stack" button moves around with the stack. It should remain centered along the bottom of the group in between the "new tab" button and the resizer mark.
This, although small, definitely blocks final release.
blocking2.0: --- → ?
Priority: -- → P1
Target Milestone: --- → Firefox 4.0
Summary: Expand stack icon moves incorrectly → Expand stack icon moves incorrectly and isn't centered
I'm worried about the number of blockers we're taking on tab candy, and how they divide into visual polish and functional polish. Speculatively blocking on this, but I'd kick it out of scope in favour of functional blockers, for sure.
blocking2.0: ? → final+
Expand button is now centered under the stack.
Comment on attachment 478857 [details] [diff] [review] v1 Looks good
Comment on attachment 478857 [details] [diff] [review] v1 >+ top: childBB.top + childBB.height - parentBB.top + padding, >+ left: parentBB.width/2 - this.$expander.width()/2 If the expander is supposed to be fixed on the group rect (at the bottom, centered horizontally), why not use a |bottom| rule, and ignore the childBB entirely?
(In reply to comment #5) > If the expander is supposed to be fixed on the group rect (at the bottom, > centered horizontally), why not use a |bottom| rule, and ignore the childBB > entirely? Aza wants it centered, hanging under the stack itself.
dolske: review+ ian: feedback+ Pushed to try as 10802c9008c7
ready for landing
Attachment #481929 - Attachment is obsolete: true
http://hg.mozilla.org/mozilla-central/rev/14e4622db30f Sean, a couple comments on the patch for check in: * Please list your email address in your user specification (right now it's "Sean Dunn <seanedunn>", should be "Sean Dunn <email@example.com>"). * The commit message needs an "a=", otherwise mozilla-central rejects it. In this case it would be "a=blocking". I've fixed it in this case; just for future reference.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
verified with nightly minefield builds of 20101021 expand icon remains centered under the stack
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.