Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

In CreateViewForFrame() removed code that was hiding view if the 'visibility'

Browse Source 
property is 'hidden'


git-svn-id: svn://10.0.0.236/trunk@47531 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
troy%netscape.com
1999-09-15 04:54:00 +00:00
parent 103fbb9f76
commit 0ab7c093cb
2 changed files with 8 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
mozilla/layout/generic/nsHTMLContainerFrame.cpp
View File

@@ -403,23 +403,15 @@ nsHTMLContainerFrame::CreateViewForFrame(nsIPresContext& aPresContext,
viewManager->InsertChild(parentView, view, zIndex);
}
// If the background color is transparent or the visibility is hidden
// If the background color is transparent or the visibility is hidden,
// then mark the view as having transparent content. The reason we
// need to do it for visibility of hidden is that child elements can
// override their parent's visibility and be visible
// XXX We could try and be smarter about this and check whether there's
// a background image. If there is a background image and the image is
// fully opaque then we don't need to mark the view as having transparent
// content...
// need to do it for hidden visibility is that child elements can
// override their parent's visibility and be visible.
if ((NS_STYLE_BG_COLOR_TRANSPARENT & color->mBackgroundFlags) ||
!display->mVisible) {
(NS_STYLE_VISIBILITY_HIDDEN == display->mVisible)) {
viewManager->SetViewContentTransparency(view, PR_TRUE);
}
// Set the initial visiblity of the view -EDV
view->SetVisibility(NS_STYLE_VISIBILITY_HIDDEN == display->mVisible ?
nsViewVisibility_kHide : nsViewVisibility_kShow);
// XXX If it's fixed positioned, then create a widget so it floats
// above the scrolling area
const nsStylePosition* position = (const nsStylePosition*)

16
mozilla/layout/html/base/src/nsHTMLContainerFrame.cpp
View File

@@ -403,23 +403,15 @@ nsHTMLContainerFrame::CreateViewForFrame(nsIPresContext& aPresContext,
viewManager->InsertChild(parentView, view, zIndex);
}
// If the background color is transparent or the visibility is hidden
// If the background color is transparent or the visibility is hidden,
// then mark the view as having transparent content. The reason we
// need to do it for visibility of hidden is that child elements can
// override their parent's visibility and be visible
// XXX We could try and be smarter about this and check whether there's
// a background image. If there is a background image and the image is
// fully opaque then we don't need to mark the view as having transparent
// content...
// need to do it for hidden visibility is that child elements can
// override their parent's visibility and be visible.
if ((NS_STYLE_BG_COLOR_TRANSPARENT & color->mBackgroundFlags) ||
!display->mVisible) {
(NS_STYLE_VISIBILITY_HIDDEN == display->mVisible)) {
viewManager->SetViewContentTransparency(view, PR_TRUE);
}
// Set the initial visiblity of the view -EDV
view->SetVisibility(NS_STYLE_VISIBILITY_HIDDEN == display->mVisible ?
nsViewVisibility_kHide : nsViewVisibility_kShow);
// XXX If it's fixed positioned, then create a widget so it floats
// above the scrolling area
const nsStylePosition* position = (const nsStylePosition*)