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

Bug 513115

Summary: GTK: MenuDetect issues with toolbar
Product: [Eclipse Project] Platform Reporter: Thomas Singer <eclipse>
Component: SWTAssignee: Alexander Kurtakov <akurtakov>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: akurtakov, ericwill, loskutov
Version: 4.7Keywords: triaged
Target Milestone: ---   
Hardware: PC   
OS: Linux   
See Also: https://git.eclipse.org/r/93503
https://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/?id=8bd9620059daff78087c993d22fdbe714fe2d2ac
https://git.eclipse.org/r/112484
https://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/?id=056a3d562f2f340d63370d8408fc17e81209a712
https://git.eclipse.org/r/112512
https://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/?id=5a6656cb7a403b102a129c5017dc01e479e1ccda
Whiteboard:
Bug Depends on: 527860    
Bug Blocks:    
Attachments:
Description Flags
Code sample none

Description Thomas Singer CLA 2017-03-05 08:45:51 EST
Please launch the attached sample code. When right-clicking the toolbar right beside "Long Item", "Menu detected" will be printed on the console - fine. Right-clicking on a toolbar button causes "Menu detected" to be printed 2 times - bad. Right-clicking on the separator causes no "Menu detected" to be printed - bad.
Comment 1 Thomas Singer CLA 2017-03-05 08:46:14 EST
Created attachment 267109 [details]
Code sample
Comment 2 Eclipse Genie CLA 2017-03-21 08:32:31 EDT
New Gerrit change created: https://git.eclipse.org/r/93503
Comment 4 Alexander Kurtakov CLA 2017-03-21 08:49:18 EDT
(In reply to Eclipse Genie from comment #3)
> Gerrit change https://git.eclipse.org/r/93503 was merged to [master].
> Commit:
> http://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/
> ?id=8bd9620059daff78087c993d22fdbe714fe2d2ac

Double "Menu detected" fixed. Separator issue needs more investigating.
Comment 5 Andrey Loskutov CLA 2017-11-28 11:32:59 EST
@Thomas: was this issue a *regression* in 4.7, or was this always like this on GTK3 SWT?

I'm asking to understand better if the partial fix can be reverted, because it caused a severe regression in bug 527860.
Comment 6 Thomas Singer CLA 2017-11-28 11:40:23 EST
(In reply to Andrey Loskutov from comment #5)
> @Thomas: was this issue a *regression* in 4.7, or was this always like this
> on GTK3 SWT?

I don't know, because of the large number of GTK3 problems we did not use it a long time. Because the multiple issues with the GTK toolbar we wrote our own self-painted one for SmartGit.
Comment 7 Eclipse Genie CLA 2017-11-28 13:45:07 EST
New Gerrit change created: https://git.eclipse.org/r/112484
Comment 9 Eclipse Genie CLA 2017-11-28 16:25:31 EST
New Gerrit change created: https://git.eclipse.org/r/112512
Comment 10 Alexander Kurtakov CLA 2017-11-29 03:22:41 EST
Thomas, can you create a junit test for this issue? We need these issues catched by tests as otherwise there is always smth to miss.
Comment 13 Eric Williams CLA 2018-03-28 11:00:44 EDT
Is there anything left to be done here?