Closed
Bug 586679
Opened 14 years ago
Closed 14 years ago
Figure out how to show glass through TabView iframe
Categories
(Firefox Graveyard :: Panorama, defect)
Firefox Graveyard
Panorama
Tracking
(Not tracked)
RESOLVED
FIXED
Future
People
(Reporter: Mardak, Assigned: fryn)
References
Details
Bug 581813 kidna works around the issue of the TabView iframe preventing glass from showing through, so it ends up being black. Note that with personas on, glass is turned off and tabview doesn't black out the background and instead shows the personas.
Reporter | ||
Comment 1•14 years ago
|
||
Oh, the reason why we would want a better solution is that the iframe is shrinking with the fix for bug 581813 to hide behind the tabbrowser. But this can cause the contents of the tabview iframe to behave oddly.
Comment 2•14 years ago
|
||
My understanding is that as long as iframes get their own native widgets (i.e. until 130078 lands), this isn't possible.
Comment 4•14 years ago
|
||
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: -- → ---
Updated•14 years ago
|
QA Contact: tabcandy → tabcandy
Updated•14 years ago
|
Target Milestone: --- → Firefox 4.0
Comment 5•14 years ago
|
||
Looks like the black is not the iframe, but the #browser. <hack>I just added a margin-top:5000px style to #browser (and removed #bg in the tabview itself) and it works!
http://img.skitch.com/20100827-fbejyra29kqq9d4mf9mm2a4ikq.jpg
</hack>
Should we pursue this hacky hacky approach?
Comment 6•14 years ago
|
||
May I chime in here? Is a huge window which is entirely glass really something that we want to ship to our users? The screenshot in comment 5 is probably the *worst* thing I've ever seen on a computer screen!
Comment 7•14 years ago
|
||
(In reply to comment #6)
> May I chime in here? Is a huge window which is entirely glass really something
> that we want to ship to our users? The screenshot in comment 5 is probably the
> *worst* thing I've ever seen on a computer screen!
Agreed. That looks like a major bug. You can't tell what the active window is, you see floating min/max/close buttons and a floating MInefield button. With the three tab groups open it looks like three separately running WIndows programs.
Comment 8•14 years ago
|
||
(In reply to comment #6)
> May I chime in here? Is a huge window which is entirely glass really something
> that we want to ship to our users? The screenshot in comment 5 is probably the
> *worst* thing I've ever seen on a computer screen!
That effect is sure to be too slow on older machines. I think it may also go against Microsoft's guidelines for using glass (citation not handy). We really need to reconsider the design here.
Assignee | ||
Comment 9•14 years ago
|
||
(In reply to comment #6)
> May I chime in here? Is a huge window which is entirely glass really something
> that we want to ship to our users? The screenshot in comment 5 is probably the
> *worst* thing I've ever seen on a computer screen!
I concur that the screenshot looks horrendous as is, but if we decide that glass makes sense, we should look into make the glass less transparent. Dao made a patch to do that for the classic menu bar. (I'll look up the bug later.) Shorlander posted this mockup: http://www.stephenhorlander.com/images/blog-posts/incontent-ui/win7-tabcandy-glass.jpg I think it could be even more opaque than that.
Comment 10•14 years ago
|
||
There is also the alternative option of using the in-content styling:
http://www.stephenhorlander.com/images/blog-posts/incontent-ui/win7-tabcandy-textured.jpg
Some kind of translucent overlay could work but that would likely be just as slow on older machines.
Comment 11•14 years ago
|
||
Let's keep this bug for the technical issue of *whether* showing glass is possible... I've created a new bug (591652) for discussing whether this is a Good Idea or not.
Comment 12•14 years ago
|
||
It's possible, I managed it while making an extension for TabCandy. If there's interest in this I'll see if I can whip up a patch.
xul:iframe with glass example (the sidebar):
http://screencast.com/t/YWZhODQ4MjY
Comment 13•14 years ago
|
||
Olivier, please do. If you come into irc.mozilla.org#tabcandy we could collaborate as well.
Did this become suddenly possible because the Bane (bug 130078) was fixed?
Assignee: nobody → theappleofpineappe
Status: NEW → ASSIGNED
blocking2.0: --- → ?
Target Milestone: Firefox 4.0 → Firefox 4.0b6
Comment 14•14 years ago
|
||
@Oliver: That is very cool!
I'm removing this from blocking.
blocking2.0: ? → ---
Target Milestone: Firefox 4.0b6 → Future
Comment 15•14 years ago
|
||
Effectively impossible without a beta 5pre nightly.
With a beta 5pre nightly or more recent, remove the bg div.
For prosperity, let it be recorded here that I spent hours trying to get it working correctly on beta 4 only to give up, and then spent less than five minutes getting it working on beta 5pre.
Please note however, that tab thumbnails still display artifacts (black rectangles) when zooming in/out. This is probably related to the Bane not yet being completely fixed.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → INCOMPLETE
Assignee | ||
Comment 16•14 years ago
|
||
(In reply to comment #15)
> Effectively impossible without a beta 5pre nightly.
> With a beta 5pre nightly or more recent, remove the bg div.
If I understand this correctly, this is targeting the trunk, which is always the latest code, so it doesn't matter if it doesn't work on beta 4 or earlier.
Why was this marked RESOLVED INCOMPLETE?
Comment 17•14 years ago
|
||
I was referring to beta 4 because Micheal initially told me to work on it instead of a nightly. As for resolved incomplete, it is a mistake. First time using bugzilla and I assumed 'resolved: incomplete' meant 'partial-solution-found/implemented' (there is still have the tab thumbnail zoom in/out artifacts), not 'incomplete-bug-description'. I'm guilty of not reading the manual, and should be flogged.
Marked as resolved-fixed even if there is no patch checked in, since the issue was to see if it was possible, not to implement it.
Resolution: INCOMPLETE → FIXED
Comment 18•14 years ago
|
||
Again, guilty of not reading the manual closely. Marked correctly as resolved-invalid.
Resolution: FIXED → INVALID
Reporter | ||
Comment 19•14 years ago
|
||
(In reply to comment #15)
> With a beta 5pre nightly or more recent, remove the bg div.
This hasn't been done yet? So this bug is still open, no?
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Comment 20•14 years ago
|
||
I would not believe so, the issue was 'is it possible to show glass', not 'show glass through the iframe for TC'. I believe there is a separate bug for how (and if) to show/style the glass, assuming it was possible. (bug 591652)
Assignee | ||
Comment 21•14 years ago
|
||
This is now trivial, thanks to fixes to bug 130078, etc.
Simply disable the following lines from browser/themes/winstripe/browser/tabview/tabview.css when glass is enabled.
#bg {
background: url("chrome://browser/skin/tabview/grain.png") repeat scroll center top,
-moz-linear-gradient(center top , #CCD9EA, #C7D5E7) repeat scroll 0 0 transparent;
}
Assignee: theappleofpineappe → fryn
Status: REOPENED → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•