This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 427133 - [Navigator] View Menu disappears in Project Explorer
Summary: [Navigator] View Menu disappears in Project Explorer
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.3.1   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 4.3.2   Edit
Assignee: Wojciech Sudol CLA
QA Contact: Eric Moffatt CLA
URL:
Whiteboard:
Keywords:
Depends on: 417212
Blocks:
  Show dependency tree
 
Reported: 2014-01-31 10:29 EST by Wojciech Sudol CLA
Modified: 2014-02-05 11:55 EST (History)
2 users (show)

See Also:
daniel_megert: pmc_approved+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Wojciech Sudol CLA 2014-01-31 10:29:57 EST
Scenario:
1. In a fresh workspace open Project Explorer. -> view menu disappears immediately after opening the view
2. Minimize and then restore the view -> view menu is visible
3. Create a java project.
4. In Project Explorer unfold the content of the project and select any item -> view menu disappears again.

Found in Eclipse 4.3.2 M20140129-0800.
I cannot reproduce the bug in 4.4 M5 and 4.3.1.
Comment 1 Wojciech Sudol CLA 2014-01-31 10:35:18 EST
May be related to the bug 420902.
Comment 2 Paul Webster CLA 2014-01-31 10:37:33 EST
Wojtek and Daniel, could you please investigate this on Monday to see if we can find the cause?  I'd like to see if we could fix this before 4.3.2 goes out with this problem.

Thanks,
Paul
Comment 3 Wojciech Sudol CLA 2014-02-03 01:31:51 EST
The bug has been introduced in the commit 59b0576c7c3f0152436fe38a24bc456b42c84231 (Fix for Bug 416650 - CTabFolder layouts content too often / too many of them).

Fix review URL: https://git.eclipse.org/r/#/c/21433/ .
Comment 4 Paul Webster CLA 2014-02-03 06:10:07 EST
Eric, could you please take a look at this fix?

PW
Comment 5 Eric Moffatt CLA 2014-02-03 14:13:33 EST
Wojciech, good pickup ! Unfortunately the patch may fix the defect but breaks the original performance fix in doing so...

Can you investigate *why* the view menu disappears ? Since it doesn't happen (I hope) for the Package Explorer I suspect that there's some extra logic used by the Project Explorer in managing its menu ?

This may also be a painting bug; I seem to remember a case where the 'pack' call wasn't actually changing the size of the control it wasn't being repainted...
Comment 6 Eric Moffatt CLA 2014-02-03 14:31:05 EST
If instead of layout out the whole CTF you just layout the 'trComp' it seems to work...I just added 'trComp.layout(true, true);' after the 'trComp.pack()' call at line 673 in the StackRenderer and it seems to work ok, could you check this out for me.

I'd still be interested in why the Project Explorer's behavior is different than the Package Exploter's though...;-).
Comment 7 Wojciech Sudol CLA 2014-02-03 15:06:46 EST
I have update the code and it still does fix the bug.
Comment 8 Wojciech Sudol CLA 2014-02-03 16:47:46 EST
It seems that this is a duplicate of Bug 417960 - "[EditorMgmt] History view dropdown menu disappears" that is already fixed in 4.4.
Since it is not a small fix, should we port it to 4.3?
Comment 9 Paul Webster CLA 2014-02-04 09:11:07 EST
Dani, I'd like to consider this bug for 4.3.2 RC3.  Having the view menu disappear in SR2 I think is pretty major (even though you can get it back) and I think the fix is safe.

PW
Comment 10 Dani Megert CLA 2014-02-05 05:27:20 EST
(In reply to Wojciech Sudol from comment #8)
> It seems that this is a duplicate of Bug 417960 - "[EditorMgmt] History view
> dropdown menu disappears" that is already fixed in 4.4.
> Since it is not a small fix, should we port it to 4.3?

I verified that the proposed fix also fixes bug 417960 in 4.3.2 and hence it's the best fix at this point since it is too risky so late in the game for the "bug" fix.

Submitted with http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=7de46ea4313cf06e21313f752f63f9e42ba8bace
Comment 11 Dani Megert CLA 2014-02-05 11:55:16 EST
Verified in M20140205-0800.