-
Notifications
You must be signed in to change notification settings - Fork 669
Migrate UI to textual #3997
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
base: master
Are you sure you want to change the base?
Migrate UI to textual #3997
Conversation
…nto migrate-textual
|
Looks cool @svartkanin :D Tested a little bit, I came across was f1 help screen crashed and user setup screen was missing The rest seems like it's already fully functional (considering what you said about the flicker), like how you can use cursor, very bamby friendly 🖱️ Would be cool to add globally visible hints for Then more specific go in help screen ? |
|
Thanks @h8d13 for testing!
|
|
Most submenus (The ones with a |
|
Yeah I thought it would be a good thing if it also worked in these menus reduce keys needed to get around Had another question, I saw that runners were going to be more restrictive (and paid) for github and always thought the ISO one takes a while (seems unnecessary for smaller changes) . Is there anyway we can disable it unless on major releases/needed ? |


This PR migrates all menus to the
textualframework introduced with #3879.The current implementation is creating a new textual app and destroying it with every menu/submenu which is not how it is supposed to be.
The final version will use a single app instance and switch between screens smoothly but to keep things (somewhat) manageable, that change will be done in a follow-up PR in which the code will be moved to an async model (see the wifi screen that is already doing that https://github.com/svartkanin/archinstall/blob/fdb002fa31471d86e1d5e99bc6ec4c21ba1a3b15/archinstall/lib/network/wifi_handler.py?plain=1#L1)
I have tested everything somewhat but not extensively yet. I have opened the PR regardles for now in case someone wants to have a go at it and provide some feedback.
[X] Test everything thoroughly