Bug 553865 - NewWizard: category closes after being opened
Summary: NewWizard: category closes after being opened
Status: NEW
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.8   Edit
Hardware: PC All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Platform-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 430837 561695 (view as bug list)
Depends on:
Blocks:
 
Reported: 2019-12-06 08:46 EST by Nils Peters CLA
Modified: 2020-04-29 06:55 EDT (History)
8 users (show)

See Also:


Attachments
NewWizard affected category "Eclipse 4" (18.49 KB, image/png)
2019-12-06 08:46 EST, Nils Peters CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nils Peters CLA 2019-12-06 08:46:27 EST
Created attachment 280894 [details]
NewWizard affected category "Eclipse 4"

I encoutered a strange behavior in the NewWizard when opening the "Eclipse 4" category.
The first time opening this category causes the subelements to show, but only for a short period of time (~100ms). Then the category gets closed again - without any user interaction. The second attempt to open the category will result in the expected behavior: the subelements stay visible. All other categories are working as expected.

Steps to reproduce:
1. Open Eclipse SDK (tested with 2019-09 and 4.14RC1) with installed Eclipse e4 Tools
2. Open the NewWizard via File -> New -> Other...
3. Open the category/folder "Eclipse 4" (with double click or a click on the expand arrow)
4. Watch the behavior explained above.
Comment 1 Rolf Theunissen CLA 2019-12-06 10:56:20 EST
The same happens for the "Examples" category. Both cases only if not another category is opened first.

Both "Eclipse 4" and "Examples" only contain sub-categories.
Comment 2 Lars Vogel CLA 2019-12-06 10:56:47 EST
(In reply to Rolf Theunissen from comment #1)
> The same happens for the "Examples" category. Both cases only if not another
> category is opened first.
> 
> Both "Eclipse 4" and "Examples" only contain sub-categories.

Happens on Windows and Linux.
Comment 3 Andrey Loskutov CLA 2019-12-06 11:34:01 EST
I wondering nobody noticed this yet. Looks like this is coming from bug 257176.
If I remove focusLost() handler from filtered text, everything works as expected (org.eclipse.ui.dialogs.FilteredTree.createFilterText(...).new FocusAdapter() {...}.focusLost(FocusEvent)). The handler triggers tree refresh that killed the expanded state.
Comment 4 Andrey Loskutov CLA 2019-12-06 11:37:25 EST
I was able to see it on all versions back to 4.8, where we added "Eclipse 4" category.
Comment 5 Lars Vogel CLA 2019-12-13 09:48:04 EST
(In reply to Andrey Loskutov from comment #3)
> I wondering nobody noticed this yet. Looks like this is coming from bug
> 257176.
> If I remove focusLost() handler from filtered text, everything works as
> expected (org.eclipse.ui.dialogs.FilteredTree.createFilterText(...).new
> FocusAdapter() {...}.focusLost(FocusEvent)). The handler triggers tree
> refresh that killed the expanded state.

So we remove the focusLost handler?
Comment 6 Andrey Loskutov CLA 2019-12-14 02:28:42 EST
(In reply to Lars Vogel from comment #5)
> (In reply to Andrey Loskutov from comment #3)
> > I wondering nobody noticed this yet. Looks like this is coming from bug
> > 257176.
> > If I remove focusLost() handler from filtered text, everything works as
> > expected (org.eclipse.ui.dialogs.FilteredTree.createFilterText(...).new
> > FocusAdapter() {...}.focusLost(FocusEvent)). The handler triggers tree
> > refresh that killed the expanded state.
> 
> So we remove the focusLost handler?

I didn't said that. I only wanted to show *what* is the root cause for the bug. One need to understand first, *why* it was needed (see bug 257176) and if there is still need for that code. If the code isn't necessary anymore we can remove it, if it is still needed, one should find a better fix for bug 257176.
Comment 7 Andrey Loskutov CLA 2019-12-14 02:33:46 EST
If I read it right (bug 257176 comment 7), we might have dependency on bug 4655 due some Windows behavior. May be bug 4655 is obsoleted already, one need to check that.
Comment 8 Dani Megert CLA 2020-01-07 10:02:25 EST
Anyone working on this? If not, please remove the target milestone.
Comment 9 Rolf Theunissen CLA 2020-04-02 08:52:15 EDT
*** Bug 561695 has been marked as a duplicate of this bug. ***
Comment 10 Prachi Singla CLA 2020-04-03 01:56:48 EDT
Dear Team,

This issue is reproducible in our application also. Please let us know when it'll be resolved.

Thanks.
Comment 11 Prachi Singla CLA 2020-04-03 01:58:37 EDT
(In reply to Prachi Singla from comment #10)
> Dear Team,
> 
> This issue is reproducible in our application also. Please let us know when
> it'll be resolved.
> 
> Thanks.

Moreover,
Isn't this case dulpicate of the below case??
https://bugs.eclipse.org/bugs/show_bug.cgi?id=430837
Comment 12 Rolf Theunissen CLA 2020-04-03 03:33:50 EDT
*** Bug 430837 has been marked as a duplicate of this bug. ***
Comment 13 Dani Megert CLA 2020-04-03 04:13:09 EDT
(In reply to Prachi Singla from comment #10)
> Dear Team,
> 
> This issue is reproducible in our application also. Please let us know when
> it'll be resolved.
> 
> Thanks.
When someone steps up t fix it.No ETA.
Comment 14 Prachi Singla CLA 2020-04-29 06:36:02 EDT
Hi,

Anyone started working on this case?
Comment 15 Lars Vogel CLA 2020-04-29 06:55:49 EDT
(In reply to Prachi Singla from comment #14)
> Hi,
> 
> Anyone started working on this case?

No, Gerrits are welcome