Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pop up Windows sometimes aren't shown correctly #10

Closed
craigloewen-msft opened this issue Feb 11, 2021 · 15 comments
Closed

Pop up Windows sometimes aren't shown correctly #10

craigloewen-msft opened this issue Feb 11, 2021 · 15 comments
Assignees
Labels
bug Something isn't working fixinbound

Comments

@craigloewen-msft
Copy link
Member

Environment

Windows build number: 21298 rsmaster
Your Distribution version: Ubuntu 20.04
Your WSLG version: 0.2.11

Steps to reproduce

  • Install gitkraken (I used the download link and installed the .deb https://www.gitkraken.com/download )
  • Run gitkraken
  • Open a repository by clicking 'Browse' but do not select one
  • Navigate away from the Window and open other Windows infront of it
  • Click on gitkraken again
  • Only the gitkraken window will pop up, and it will be non-interactable. I'd expect the modal Window to pop up too.

Expected behavior

The pop up Window should pop up in place over the git kraken Window.

Actual behavior

The pop up Window remains hidden behind all other Windows, causing the user to go find it before they can go interact with the app again.

gif4

@hideyukn88
Copy link
Member

Yes, this is known issue and we need proper Z order reporting with RDP/RAIL protocol.

@JYInMyHeart
Copy link

Really hope this problem can be fixed as soon as possible🥺.

@JYInMyHeart
Copy link

Really hope this problem can be fixed as soon as possible🥺.

@hideyukn88
Copy link
Member

@JYInMyHeart , thanks for your feedback. I just submit the fix for this issue. Once the release with fix is out, please try it, thanks!

@JYInMyHeart
Copy link

@JYInMyHeart , thanks for your feedback. I just submit the fix for this issue. Once the release with fix is out, please try it, thanks!

OK, no problem. Thank you, too.

@onomatopellan
Copy link
Contributor

onomatopellan commented May 12, 2021

@hideyukn88 I'm testing the fix and now some wayland apps like epiphany browser won't show the menu, although I can see it in the taskbar thumbnail and I can open and see the About dialog. With WSLg 1.0.20 it shows the menu but it doesn't show the About dialog.

sudo apt install epiphany-browser
epiphany

2021-05-12 (1)

$ cat /mnt/wslg/versions.txt
WSLg ( x86_64 ): <current>
Mariner: VERSION="1.0.20210224"
FreeRDP: 5f083fa0b97d433d6204985f6047886e29c1c61e
weston: 0af13ff2a2d8af2f9a0fe33f6b6a01f06c20a734
pulseaudio: 2f0f0b8c3872780f15e275fc12899f4564f01bd5
mesa: 2ad0684038f5732f7e4bd1a391ec9d833685fb48

@hideyukn88
Copy link
Member

@onomatopellan , thank you very much for report, I confirmed the issue, it looks wl_subsurface is not properly handled, I will look into more details, thanks!

@Tib-z
Copy link

Tib-z commented May 15, 2021

@JYInMyHeart , thanks for your feedback. I just submit the fix for this issue. Once the release with fix is out, please try it, thanks!

I'm testing the issue with phpStorm popup windows reported in #96

Just updated to 21382, but the problem still exists. Is this fix included in the 21382 build?
Running wsl --update says
Checking for updates...
No updates are available.
Kernel version: 5.10.16

Do I miss something? How should I update WSLg to run on the latest version and test this fix?

Details:
Windows build number: 10.0.21382.0
Distribution version: Ubuntu-20.04
WSLG version: 1.0.19

cat /mnt/wslg/versions.txt
WSLg ( x86_64 ): 1.0.19+Branch.main.Sha.2a47bf20c239b523d7413f315cbc9ee60dfde831
Mariner: VERSION="1.0.20210224"
FreeRDP: 5f083fa0b97d433d6204985f6047886e29c1c61e
weston: 16de531f00aa3dfd17e0de74c8f49e9fd7cec617
pulseaudio: 2f0f0b8c3872780f15e275fc12899f4564f01bd5
mesa:

@onomatopellan
Copy link
Contributor

onomatopellan commented May 15, 2021

@Carobit There is no need to update the Windows build. Most important part from WSLg is the System distro package that comes via Windows update. You can wait for the updates or also just create a system.vhd with the latest fixes following the guide.

Thanks to latest @hideyukn88 fixes I compiled a new system.vhd and I have no more problems in wayland apps like Epiphany-browser.

The PhpStorm issue is not fixed though because it's actually a bug that also happens in LInux. See https://youtrack.jetbrains.com/issue/IDEA-265390

@JYInMyHeart
Copy link

@onomatopellan I had seen this jetbrains issue. But in x-server such as GWSL there is no problem to use find in files.

@onomatopellan
Copy link
Contributor

@JYInMyHeart Interesting. Then it could also be a XWayland bug.

@JYInMyHeart
Copy link

Maybe. Now this issue and #188 still prevent me to use wslg.

@hideyukn88
Copy link
Member

@onomatopellan , thanks for verifying the my change, very appreciate. Yes, PnpStrom's "File In Files" issue is not Z order issue, since window is closed immediately.

@Carobit , currently we haven't released the update with this fix, you will need to build system-distro VHD by yourself to try out the fix at this point, but I'm hoping this will be released soon.

@Tib-z
Copy link

Tib-z commented May 17, 2021

Thanks for the information. For me too, this issue and the #188 is a blocker for now.. Also sometimes the main menu disappears like the find in file popup, and I noticed some minor gui glitches. Looking forward to have these issues resolved, as wslg is a pretty promising feature.

@spronovo
Copy link
Collaborator

WSLg 1.0.22 was just released externally. You can pick it up by doing the following from an elevated command prompt:

wsl --shutdown
wsl --update

This contains Hideyuki's fix for the z ordering issue and should address most issues with child window poping behind their parents. I'm going to close this issue. If you are still experiencing issue with some applications, please open new issue with details of the repro so we can track those down. For phpstorm, i've reactivated #96 to track that one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fixinbound
Projects
None yet
Development

No branches or pull requests

6 participants