-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
Some issues discovered with whiptail output (BMC tested: smaller screen) #1237
Comments
Attempted to pass vga=795 to kernel, but without change in behavior. |
But unfortunately, not everything fits under 20x90. One of those is oem-factory-reset script: I'll continue to try to find a way to increase ast2500 console settings. |
So @JonathonHall-Purism I understand fixating 80 (the width to minimal display size) or maybe configure it under board configs. |
Definitely agree with dynamic height. It doesn't make sense to leave those small prompts swimming in empty space, especially for short yes/no where the prompt and selections are separated by a lot of blank lines. Re: width, just from having looked at the above, I actually like the dynamic width. It looks like all our prompts have hard line breaks anyway to accommodate fbwhiptail, so I think the dynamic width looks best. The main boot menu in particular looks better without a bunch of extra whitespace, IMO. I don't think there is a UX consistency issue (on the contrary, I think it is easier to read at a glance). But otherwise, I'd say to fix it globally to 80, not make it board-specific. IMO being able to raise the width per board is not that useful right now since the prompts need hard line breaks for fbwhiptail anyway. |
@JonathonHall-Purism there is some place in code that are using busybox's "fold" for long strings, without having to fuss around into manipulating strings to match constant width. From my succing tests, I haven't seen any regression in GUIs setting the width to 80. Will push PR soon to set width to 80 and height to 0 (dynamic). |
Ok, testing those on talos II where artifacts under https://app.circleci.com/pipelines/github/tlaurion/heads/1253/workflows/08e0593b-1726-4af7-a7f5-059c740bdef2/jobs/11935/artifacts were flashed. Which are linked to talos II specific : https://github.com/tlaurion/heads/tree/SergiiDmytruk_flashrom_fff474a_9bb6be887457a3cb28ac831c1547a33752429184 And branch on top of master https://github.com/tlaurion/heads/tree/whiptail_dynamic_height_fixated_width_to_80 Some output with 0x80 over BMC: |
Some tests conducted on https://github.com/osresearch/heads/tree/1cb0324a12aa4aec01287142c76b5a41ed51a2cb
Whiptail can fixate (optional) the size of its windows with calls of Height x Width.
As of now on master, it's 30 x 90, where 90 is too wide for BMC output. This causes some issues.
Also, some output is considered to be of a static height, causing issues as well.
It is understandable to want to have fixated Height x Width to have consistent UX through menus, but that should not cause bogus output.
Main menu (gui-init's show_main_menu function):
30x90
![2022-11-08-133047](https://user-images.githubusercontent.com/827570/200660936-f9763ff5-2571-431b-a6ef-8c30e8050a26.png)
30x80
![2022-11-08-150634](https://user-images.githubusercontent.com/827570/200664510-c7da4aa2-9350-49b8-8859-49b660270e9d.png)
20x80:
![2022-11-08-144920](https://user-images.githubusercontent.com/827570/200661188-294f3d21-e02a-4d7f-ad17-1ba3274cc1d4.png)
0x80:
![2022-11-08-150908](https://user-images.githubusercontent.com/827570/200664959-c7d3ec15-88b7-4b0f-a8f0-d652d3b7a1c0.png)
0x0
![2022-11-08-150808](https://user-images.githubusercontent.com/827570/200664796-dcab76b6-8ed7-47a9-b3cf-6caff6c75292.png)
Some menus just don't show properly without proper selections, as seen above, but also can hide option boxes:
Here, tampered with a single kexec_hashes.txt entree and selecting default boot option.
This happens under gui-init's verify_global_hashes:
30x90:
![2022-11-08-151802](https://user-images.githubusercontent.com/827570/200666722-ebd83533-85ba-4c12-be43-95224c14eca3.png)
30x80:
![2022-11-08-151445](https://user-images.githubusercontent.com/827570/200666104-d87d9680-e607-4838-8add-2ac47afa9954.png)
20x80:
![2022-11-08-151940](https://user-images.githubusercontent.com/827570/200666913-1b41923a-8181-4ef9-bcf1-e69ff9c4cc05.png)
0x80
![2022-11-08-152047](https://user-images.githubusercontent.com/827570/200667106-1812cf22-6a58-4107-9d3e-59342e658806.png)
The text was updated successfully, but these errors were encountered: