chore(color): radio hardware page layout breaks if additional lines added (#6057)

This commit is contained in:
philmoz
2025-04-10 10:04:07 +10:00
committed by GitHub
parent 50b32e6cfc
commit 5d499e6ec4
4 changed files with 10 additions and 14 deletions
@@ -68,6 +68,7 @@ class BTDetailsDialog : public BaseDialog
BluetoothConfigWindow::BluetoothConfigWindow(Window* parent, FlexGridLayout& grid)
{
auto line = parent->newLine(grid);
line->padLeft(PAD_SMALL);
new StaticText(line, rect_t{}, STR_MODE);
auto box = new Window(line, rect_t{});
@@ -93,6 +94,7 @@ BluetoothConfigWindow::BluetoothConfigWindow(Window* parent, FlexGridLayout& gri
// BT radio name
nameEdit = parent->newLine(grid);
nameEdit->padLeft(PAD_SMALL);
new StaticText(nameEdit, rect_t{}, STR_NAME);
box = new Window(nameEdit, rect_t{});
@@ -28,6 +28,7 @@
ExternalModuleWindow::ExternalModuleWindow(Window *parent, FlexGridLayout& grid)
{
auto line = parent->newLine(grid);
line->padLeft(PAD_SMALL);
new StaticText(line, rect_t{}, STR_SAMPLE_MODE);
@@ -32,6 +32,7 @@
InternalModuleWindow::InternalModuleWindow(Window *parent, FlexGridLayout& grid)
{
auto line = parent->newLine(grid);
line->padLeft(PAD_SMALL);
new StaticText(line, rect_t{}, STR_TYPE);
auto internalModule =
new Choice(line, rect_t{}, STR_MODULE_PROTOCOLS, MODULE_TYPE_NONE,
@@ -43,6 +44,7 @@ InternalModuleWindow::InternalModuleWindow(Window *parent, FlexGridLayout& grid)
#if defined(INTERNAL_MODULE_PXX1) && defined(EXTERNAL_ANTENNA)
ant_box = parent->newLine(grid);
ant_box->padLeft(PAD_SMALL);
new StaticText(ant_box, rect_t{}, STR_ANTENNA);
new Choice(
ant_box, rect_t{}, STR_ANTENNA_MODES, ANTENNA_MODE_INTERNAL,
@@ -69,6 +71,7 @@ InternalModuleWindow::InternalModuleWindow(Window *parent, FlexGridLayout& grid)
#if defined(CROSSFIRE)
br_box = parent->newLine(grid);
br_box->padLeft(PAD_SMALL);
new StaticText(br_box, rect_t{}, STR_BAUDRATE);
new Choice(
br_box, rect_t{}, STR_CRSF_BAUDRATE, 0, CROSSFIRE_MAX_INTERNAL_BAUDRATE,
@@ -46,15 +46,13 @@
#if PORTRAIT_LCD
static const lv_coord_t col_dsc[] = {LV_GRID_FR(13), LV_GRID_FR(19),
LV_GRID_TEMPLATE_LAST};
static const lv_coord_t row_dsc[] = {LV_GRID_CONTENT, LV_GRID_CONTENT,
LV_GRID_TEMPLATE_LAST};
#else
static const lv_coord_t col_dsc[] = {LV_GRID_FR(1), LV_GRID_FR(2),
LV_GRID_TEMPLATE_LAST};
static const lv_coord_t row_dsc[] = {LV_GRID_CONTENT, LV_GRID_CONTENT,
LV_GRID_TEMPLATE_LAST};
#endif
static const lv_coord_t row_dsc[] = {LV_GRID_CONTENT, LV_GRID_TEMPLATE_LAST};
RadioHardwarePage::RadioHardwarePage() :
PageTab(STR_HARDWARE, ICON_RADIO_HARDWARE, PAD_TINY)
{
@@ -169,26 +167,18 @@ void RadioHardwarePage::build(Window* window)
FlexGridLayout grid(col_dsc, row_dsc, PAD_TINY);
FormLine* line;
#if defined(HARDWARE_INTERNAL_MODULE)
new Subtitle(window, STR_INTERNALRF);
line = window->newLine(grid);
line->padLeft(PAD_SMALL);
new InternalModuleWindow(line, grid);
new InternalModuleWindow(window, grid);
#endif
#if defined(HARDWARE_EXTERNAL_MODULE)
new Subtitle(window, STR_EXTERNALRF);
line = window->newLine(grid);
line->padLeft(PAD_SMALL);
new ExternalModuleWindow(line, grid);
new ExternalModuleWindow(window, grid);
#endif
#if defined(BLUETOOTH)
new Subtitle(window, STR_BLUETOOTH);
line = window->newLine(grid);
line->padLeft(PAD_SMALL);
new BluetoothConfigWindow(window, grid);
#endif