[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)

(Reporter)

Description

9 years ago
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
(Reporter)

Comment 1

9 years ago

Updated

9 years ago
Blocks: 586556

Comment 2

9 years ago
Mass moving all Tab Candy bugs from Mozilla Labs to Firefox::Tab Candy.  Filter the bugmail spam with "tabcandymassmove".
Component: TabCandy → TabCandy
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+

Updated

9 years ago
Summary: [Win7] Aero-Peek issue with Navigation Bar when TabCandy is displayed → [Win7] Aero-Peek issue with Navigation Bar redrawing when TabCandy is displayed

Updated

9 years ago
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

Comment 5

9 years ago
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 → ---
(Assignee)

Comment 6

9 years ago
(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.)

Updated

9 years ago
Depends on: 522035
OS: Windows 7 → Windows XP
(Assignee)

Comment 7

9 years ago
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+ → ---
(Assignee)

Comment 10

9 years ago
I have a patch for this in my queue. just waiting on bug 538487 to settle.
Depends on: 538487
blocking2.0: --- → -
(Assignee)

Comment 11

9 years ago
Posted patch fix v.1Splinter Review
Assignee: raymond → jmathies
(Assignee)

Updated

9 years ago
Attachment #497817 - Flags: review?(dao)

Comment 12

8 years ago
bugspam (moving b9 to b10)
Blocks: 608028

Comment 13

8 years ago
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+
(Assignee)

Updated

8 years ago
Attachment #497817 - Flags: approval2.0?
Comment on attachment 497817 [details] [diff] [review]
fix v.1

a=beltzner
Attachment #497817 - Flags: approval2.0? → approval2.0+
(Assignee)

Updated

8 years ago
Blocks: 609080
(Assignee)

Comment 17

8 years ago
http://hg.mozilla.org/mozilla-central/rev/953be21166fe
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED

Comment 18

8 years ago
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.