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

layer-shell: add support for popups #4307

Merged
merged 1 commit into from
Aug 14, 2019
Merged

layer-shell: add support for popups #4307

merged 1 commit into from
Aug 14, 2019

Conversation

ddevault
Copy link
Contributor

@ddevault ddevault commented Jul 5, 2019

Supports nested popups as well.

sway/desktop/layer_shell.c Outdated Show resolved Hide resolved
sway/desktop/layer_shell.c Outdated Show resolved Hide resolved
@ddevault

This comment has been minimized.

@ddevault ddevault force-pushed the layer-shell-popups branch 2 times, most recently from b8ae21f to 60a8cfd Compare July 5, 2019 20:40
@ddevault
Copy link
Contributor Author

ddevault commented Jul 5, 2019

This should be ready to go.

sway/desktop/layer_shell.c Outdated Show resolved Hide resolved
@ddevault ddevault force-pushed the layer-shell-popups branch 2 times, most recently from ac9d297 to 08ad82b Compare July 7, 2019 15:54
@ddevault
Copy link
Contributor Author

Bump

@luispabon
Copy link

I'm testing this together with mate's wayland port of their panel.

  • Add panel
  • Add application menu
  • Click on it
  • Click away to close it

image

Similar stuff happens with context menues on the panel.

sway/desktop/layer_shell.c Outdated Show resolved Hide resolved
@Kommynct
Copy link

Is this going to make it into 1.2 or is it planned for later?

@emersion
Copy link
Member

No, only bugfixes can make it. I was away so I couldn't get this reviewed in time for 1.2.

Copy link
Member

@emersion emersion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@emersion emersion merged commit cb8f68d into master Aug 14, 2019
@emersion emersion deleted the layer-shell-popups branch August 14, 2019 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants