Bug 547811 - [GTK] DateTime drop-down popup shell does not hiding on focus lost
Summary: [GTK] DateTime drop-down popup shell does not hiding on focus lost
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.12   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 4.13 M3   Edit
Assignee: Eric Williams CLA
QA Contact:
URL:
Whiteboard: RHT
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2019-05-31 08:54 EDT by Ilnur Zalyaev CLA
Modified: 2019-08-20 10:19 EDT (History)
1 user (show)

See Also:


Attachments
snippet to reproduce (1.36 KB, application/octet-stream)
2019-05-31 08:55 EDT, Ilnur Zalyaev CLA
no flags Details
video reproducing the problem (906.60 KB, video/mp4)
2019-05-31 08:56 EDT, Ilnur Zalyaev CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ilnur Zalyaev CLA 2019-05-31 08:54:47 EDT
Debian Stretch
GTK 3.22.30
Eclipse SDK 4.12M3.

When DateTime losts focus or when moving shell, drop down popup does not dissappear.
It also reflects on GTK2 on SWT 3.108.

It appears on the main shell of the application, but it does not appear on sub shell with style SWT.ON_TOP.

If we comment the string DateTime.java:517 (handleFocus(SWT.FocusOut);) everything works fine. But i don't what this popupFilter needs for.

I also attached snippet and video which reproduce the problem.
Comment 1 Ilnur Zalyaev CLA 2019-05-31 08:55:43 EDT
Created attachment 278791 [details]
snippet to reproduce
Comment 2 Ilnur Zalyaev CLA 2019-05-31 08:56:25 EDT
Created attachment 278792 [details]
video reproducing the problem
Comment 3 Eric Williams CLA 2019-05-31 12:37:13 EDT
Thanks for the bug report & snippet, I can reproduce the issue. We'll investigate.
Comment 4 Eric Williams CLA 2019-07-11 14:46:47 EDT
Do you observe this behaviour on Wayland or X11?
Comment 5 Eclipse Genie CLA 2019-07-23 15:37:30 EDT
New Gerrit change created: https://git.eclipse.org/r/146530
Comment 7 Eric Williams CLA 2019-07-29 16:21:10 EDT
(In reply to Eclipse Genie from comment #6)
> Gerrit change https://git.eclipse.org/r/146530 was merged to [master].
> Commit:
> http://git.eclipse.org/c/platform/eclipse.platform.swt.git/commit/
> ?id=78c36186152307c5ba7409e0e24e450dd99b7d9f

In master now.
Comment 8 Eric Williams CLA 2019-08-20 10:19:17 EDT
Verified in I20190820-0600.