Skip to content

Commit cbedf50

Browse files
committed
Modernize super() calls and fix formatting in wx_widgets.py
- Replace super(Class, self) with modern super() syntax - Break long lines for better PEP 8 compliance - Add type annotation for SizedPanel pane variable - Fix SizedFrame.set_padding() to use correct SetSizerProp API
1 parent 03918d2 commit cbedf50

File tree

1 file changed

+14
-8
lines changed

1 file changed

+14
-8
lines changed

gui_builder/widgets/wx_widgets.py

Lines changed: 14 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -802,7 +802,9 @@ def get_x_y_from_insertion_point(self, insertion_point: int) -> Tuple[int, int]:
802802
def clear(self) -> None:
803803
return self.control.Clear()
804804

805-
def _to_wx_colour(self, color: Union[wx.Colour, str, Tuple[int, int, int]]) -> wx.Colour:
805+
def _to_wx_colour(
806+
self, color: Union[wx.Colour, str, Tuple[int, int, int]]
807+
) -> wx.Colour:
806808
"""Convert various color formats to wx.Colour.
807809
808810
Args:
@@ -912,7 +914,9 @@ def _text_attr_to_dict(self, attr: wx.TextAttr) -> Dict[str, Any]:
912914
result["text_color"] = self._from_wx_colour(attr.GetTextColour())
913915

914916
if attr.HasBackgroundColour():
915-
result["background_color"] = self._from_wx_colour(attr.GetBackgroundColour())
917+
result["background_color"] = self._from_wx_colour(
918+
attr.GetBackgroundColour()
919+
)
916920

917921
if attr.HasAlignment():
918922
result["alignment"] = attr.GetAlignment()
@@ -1565,11 +1569,11 @@ class SizedPanel(BaseContainer[FieldType, sc.SizedPanel]):
15651569
focusable = False
15661570

15671571
def __init__(self, sizer_type="vertical", *args, **kwargs):
1568-
super(SizedPanel, self).__init__(*args, **kwargs)
1572+
super().__init__(*args, **kwargs)
15691573
self.sizer_type = sizer_type
15701574

15711575
def render(self):
1572-
super(SizedPanel, self).render()
1576+
super().render()
15731577
self.control.SetSizerType(self.sizer_type)
15741578

15751579

@@ -1584,7 +1588,7 @@ def __init__(self, maximized: bool = False, *args, **kwargs):
15841588
super(BaseFrame, self).__init__(*args, **kwargs)
15851589

15861590
def render(self, *args, **kwargs):
1587-
super(BaseFrame, self).render(*args, **kwargs)
1591+
super().render(*args, **kwargs)
15881592
if self.control_maximized:
15891593
self.maximize()
15901594

@@ -1618,8 +1622,8 @@ def set_content_padding(self, padding: int) -> None:
16181622
Args:
16191623
padding (int): Padding in pixels around all content
16201624
"""
1621-
pane = self.control.GetContentsPane()
1622-
pane.SetSizerProps(border=("all", padding))
1625+
pane: sc.SizedPanel = self.control.GetContentsPane()
1626+
pane.SetSizerProp("border", padding)
16231627

16241628

16251629
class MDIParentFrame(BaseFrame[FieldType, wx.MDIParentFrame]):
@@ -1666,7 +1670,9 @@ def on_navigation_key(evt: wx.Event):
16661670

16671671
# Don't interfere with Ctrl+Tab or Ctrl+PageUp/PageDown - let wxPython handle tab switching
16681672
if self.control_down:
1669-
logger.debug("Ctrl pressed at page level, allowing default notebook tab switching")
1673+
logger.debug(
1674+
"Ctrl pressed at page level, allowing default notebook tab switching"
1675+
)
16701676
nav_evt.Skip()
16711677
return
16721678

0 commit comments

Comments
 (0)