Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 344173 - [Compatibility] Folder layout with only placeholders gets rendered
Summary: [Compatibility] Folder layout with only placeholders gets rendered
Status: VERIFIED FIXED
Alias: None
Product: e4
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 1.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 4.1 M7   Edit
Assignee: Remy Suen CLA
QA Contact: Remy Suen CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 344172
  Show dependency tree
 
Reported: 2011-04-28 14:12 EDT by Remy Suen CLA
Modified: 2013-06-06 07:12 EDT (History)
0 users

See Also:


Attachments
Screenshot depicting the problem in question. (32.70 KB, image/png)
2011-04-28 14:12 EDT, Remy Suen CLA
no flags Details
Perspective rendering patch v1 (2.25 KB, patch)
2011-04-28 14:32 EDT, Remy Suen CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Remy Suen CLA 2011-04-28 14:12:58 EDT
Created attachment 194296 [details]
Screenshot depicting the problem in question.

I've seen this problem even on 3.x too actually (though I forgot how to reproduce the problem). I'm going to guess this perspective doesn't have problems on 3.x though or else they would've found out about it by now...I think.
Comment 1 Remy Suen CLA 2011-04-28 14:29:36 EDT
(In reply to comment #0)
> I've seen this problem even on 3.x too actually (though I forgot how to
> reproduce the problem). I'm going to guess this perspective doesn't have
> problems on 3.x though or else they would've found out about it by now...I
> think.

Okay I was wrong, it looks like it's the same problem, see bug 344173.
Comment 2 Remy Suen CLA 2011-04-28 14:32:23 EDT
Created attachment 194300 [details]
Perspective rendering patch v1

The fix is to make generated part stacks of folder layouts not be rendered by default. When a view is added to it, then we will force it (the view's parent) to be rendered.
Comment 3 Remy Suen CLA 2011-04-28 14:33:07 EDT
(In reply to comment #2)
> Created attachment 194300 [details]
> Perspective rendering patch v1

Fix released to CVS HEAD.
Comment 4 Remy Suen CLA 2011-04-29 14:09:29 EDT
Verified with I20110429-0945 on Windows XP.