[Win7] Aero-Peek issue with Navigation Bar redrawing when TabCandy is displayed

VERIFIED FIXED in Future

Status

defect
P2
normal
VERIFIED FIXED
9 years ago
3 years ago

People

(Reporter: locke105, Assigned: jimm)

Tracking

(Depends on 2 bugs)

Trunk
Future
x86_64
Windows XP
Dependency tree / graph

Firefox Tracking Flags

(blocking2.0 -)

Details

(Whiteboard: [visual][polish])

Attachments

(2 attachments)

User-Agent:       Mozilla/5.0 (Windows; Windows NT 6.1; WOW64; rv:2.0b3pre) Gecko/20100723 Minefield/4.0b3pre (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; Windows NT 6.1; WOW64; rv:2.0b3pre) Gecko/20100723 Minefield/4.0b3pre (.NET CLR 3.5.30729)

Found this issue when I opened the TabCandy viewer and then was using the AeroPeek fuctionality of Win7.

The navigation bar is rendering the top of the TabCandy view while the Aero-peek is rendering the actual FireFox tab display.

I have a picture to attach that shows what happens.

Reproducible: Always

Steps to Reproduce:
1. Open TabCandy viewer
2. Hover over Firefox task in taskbar
3. Observe
Actual Results:  
Navigation bar appears as top of TabCandy
Blocks: 586556
Mass moving all Tab Candy bugs from Mozilla Labs to Firefox::Tab Candy.  Filter the bugmail spam with "tabcandymassmove".
Product: Mozilla Labs → Firefox
Target Milestone: -- → ---
Version: unspecified → Trunk
QA Contact: tabcandy → tabcandy
This is still a problem in the latest 4.0b5pre (20100827). One more Windows 7 aero peek problem.
Status: UNCONFIRMED → NEW
blocking2.0: --- → ?
Ever confirmed: true
Summary: Aero-Peek issue with Navigation Bar when TabCandy is displayed → [Win7] Aero-Peek issue with Navigation Bar when TabCandy is displayed
blocking2.0: ? → betaN+
Summary: [Win7] Aero-Peek issue with Navigation Bar when TabCandy is displayed → [Win7] Aero-Peek issue with Navigation Bar redrawing when TabCandy is displayed
Assignee: nobody → raymond
Priority: -- → P3
Priority: P3 → P2
Blocks: 598154
Depends on: 586155
We expect bug 605338 to land before FF4.0, and bug 586155 after FF4.0, so punting this to the future.
No longer blocks: 598154
Target Milestone: --- → Future
Moving this back to our current work list as Aero Peek will most likely remain on by default and available in general. See comments 4 and 5 in bug 608338 for more info as to why.
Blocks: 598154
No longer depends on: 586155
Target Milestone: Future → ---
(In reply to comment #5)
> Moving this back to our current work list as Aero Peek will most likely remain
> on by default and available in general. See comments 4 and 5 in bug 608338 for
> more info as to why.

I'm working on some changes to the default behavior in bug 522035 which may make this problem simpler to solve. (assuming it works and gets approved.)
Depends on: 522035
OS: Windows 7 → Windows XP
I've posted a simple patch in bug 538487 for print preview that turns tab previews off when the print preview comes up. This might be useful here. I didn't see an event tied to tab candy hide/show that I could hook up to, but I imagine it would be easy to add something like that.
(In reply to comment #7)
> I've posted a simple patch in bug 538487 for print preview that turns tab
> previews off when the print preview comes up. This might be useful here. I
> didn't see an event tied to tab candy hide/show that I could hook up to, but I
> imagine it would be easy to add something like that.

Cool! Panorama sends out "tabviewhidden" and "tabviewshown" events when the Panorama UI is hidden or shown, respectively.
We don't need to block on this if we continue to block on bug 605338
blocking2.0: betaN+ → ---
I have a patch for this in my queue. just waiting on bug 538487 to settle.
Depends on: 538487
blocking2.0: --- → -
Posted patch fix v.1Splinter Review
Assignee: raymond → jmathies
Attachment #497817 - Flags: review?(dao)
bugspam (moving b9 to b10)
Blocks: 608028
bugspam (removing b9)
No longer blocks: 598154
I want to punt this, as we've done with Aero Peek integration in general, but this seems particularly jarring. Dao, could you review this, or could you change the review flag to someone else?
Blocks: 627096
No longer blocks: 608028
Whiteboard: [visual][polish]
Nevermind... Aero Peek is disabled by default for fx4 (bug 605338) so we'll punt on this.
No longer blocks: 627096
Target Milestone: --- → Future
Attachment #497817 - Flags: review?(dao) → review+
Attachment #497817 - Flags: approval2.0?
Comment on attachment 497817 [details] [diff] [review]
fix v.1

a=beltzner
Attachment #497817 - Flags: approval2.0? → approval2.0+
Blocks: 609080
http://hg.mozilla.org/mozilla-central/rev/953be21166fe
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Depends on: 631932
Mozilla/5.0 (Windows NT 6.1; rv:2.0b12pre) Gecko/20110215 Firefox/4.0b12pre

Issue was verified and it's no longer reproducible.
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.