weyne/dcs_liberation
1
0
Fork 0
mirror of https://github.com/dcs-liberation/dcs_liberation.git synced 2025-11-10 14:22:26 +00:00
Code Issues Packages Projects Releases Wiki Activity

Do not draw empty property rows.

Browse Source
This commit is contained in:
Dan Albert 2023-07-22 17:12:01 -07:00
parent 6c5b35d704
commit efc2915628
1 changed files with 20 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
qt_ui/windows/mission/flight/payload/propertyeditor.py
View File

@ -1,3 +1,4 @@
import itertools
import logging
from typing import Callable
@ -23,7 +24,9 @@ class PropertyEditor(QGridLayout):
self.flight_member = flight_member
self.flight_member_update_listeners: list[Callable[[FlightMember], None]] = []
for row, prop in enumerate(flight.unit_type.iter_props()):
last_label: QWidget | None = None
row = itertools.count()
for prop in flight.unit_type.iter_props():
if prop.label is None:
if prop.control != "label":
logging.error(
@ -44,15 +47,26 @@ class PropertyEditor(QGridLayout):
)
continue
# Draw any deferred label if this is a real control.
if last_label is not None and widget is not None:
self.addWidget(last_label, next(row), 0)
last_label = None
label = prop.label
if widget is None:
label = f"<strong>{label}</label>"
self.addWidget(QLabel(label), row, 0)
# If prop.control is "label", widget will be None. We only need to add the
# label, not the control.
if widget is not None:
self.addWidget(widget, row, 1)
label = QLabel(label)
if widget is None:
# The "control" is only a section label. Defer adding it to the layout
# so that we can skip empty sections.
last_label = label
else:
# Else the label is for the control itself and should be drawn
# immediately.
this_row = next(row)
self.addWidget(label, this_row, 0)
self.addWidget(widget, this_row, 1)
def control_for_property(self, prop: UnitPropertyDescription) -> QWidget | None:
# Valid values are: