code review

nicegui/elements/expansion.py

@@ -22,7 +22,7 @@ class Expansion(TextElement, ValueElement, DisableableElement):
         :param text: title text
         :param caption: optional caption (or sub-label) text
         :param icon: optional icon (default: None)
-        :param group: optional group name for coordinated open/close state within the group a.k.a. 'accordion mode'
+        :param group: optional group name for coordinated open/close state within the group a.k.a. "accordion mode"
         :param value: whether the expansion should be opened on creation (default: `False`)
         :param on_value_change: callback to execute when value changes
         """

tests/test_expansion.py

@@ -18,27 +18,29 @@ def test_open_close_expansion(screen: Screen):
     screen.wait(0.5)
     screen.should_not_contain('Content')
 
+
 def test_caption(screen: Screen):
-    with ui.expansion('Expansion', caption='Caption') as e:
+    with ui.expansion('Expansion', caption='Caption'):
         ui.label('Content')
 
     screen.open('/')
     screen.should_contain('Expansion')
     screen.should_contain('Caption')
     screen.should_not_contain('Content')
-    
+
     screen.click('Expansion')
     screen.wait(0.5)
     screen.should_contain('Expansion')
     screen.should_contain('Caption')
     screen.should_contain('Content')
 
+
 def test_group(screen: Screen):
-    with ui.expansion('Expansion A', group='group') as a:
+    with ui.expansion('Expansion A', group='group'):
         ui.label('Content A')
-    with ui.expansion('Expansion B', group='group') as b:
+    with ui.expansion('Expansion B', group='group'):
         ui.label('Content B')
-    with ui.expansion('Expansion C', group='group') as c:
+    with ui.expansion('Expansion C', group='group'):
         ui.label('Content C')
 
     screen.open('/')

website/documentation/content/expansion_documentation.py

@@ -28,7 +28,7 @@ def expansion_with_caption():
 
 
 @doc.demo('Expansion with Grouping', '''
-    An expansion group can be defined to enable coordinated open/close states a.k.a. 'accordion mode'.
+    An expansion group can be defined to enable coordinated open/close states a.k.a. "accordion mode".
 ''')
 def expansion_with_grouping():
     with ui.expansion(text='Expand One!', group='group'):