Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 394386

Summary: [Contributions] Open Type | Search buttons interchanged
Product: [Eclipse Project] Platform Reporter: Ed Willink <ed>
Component: UIAssignee: Platform UI Triaged <platform-ui-triaged>
Status: CLOSED WONTFIX QA Contact:
Severity: normal    
Priority: P3 CC: ben.cox, daniel_megert, krzysztof.daniel, Lars.Vogel, loskutov, marc.khouzam, markus.kell.r, pawel.1.piech, pchuong, psuzzi, pwebster, stepper, sudol.wojciech
Version: 4.3   
Target Milestone: ---   
Hardware: All   
OS: All   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=463245
https://bugs.eclipse.org/bugs/show_bug.cgi?id=420956
Whiteboard: stalebug
Attachments:
Description Flags
Screenshot
none
Screenshot on Linux 64 bit gtk SDK build of M5a
none
Picture on Windows 7 and Ubuntu none

Description Ed Willink CLA 2012-11-15 09:24:21 EST
I've just switched from 3.8.1 to 4.2M3 and find that the debug tool bar has moved (very irritating, keep hitting Run app rather than Run to BP).

and worse the ordering of Step-Into, Step-Over and Step-Return has been reversed.

Is this a deliberate attempt to make e4 unpopular?
Comment 1 Markus Keller CLA 2012-11-16 07:05:33 EST
> Is this a deliberate attempt to make e4 unpopular?

