-
Notifications
You must be signed in to change notification settings - Fork 829
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
Select widget #2501
Select widget #2501
Conversation
@@ -38,6 +38,9 @@ class Message: | |||
namespace: ClassVar[str] = "" # Namespace to disambiguate messages | |||
|
|||
def __init__(self) -> None: | |||
self.__post_init__() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This allows creating messages from dataclasses.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Second coffee of the morning read-through and code play, couple of questions.
Co-authored-by: Dave Pearson <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Main questions resolved, looks good to go for me. Looking forward to making use of this.
I'm actually really taken by how good and smooth it all looks.
overlay:
andconstrain:
.Region.translate_inside
Setting
overlay: screen
display a widget above everything else. Like there is an implicit top level layer.Setting
constrain:
tox
,y
, orboth
ensures that the widget doesn't get clipped of the edge of the screen.This two rules are undocumented as they are experimental. They simplify the implementation of pop-up controls like this Select widget. In the future we can also use them for auto-complete lists and tool tips.
Screen.Recording.2023-05-07.at.14.51.10.mov