Dragging a tab in e10s windows shows an ugly tab preview/thumbnail (fully opaque, drop shadow, beveled borders)

VERIFIED FIXED in Firefox 49

Status

()

Firefox
Tabbed Browser
P1
normal
VERIFIED FIXED
2 years ago
a year ago

People

(Reporter: jaws, Assigned: Neil Deakin (not available until Aug 9))

Tracking

Trunk
Firefox 50
Points:
---

Firefox Tracking Flags

(e10s+, firefox44 affected, firefox47 wontfix, firefox48 wontfix, firefox49 verified, firefox50 verified)

Details

(Whiteboard: [fce-active])

Attachments

(1 attachment, 1 obsolete attachment)

STR:
Drag a tab horizontally through the tabbar and look at the tab preview.

On non-e10s windows, the preview is translucent whereas on e10s the preview is opaque and has a drop shadow with beveled borders.

Looking around, this may be related to bug 1194311?

Seen on Windows 10 with HiDPI.
I can reproduce on Mac.  Works fine in New non-e10s window.
OS: Unspecified → All
Hardware: Unspecified → All
On e10s we use a panel for the drag image rather than the OS-supplied drag feedback. We should investigate improving its appearance.
I filed bug 1208084 which describes an alternate styling that we could maybe use the panel for.
Created attachment 8666876 [details] [diff] [review]
Improve drag panel appearance

This makes the appearance appear like non-e10s on Mac, and mostly like it on Windows. It adds transparency and removes the drop shadow. On Windows, there is a border around it which I'm looking into how to remove.
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
tracking-e10s: ? → +
Jim, when I try this patch on Windows, the panel widget for the drag feedback image still has a bevelled border on it. I couldn't find any combination of the style bits or WS_EX* extended style bits that seemed to be adding it. Do you know what I'm missing here?
Flags: needinfo?(jmathies)

Comment 6

2 years ago
Really strange, I don't see many differences between this and our tooltip windows. The few differences i do see I removed and still ended up with the beveled edge. Are we sure that's not getting added via content?

panel:
[11F73500] CreateWindow mWindowType=2 mBorderStyle=-1 mIsTopWidgetWindow=0 style=82000000 exstyle=2000080
[11F73500] SetWindowTranslucencyInner transparent=1 style=86000000 exstyle=2080080

tooltip:
[C275400] CreateWindow mWindowType=2 mBorderStyle=-1 mIsTopWidgetWindow=1 style=82000000 exstyle=2000088
[C275400] SetWindowTranslucencyInner transparent=1 style=86000000 exstyle=2080088

- ex style difference is WS_EX_TOPMOST

the tooltip does not have a parent widget, but I removed that and it didn't change anything. I also removed the drop shadow class which didn't help.

Does the panel have a xul window+markup associated with it?
Flags: needinfo?(jmathies)

Comment 7

2 years ago
(In reply to Jim Mathies [:jimm] from comment #6)
> the tooltip does not have a parent widget, but I removed that and it didn't

the tooltip *doesn't* have a parent.
Hi,

I have managed to reproduce this on Windows 10, Windows 7, Ubuntu 14.04 and Mac OS X 10.7 on the latest Aurora (46.0a2) only with e10s enabled.

Build ID    20160207004019
User Agent  Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:46.0) Gecko/20100101 Firefox/46.0
User Agent  Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:46.0) Gecko/20100101 Firefox/46.0
User Agent  Mozilla/5.0 (X11; Linux x86_64; rv:46.0) Gecko/20100101 Firefox/46.0
User Agent  Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:46.0) Gecko/20100101 Firefox/46.0

Thanks,
Cipri

Updated

a year ago
Summary: Dragging a tab in e10s windows shows an ugly tab preview (fully opaque, drop shadow, beveled borders) → Dragging a tab in e10s windows shows an ugly tab preview/thumbnail (fully opaque, drop shadow, beveled borders)

Updated

a year ago
Priority: -- → P1
Whiteboard: [fct-active]
Whiteboard: [fct-active] → [fce-active]
Created attachment 8762077 [details] [diff] [review]
Updated patch

Still unsure about the border on Windows, but this patch improves things on all platforms, so we can further investigate later.
Attachment #8666876 - Attachment is obsolete: true
Attachment #8762077 - Flags: review?(jaws)
Comment on attachment 8762077 [details] [diff] [review]
Updated patch

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

This looks fine to me. I can confirm that the border appears for me on Windows too, but this feels like an improvement still.
Attachment #8762077 - Flags: review?(jaws) → review+
I should add that the borders may not be such a problem if bug 1208084 is fixed.
https://hg.mozilla.org/integration/mozilla-inbound/rev/9c9a0e822c0c8a009e6b96926279b8868b0e2f69
Bug 1208078, improve drag feedback appearance in e10s, r=jared

Comment 13

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/9c9a0e822c0c
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Neil, should we uplift this e10s bug fix from Nightly 50 to Aurora 49 or even Beta 48? We plan to enable e10s by default for some release channel users with Firefox 48.
status-firefox47: --- → wontfix
status-firefox48: --- → ?
status-firefox49: --- → ?
Flags: needinfo?(enndeakin)
Comment on attachment 8762077 [details] [diff] [review]
Updated patch

Approval Request Comment
[Feature/regressing bug #]: None, e10s only
[User impact if declined]: minor appearance polish of the translucent drag feedback to remove border/drop shadow 
[Describe test coverage new/current, TreeHerder]: manual testing
[Risks and why]: none
[String/UUID change made/needed]: none
Flags: needinfo?(enndeakin)
Attachment #8762077 - Flags: approval-mozilla-aurora?
wontfix for Beta 48.
status-firefox48: ? → wontfix
status-firefox49: ? → affected
Comment on attachment 8762077 [details] [diff] [review]
Updated patch

Fix for dragged tabs ugliness, please uplift to 49 aurora.
Attachment #8762077 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Cornel, this might be good for your team to verify on aurora once it lands.
Flags: needinfo?(cornel.ionce)

Comment 19

a year ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/1aa23cad1885
status-firefox49: affected → fixed
Verified fixed on 50.0a1 (2016-07-03), 49.0a2 (2016-07-04) Win 7, Ubuntu 14.04, OS X 10.12.
Found a new related issue on linux - bug 1284244
Status: RESOLVED → VERIFIED
status-firefox49: fixed → verified
status-firefox50: fixed → verified
Flags: needinfo?(cornel.ionce)
You need to log in before you can comment on or make changes to this bug.