No, I guess it's just a reflection of e4's quality...
Comment 2 Pawel Piech CLA 2012-11-16 13:09:55 EST
I think the bug introduced in _4.3M3_, because I can't seem to reproduce with M2.  Another bizzare detail is that when the IDE is started with a clean workspace, the ordering is correct, only when re-using the workspace ordering is reversed.  Also, not only debug actions are affected, I see that "Open Type" and "Search" actions traded places as well.
Comment 3 Markus Keller CLA 2012-11-16 13:47:09 EST
OK, let's move this to Platform UI. The orders must be like in 3.x everywhere.
Comment 4 Ed Willink CLA 2012-11-17 02:13:09 EST
(In reply to comment #0)
> and worse the ordering of Step-Into, Step-Over and Step-Return has been
> reversed.

Today (after restarting) the ordering is back to normal.

(In reply to comment #2)
> only when re-using the workspace ordering is reversed.

Thanks for the conformation. This seems to be the key. It is a 3.8.1 to 4.3M3 workspace re-use migration problem, possibly aggravated by doing 3.8.1 to 4.3M3 to 3.8.1 to 4.3M3 to check up on strange EGIT behaviours.
Comment 5 Pawel Piech CLA 2013-04-19 12:23:45 EDT
*** Bug 405969 has been marked as a duplicate of this bug. ***
Comment 6 Pawel Piech CLA 2013-04-19 12:25:55 EDT
Is this likely to be addressed in Kepler?  I can easily change the ordering of the declaration of actions in the action set, but there is still going to be an inconsistency between when launching with a new workspace or not.
Comment 7 Ed Willink CLA 2013-06-28 13:34:11 EDT
(In reply to comment #6)
> Is this likely to be addressed in Kepler? 

I'm really surprised that what seems like a trivial but MAJOR fix has been allowed out unfixed.

I'm inclined now to WONTFIX since there will Kepler users who gow used to the changed ordering.

Certainly I have been using random ordering (outer/inner Eclipse) for so long now that I frequently have to consult the hover text to tell me which button is which. The icons no longer make any sense to me. If I don't use the hover text I frequently make the wrong selection.
Comment 8 Ed Willink CLA 2013-07-18 16:48:10 EDT
(In reply to comment #6)
> Is this likely to be addressed in Kepler?  I can easily change the ordering
> of the declaration of actions in the action set, but there is still going to
> be an inconsistency between when launching with a new workspace or not.

I work on two different machines both of wose workspaces ahve been upgraded over many months if not years.

One has one order, the other is reversed. No wonder I'm confused/depressed.
Comment 9 Dani Megert CLA 2013-07-23 06:09:25 EDT
(In reply to comment #8)
> (In reply to comment #6)
> > Is this likely to be addressed in Kepler?  I can easily change the ordering
> > of the declaration of actions in the action set, but there is still going to
> > be an inconsistency between when launching with a new workspace or not.
> 
> I work on two different machines both of wose workspaces ahve been upgraded
> over many months if not years.
> 
> One has one order, the other is reversed. No wonder I'm confused/depressed.

Reset Perspective... might bring it back in order.

Using new workspaces, the order is the same for 3.x and 4.x.
Comment 10 Markus Keller CLA 2013-07-23 06:53:53 EDT
Created attachment 233701 [details]
Screenshot

Steps in 4.3 (no older release necessary to reproduce, just a restart):

- launch new workspace, close Welcome, open Debug perspective
 => order is correct

- File > Restart
 => order is mangled

The mangling only seems to happen when you restart while the Debug perspective is open.

WORKAROUND: Switch to Java perspective before shutting down Eclipse.
Comment 11 Dani Megert CLA 2013-07-23 07:11:35 EDT
(In reply to comment #10)
> Created attachment 233701 [details]
> Screenshot
> 
> Steps in 4.3 (no older release necessary to reproduce, just a restart):
> 
> - launch new workspace, close Welcome, open Debug perspective
>  => order is correct
> 
> - File > Restart
>  => order is mangled
> 
> The mangling only seems to happen when you restart while the Debug
> perspective is open.


And if so, the perspective is corrupt forever in its current window, i.e. resetting the perspective does not help. Opening the Debug perspective in a new window fixes the problem.
Comment 12 Dani Megert CLA 2013-08-06 04:21:38 EDT
*** Bug 414395 has been marked as a duplicate of this bug. ***
Comment 13 Krzysztof Daniel CLA 2013-08-06 04:26:57 EDT
This bug may be related to bug 399401.
Comment 14 Ed Willink CLA 2014-04-08 09:53:25 EDT
Are you really not going to fix this for Luna? It was bad enough this UI being broken in Kepler, but Luna too?
Comment 15 Paul Webster CLA 2014-04-08 10:08:53 EDT
Not in Luna, unless it gets fixed as part of Bug 385565

PW
Comment 16 Daniel Rolka CLA 2014-11-06 10:33:23 EST
Wojtek,

Please verify if it is still valid. If not, I will remove it from the priority list of Mars

thanks,
Daniel
Comment 17 Wojciech Sudol CLA 2014-11-07 04:02:29 EST
I tested Eclipse 4.4.0, 4.4.1 and 4.5 I20141027-2000.
The order of Step Into/Over/Return buttons is now correct and stay correct after restart, but the problem with order of "Search" and "Open Type" buttons still exists.
Comment 18 Ed Willink CLA 2014-11-07 04:47:42 EST
(In reply to Wojciech Sudol from comment #17)
I tested Eclipse 4.5M3.

Markus' repro from Comment #10 doesn't seem to be repro any more.

I'll keep my eye out to see if it's stable, but I've got used to it being wrong now.
Comment 19 Andrey Loskutov CLA 2015-02-07 18:17:25 EST
Everything is OK now in 4.5 M5 (open type/search order is OK, step in/out too). Closing as fixed, but I have no idea where the fix was actually introduced.
Comment 20 Dani Megert CLA 2015-02-09 06:16:45 EST
Using 4.5 M5a, I still see the problem (only checked Search / Open Type):

With a new workspace (or new Java perspective being opened), the order is wrong:
Search | Open Type

After one restart it becomes correct:  Open Type | Search
Comment 21 Andrey Loskutov CLA 2015-02-09 17:37:34 EST
(In reply to Dani Megert from comment #20)
> Using 4.5 M5a, I still see the problem (only checked Search / Open Type):
> 
> With a new workspace (or new Java perspective being opened), the order is
> wrong:
> Search | Open Type
> 
> After one restart it becomes correct:  Open Type | Search

Dani, which product are you using? Can the problem you still observe (or my own state I've observed) be caused by some 3rd party plugins redefining the toolbar definitions?
Comment 22 Dani Megert CLA 2015-02-10 09:16:23 EST
(In reply to Andrey Loskutov from comment #21)
> (In reply to Dani Megert from comment #20)
> > Using 4.5 M5a, I still see the problem (only checked Search / Open Type):
> > 
> > With a new workspace (or new Java perspective being opened), the order is
> > wrong:
> > Search | Open Type
> > 
> > After one restart it becomes correct:  Open Type | Search
> 
> Dani, which product are you using? Can the problem you still observe (or my
> own state I've observed) be caused by some 3rd party plugins redefining the
> toolbar definitions?

That could be. I'm using "our" plain SDK build:
http://download.eclipse.org/eclipse/downloads/drops4/S-4.5M5a-201502031300/
Comment 23 Andrey Loskutov CLA 2015-02-10 15:07:48 EST
Created attachment 250691 [details]
Screenshot on Linux 64 bit gtk SDK build of M5a

(In reply to Dani Megert from comment #22)
> (In reply to Andrey Loskutov from comment #21)
> > (In reply to Dani Megert from comment #20)
> > > Using 4.5 M5a, I still see the problem (only checked Search / Open Type):
> > > 
> > > With a new workspace (or new Java perspective being opened), the order is
> > > wrong:
> > > Search | Open Type
> > > 
> > > After one restart it becomes correct:  Open Type | Search
> > 
> > Dani, which product are you using? Can the problem you still observe (or my
> > own state I've observed) be caused by some 3rd party plugins redefining the
> > toolbar definitions?
> 
> That could be. I'm using "our" plain SDK build:
> http://download.eclipse.org/eclipse/downloads/drops4/S-4.5M5a-201502031300/

Just tried exact this "plain" build out - it works fine out of the box, see attached picture. BTW I'm using 64 bit Linux gtk build and GTK3 on KDE.
Any ideas what can be wrong with your build - I guess Windows or Mac?
Comment 24 Dani Megert CLA 2015-02-11 03:21:13 EST
(In reply to Andrey Loskutov from comment #23)
> > That could be. I'm using "our" plain SDK build:
> > http://download.eclipse.org/eclipse/downloads/drops4/S-4.5M5a-201502031300/
> 
> Just tried exact this "plain" build out - it works fine out of the box, 

I don't think so ;-). I assume that you tried with an existing workspace instead of a new one, see comment 20.
Comment 25 Dani Megert CLA 2015-02-11 03:21:55 EST
Created attachment 250702 [details]
Picture on Windows 7 and Ubuntu
Comment 26 Andrey Loskutov CLA 2015-02-11 04:10:07 EST
(In reply to Dani Megert from comment #24)
> (In reply to Andrey Loskutov from comment #23)
> > > That could be. I'm using "our" plain SDK build:
> > > http://download.eclipse.org/eclipse/downloads/drops4/S-4.5M5a-201502031300/
> > 
> > Just tried exact this "plain" build out - it works fine out of the box, 
> 
> I don't think so ;-). I assume that you tried with an existing workspace
> instead of a new one, see comment 20.

I've used new workspace and fresh install - everything from scratch.
Comment 27 Dani Megert CLA 2015-02-11 04:45:19 EST
(In reply to Andrey Loskutov from comment #26)
> (In reply to Dani Megert from comment #24)
> > (In reply to Andrey Loskutov from comment #23)
> > > > That could be. I'm using "our" plain SDK build:
> > > > http://download.eclipse.org/eclipse/downloads/drops4/S-4.5M5a-201502031300/
> > > 
> > > Just tried exact this "plain" build out - it works fine out of the box, 
> > 
> > I don't think so ;-). I assume that you tried with an existing workspace
> > instead of a new one, see comment 20.
> 
> I've used new workspace and fresh install - everything from scratch.

I also managed to see it correct on one install. I might depend on the Welcome page (Browser widget) and how it is closed (timing issue?). When I add
-Dorg.eclipse.swt.browser.DefaultType=mozilla -Dorg.eclipse.swt.browser.XULRunnerPath=/none
it starts to fail again.
Comment 28 Lars Vogel CLA 2015-04-08 18:13:03 EDT
(In reply to Dani Megert from comment #20)
> Using 4.5 M5a, I still see the problem (only checked Search / Open Type):
> 
> With a new workspace (or new Java perspective being opened), the order is
> wrong:
> Search | Open Type
> 
> After one restart it becomes correct:  Open Type | Search

Can't reproduce with 4.5.0.N20150325-2000
Comment 29 Dani Megert CLA 2015-04-09 06:01:15 EDT
(In reply to Lars Vogel from comment #28)
> (In reply to Dani Megert from comment #20)
> > Using 4.5 M5a, I still see the problem (only checked Search / Open Type):
> > 
> > With a new workspace (or new Java perspective being opened), the order is
> > wrong:
> > Search | Open Type
> > 
> > After one restart it becomes correct:  Open Type | Search
> 
> Can't reproduce with 4.5.0.N20150325-2000

Same here using N20150224-2000.
Comment 30 Dani Megert CLA 2015-04-17 04:54:45 EDT
(In reply to Dani Megert from comment #29)
> (In reply to Lars Vogel from comment #28)
> > (In reply to Dani Megert from comment #20)
> > > Using 4.5 M5a, I still see the problem (only checked Search / Open Type):
> > > 
> > > With a new workspace (or new Java perspective being opened), the order is
> > > wrong:
> > > Search | Open Type
> > > 
> > > After one restart it becomes correct:  Open Type | Search
> > 
> > Can't reproduce with 4.5.0.N20150325-2000
> 
> Same here using N20150224-2000.

We probably closed it by clicking on the arrow. When I click on the [x] immediately after startup, it is still swapped. This is with eclipse-SDK-N20150416-2000-win32-x86_64.
Comment 31 Andrey Loskutov CLA 2015-04-17 05:11:18 EDT
(In reply to Dani Megert from comment #30)
> (In reply to Dani Megert from comment #29)
> > (In reply to Lars Vogel from comment #28)
> > > (In reply to Dani Megert from comment #20)
> > > > Using 4.5 M5a, I still see the problem (only checked Search / Open Type):
> > > > 
> > > > With a new workspace (or new Java perspective being opened), the order is
> > > > wrong:
> > > > Search | Open Type
> > > > 
> > > > After one restart it becomes correct:  Open Type | Search
> > > 
> > > Can't reproduce with 4.5.0.N20150325-2000
> > 
> > Same here using N20150224-2000.
> 
> We probably closed it by clicking on the arrow. When I click on the [x]
> immediately after startup, it is still swapped. This is with
> eclipse-SDK-N20150416-2000-win32-x86_64.

Using same build N20150413-2000 (different natives of course) on Windows 7 and RHEL 7 I see the issue only on Win 7, and only if I use [x] close button on the welcome view tab to close the welcome view. If I use "restore" button to un-maximize the welcome view, the buttons order is OK. But as said, this is visible only on Windows 7, on RHEL I don't see any "bad" ordering at all.

So it seems to be part closing/SWT related - the order/amount of events can depend on the SWT platform code (e.g. extra refreshes or events here and there) after part closing.
Comment 32 Eclipse Genie CLA 2020-04-30 00:40:12 EDT
This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.