Closed
Bug 1208078
Opened 10 years ago
Closed 9 years ago
Dragging a tab in e10s windows shows an ugly tab preview/thumbnail (fully opaque, drop shadow, beveled borders)
Categories
(Firefox :: Tabbed Browser, defect, P1)
Firefox
Tabbed Browser
Tracking
()
VERIFIED
FIXED
Firefox 50
People
(Reporter: jaws, Assigned: enndeakin)
References
Details
(Whiteboard: [fce-active-legacy])
Attachments
(1 file, 1 obsolete file)
1.65 KB,
patch
|
jaws
:
review+
lizzard
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
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.
Comment 1•10 years ago
|
||
I can reproduce on Mac. Works fine in New non-e10s window.
OS: Unspecified → All
Hardware: Unspecified → All
Assignee | ||
Comment 2•10 years ago
|
||
On e10s we use a panel for the drag image rather than the OS-supplied drag feedback. We should investigate improving its appearance.
Reporter | ||
Comment 3•10 years ago
|
||
I filed bug 1208084 which describes an alternate styling that we could maybe use the panel for.
Assignee | ||
Comment 4•10 years ago
|
||
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
Updated•10 years ago
|
Assignee | ||
Comment 5•10 years ago
|
||
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•10 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•10 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.
Comment 8•9 years ago
|
||
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•9 years 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•9 years ago
|
Priority: -- → P1
Updated•9 years ago
|
Whiteboard: [fct-active]
Updated•9 years ago
|
Whiteboard: [fct-active] → [fce-active]
Assignee | ||
Comment 9•9 years ago
|
||
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)
Reporter | ||
Comment 10•9 years ago
|
||
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+
Reporter | ||
Comment 11•9 years ago
|
||
I should add that the borders may not be such a problem if bug 1208084 is fixed.
Assignee | ||
Comment 12•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9c9a0e822c0c8a009e6b96926279b8868b0e2f69
Bug 1208078, improve drag feedback appearance in e10s, r=jared
Comment 13•9 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Comment 14•9 years ago
|
||
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)
Assignee | ||
Comment 15•9 years ago
|
||
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?
Comment 16•9 years ago
|
||
wontfix for Beta 48.
Comment 17•9 years ago
|
||
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+
Comment 18•9 years ago
|
||
Cornel, this might be good for your team to verify on aurora once it lands.
Flags: needinfo?(cornel.ionce)
Comment 19•9 years ago
|
||
bugherder uplift |
Comment 20•9 years ago
|
||
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
Flags: needinfo?(cornel.ionce)
Updated•7 years ago
|
Whiteboard: [fce-active] → [fce-active-legacy]
You need to log in
before you can comment on or make changes to this bug.
Description
•