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

Failsafe exit setting for awful.popup in doc example #3959

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

elig0n
Copy link
Contributor

@elig0n elig0n commented Sep 24, 2024

The first awful.widget.layoutlist popup example does not show how to bind the popup to a widget or a button press.

Nevertheless the popup is automatically shown upon creation so I would consider adding visible = false to avoid that and let a binding show it.

Also it is not removed from display unless the user picks something, not even after an awesome restart, so I think adding this option which lets the unaware user click the right mouse button to make the popup disappear without being forced to select a layout should be set as a sane default/protection.

I would also consider adding ontop so it is not covered by clients/more visible when active since it is usually an immediate type of action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant