camelCase to snake_case + rewrite util

rmorshea · rmorshea · commit f1bf70c5f064 · 2023-02-20T23:57:03.000-08:00
diff --git a/docs/source/guides/adding-interactivity/components-with-state/_examples/adding_state_variable/main.py b/docs/source/guides/adding-interactivity/components-with-state/_examples/adding_state_variable/main.py
@@ -25,7 +25,7 @@ def handle_click(event):
     url = sculpture["url"]
 
     return html.div(
-        html.button({"onClick": handle_click}, "Next"),
+        html.button({"on_click": handle_click}, "Next"),
         html.h2(name, " by ", artist),
         html.p(f"({bounded_index + 1} of {len(sculpture_data)})"),
         html.img({"src": url, "alt": alt, "style": {"height": "200px"}}),
diff --git a/docs/source/guides/adding-interactivity/components-with-state/_examples/isolated_state/main.py b/docs/source/guides/adding-interactivity/components-with-state/_examples/isolated_state/main.py
@@ -29,14 +29,14 @@ def handle_more_click(event):
     url = sculpture["url"]
 
     return html.div(
-        html.button({"onClick": handle_next_click}, "Next"),
+        html.button({"on_click": handle_next_click}, "Next"),
         html.h2(name, " by ", artist),
         html.p(f"({bounded_index + 1} or {len(sculpture_data)})"),
         html.img({"src": url, "alt": alt, "style": {"height": "200px"}}),
         html.div(
             html.button(
-                {"onClick": handle_more_click},
-                f"{'Show' if show_more else 'Hide'} details",
+                {"on_click": handle_more_click},
+                f"{('Show' if show_more else 'Hide')} details",
             ),
             (html.p(description) if show_more else ""),
         ),
diff --git a/docs/source/guides/adding-interactivity/components-with-state/_examples/multiple_state_variables/main.py b/docs/source/guides/adding-interactivity/components-with-state/_examples/multiple_state_variables/main.py
@@ -29,14 +29,14 @@ def handle_more_click(event):
     url = sculpture["url"]
 
     return html.div(
-        html.button({"onClick": handle_next_click}, "Next"),
+        html.button({"on_click": handle_next_click}, "Next"),
         html.h2(name, " by ", artist),
         html.p(f"({bounded_index + 1} or {len(sculpture_data)})"),
         html.img({"src": url, "alt": alt, "style": {"height": "200px"}}),
         html.div(
             html.button(
-                {"onClick": handle_more_click},
-                f"{'Show' if show_more else 'Hide'} details",
+                {"on_click": handle_more_click},
+                f"{('Show' if show_more else 'Hide')} details",
             ),
             (html.p(description) if show_more else ""),
         ),
diff --git a/docs/source/guides/adding-interactivity/components-with-state/_examples/when_variables_are_not_enough/main.py b/docs/source/guides/adding-interactivity/components-with-state/_examples/when_variables_are_not_enough/main.py
@@ -31,7 +31,7 @@ def handle_click(event):
     url = sculpture["url"]
 
     return html.div(
-        html.button({"onClick": handle_click}, "Next"),
+        html.button({"on_click": handle_click}, "Next"),
         html.h2(name, " by ", artist),
         html.p(f"({bounded_index + 1} or {len(sculpture_data)})"),
         html.img({"src": url, "alt": alt, "style": {"height": "200px"}}),
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/dict_remove.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/dict_remove.py
@@ -26,17 +26,17 @@ def handle_click(event):
         return handle_click
 
     return html.div(
-        html.button({"onClick": handle_add_click}, "add term"),
+        html.button({"on_click": handle_add_click}, "add term"),
         html.label(
             "Term: ",
-            html.input({"value": term_to_add, "onChange": handle_term_to_add_change}),
+            html.input({"value": term_to_add, "on_change": handle_term_to_add_change}),
         ),
         html.label(
             "Definition: ",
             html.input(
                 {
                     "value": definition_to_add,
-                    "onChange": handle_definition_to_add_change,
+                    "on_change": handle_definition_to_add_change,
                 }
             ),
         ),
@@ -45,7 +45,7 @@ def handle_click(event):
             html.div(
                 {"key": term},
                 html.button(
-                    {"onClick": make_delete_click_handler(term)}, "delete term"
+                    {"on_click": make_delete_click_handler(term)}, "delete term"
                 ),
                 html.dt(term),
                 html.dd(definition),
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/dict_update.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/dict_update.py
@@ -24,20 +24,18 @@ def handle_email_change(event):
         html.label(
             "First name: ",
             html.input(
-                {"value": person["first_name"], "onChange": handle_first_name_change},
+                {"value": person["first_name"], "on_change": handle_first_name_change}
             ),
         ),
         html.label(
             "Last name: ",
             html.input(
-                {"value": person["last_name"], "onChange": handle_last_name_change},
+                {"value": person["last_name"], "on_change": handle_last_name_change}
             ),
         ),
         html.label(
             "Email: ",
-            html.input(
-                {"value": person["email"], "onChange": handle_email_change},
-            ),
+            html.input({"value": person["email"], "on_change": handle_email_change}),
         ),
         html.p(f"{person['first_name']} {person['last_name']} {person['email']}"),
     )
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/list_insert.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/list_insert.py
@@ -16,8 +16,8 @@ def handle_click(event):
 
     return html.div(
         html.h1("Inspiring sculptors:"),
-        html.input({"value": artist_to_add, "onChange": handle_change}),
-        html.button({"onClick": handle_click}, "add"),
+        html.input({"value": artist_to_add, "on_change": handle_change}),
+        html.button({"on_click": handle_click}, "add"),
         html.ul([html.li({"key": name}, name) for name in artists]),
     )
 
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/list_re_order.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/list_re_order.py
@@ -15,8 +15,8 @@ def handle_reverse_click(event):
 
     return html.div(
         html.h1("Inspiring sculptors:"),
-        html.button({"onClick": handle_sort_click}, "sort"),
-        html.button({"onClick": handle_reverse_click}, "reverse"),
+        html.button({"on_click": handle_sort_click}, "sort"),
+        html.button({"on_click": handle_reverse_click}, "reverse"),
         html.ul([html.li({"key": name}, name) for name in artists]),
     )
 
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/list_remove.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/list_remove.py
@@ -24,14 +24,16 @@ def handle_click(event):
 
     return html.div(
         html.h1("Inspiring sculptors:"),
-        html.input({"value": artist_to_add, "onChange": handle_change}),
-        html.button({"onClick": handle_add_click}, "add"),
+        html.input({"value": artist_to_add, "on_change": handle_change}),
+        html.button({"on_click": handle_add_click}, "add"),
         html.ul(
             [
                 html.li(
                     {"key": name},
                     name,
-                    html.button({"onClick": make_handle_delete_click(index)}, "delete"),
+                    html.button(
+                        {"on_click": make_handle_delete_click(index)}, "delete"
+                    ),
                 )
                 for index, name in enumerate(artists)
             ]
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/list_replace.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/list_replace.py
@@ -17,7 +17,7 @@ def handle_click(event):
             html.li(
                 {"key": index},
                 count,
-                html.button({"onClick": make_increment_click_handler(index)}, "+1"),
+                html.button({"on_click": make_increment_click_handler(index)}, "+1"),
             )
             for index, count in enumerate(counters)
         ]
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/moving_dot.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/moving_dot.py
@@ -17,7 +17,7 @@ async def handle_pointer_move(event):
 
     return html.div(
         {
-            "onPointerMove": handle_pointer_move,
+            "on_pointer_move": handle_pointer_move,
             "style": {
                 "position": "relative",
                 "height": "200px",
@@ -36,7 +36,7 @@ async def handle_pointer_move(event):
                     "left": "-10px",
                     "top": "-10px",
                     "transform": f"translate({position['x']}px, {position['y']}px)",
-                },
+                }
             }
         ),
     )
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/moving_dot_broken.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/moving_dot_broken.py
@@ -15,7 +15,7 @@ def handle_pointer_move(event):
 
     return html.div(
         {
-            "onPointerMove": handle_pointer_move,
+            "on_pointer_move": handle_pointer_move,
             "style": {
                 "position": "relative",
                 "height": "200px",
@@ -34,7 +34,7 @@ def handle_pointer_move(event):
                     "left": "-10px",
                     "top": "-10px",
                     "transform": f"translate({position['x']}px, {position['y']}px)",
-                },
+                }
             }
         ),
     )
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/set_remove.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/set_remove.py
@@ -20,7 +20,7 @@ def handle_click(event):
         [
             html.div(
                 {
-                    "onClick": make_handle_click(index),
+                    "on_click": make_handle_click(index),
                     "style": {
                         "height": "30px",
                         "width": "30px",
diff --git a/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/set_update.py b/docs/source/guides/adding-interactivity/dangers-of-mutability/_examples/set_update.py
@@ -17,7 +17,7 @@ def handle_click(event):
         [
             html.div(
                 {
-                    "onClick": make_handle_click(index),
+                    "on_click": make_handle_click(index),
                     "style": {
                         "height": "30px",
                         "width": "30px",
diff --git a/docs/source/guides/adding-interactivity/multiple-state-updates/_examples/delay_before_count_updater.py b/docs/source/guides/adding-interactivity/multiple-state-updates/_examples/delay_before_count_updater.py
@@ -13,7 +13,7 @@ async def handle_click(event):
 
     return html.div(
         html.h1(number),
-        html.button({"onClick": handle_click}, "Increment"),
+        html.button({"on_click": handle_click}, "Increment"),
     )
 
 
diff --git a/docs/source/guides/adding-interactivity/multiple-state-updates/_examples/delay_before_set_count.py b/docs/source/guides/adding-interactivity/multiple-state-updates/_examples/delay_before_set_count.py
@@ -13,7 +13,7 @@ async def handle_click(event):
 
     return html.div(
         html.h1(number),
-        html.button({"onClick": handle_click}, "Increment"),
+        html.button({"on_click": handle_click}, "Increment"),
     )
 
 
diff --git a/docs/source/guides/adding-interactivity/multiple-state-updates/_examples/set_color_3_times.py b/docs/source/guides/adding-interactivity/multiple-state-updates/_examples/set_color_3_times.py
@@ -15,10 +15,10 @@ def handle_reset(event):
 
     return html.div(
         html.button(
-            {"onClick": handle_click, "style": {"backgroundColor": color}}, "Set Color"
+            {"on_click": handle_click, "style": {"backgroundColor": color}}, "Set Color"
         ),
         html.button(
-            {"onClick": handle_reset, "style": {"backgroundColor": color}}, "Reset"
+            {"on_click": handle_reset, "style": {"backgroundColor": color}}, "Reset"
         ),
     )
 
diff --git a/docs/source/guides/adding-interactivity/multiple-state-updates/_examples/set_state_function.py b/docs/source/guides/adding-interactivity/multiple-state-updates/_examples/set_state_function.py
@@ -17,7 +17,7 @@ def handle_click(event):
 
     return html.div(
         html.h1(number),
-        html.button({"onClick": handle_click}, "Increment"),
+        html.button({"on_click": handle_click}, "Increment"),
     )
 
 
diff --git a/docs/source/guides/adding-interactivity/responding-to-events/_examples/button_async_handlers.py b/docs/source/guides/adding-interactivity/responding-to-events/_examples/button_async_handlers.py
@@ -9,7 +9,7 @@ async def handle_event(event):
         await asyncio.sleep(delay)
         print(message)
 
-    return html.button({"onClick": handle_event}, message)
+    return html.button({"on_click": handle_event}, message)
 
 
 @component
diff --git a/docs/source/guides/adding-interactivity/responding-to-events/_examples/button_handler_as_arg.py b/docs/source/guides/adding-interactivity/responding-to-events/_examples/button_handler_as_arg.py
@@ -3,7 +3,7 @@
 
 @component
 def Button(display_text, on_click):
-    return html.button({"onClick": on_click}, display_text)
+    return html.button({"on_click": on_click}, display_text)
 
 
 @component
diff --git a/docs/source/guides/adding-interactivity/responding-to-events/_examples/button_prints_event.py b/docs/source/guides/adding-interactivity/responding-to-events/_examples/button_prints_event.py
@@ -6,7 +6,7 @@ def Button():
     def handle_event(event):
         print(event)
 
-    return html.button({"onClick": handle_event}, "Click me!")
+    return html.button({"on_click": handle_event}, "Click me!")
 
 
 run(Button)
diff --git a/docs/source/guides/adding-interactivity/responding-to-events/_examples/button_prints_message.py b/docs/source/guides/adding-interactivity/responding-to-events/_examples/button_prints_message.py
@@ -6,7 +6,7 @@ def PrintButton(display_text, message_text):
     def handle_event(event):
         print(message_text)
 
-    return html.button({"onClick": handle_event}, display_text)
+    return html.button({"on_click": handle_event}, display_text)
 
 
 @component
diff --git a/docs/source/guides/adding-interactivity/responding-to-events/_examples/prevent_default_event_actions.py b/docs/source/guides/adding-interactivity/responding-to-events/_examples/prevent_default_event_actions.py
@@ -7,7 +7,7 @@ def DoNotChangePages():
         html.p("Normally clicking this link would take you to a new page"),
         html.a(
             {
-                "onClick": event(lambda event: None, prevent_default=True),
+                "on_click": event(lambda event: None, prevent_default=True),
                 "href": "https://google.com",
             },
             "https://google.com",
diff --git a/docs/source/guides/adding-interactivity/responding-to-events/_examples/stop_event_propagation.py b/docs/source/guides/adding-interactivity/responding-to-events/_examples/stop_event_propagation.py
@@ -9,23 +9,23 @@ def DivInDiv():
 
     div_in_div = html.div(
         {
-            "onClick": lambda event: set_outer_count(outer_count + 1),
+            "on_click": lambda event: set_outer_count(outer_count + 1),
             "style": {"height": "100px", "width": "100px", "backgroundColor": "red"},
         },
         html.div(
             {
-                "onClick": event(
+                "on_click": event(
                     lambda event: set_inner_count(inner_count + 1),
                     stop_propagation=stop_propagatation,
                 ),
                 "style": {"height": "50px", "width": "50px", "backgroundColor": "blue"},
-            },
+            }
         ),
     )
 
     return html.div(
         html.button(
-            {"onClick": lambda event: set_stop_propagatation(not stop_propagatation)},
+            {"on_click": lambda event: set_stop_propagatation(not stop_propagatation)},
             "Toggle Propogation",
         ),
         html.pre(f"Will propagate: {not stop_propagatation}"),
diff --git a/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/delayed_print_after_set.py b/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/delayed_print_after_set.py
@@ -15,7 +15,7 @@ async def handle_click(event):
 
     return html.div(
         html.h1(number),
-        html.button({"onClick": handle_click}, "Increment"),
+        html.button({"on_click": handle_click}, "Increment"),
     )
 
 
diff --git a/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/print_chat_message.py b/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/print_chat_message.py
@@ -16,13 +16,14 @@ async def handle_submit(event):
         print(f"Sent '{message}' to {recipient}")
 
     return html.form(
-        {"onSubmit": handle_submit, "style": {"display": "inline-grid"}},
+        {"on_submit": handle_submit, "style": {"display": "inline-grid"}},
         html.label(
+            {},
             "To: ",
             html.select(
                 {
                     "value": recipient,
-                    "onChange": lambda event: set_recipient(event["target"]["value"]),
+                    "on_change": lambda event: set_recipient(event["target"]["value"]),
                 },
                 html.option({"value": "Alice"}, "Alice"),
                 html.option({"value": "Bob"}, "Bob"),
@@ -33,7 +34,7 @@ async def handle_submit(event):
                 "type": "text",
                 "placeholder": "Your message...",
                 "value": message,
-                "onChange": lambda event: set_message(event["target"]["value"]),
+                "on_change": lambda event: set_message(event["target"]["value"]),
             }
         ),
         html.button({"type": "submit"}, "Send"),
diff --git a/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/print_count_after_set.py b/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/print_count_after_set.py
@@ -11,7 +11,7 @@ def handle_click(event):
 
     return html.div(
         html.h1(number),
-        html.button({"onClick": handle_click}, "Increment"),
+        html.button({"on_click": handle_click}, "Increment"),
     )
 
 
diff --git a/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/send_message.py b/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/send_message.py
@@ -10,7 +10,7 @@ def App():
         return html.div(
             html.h1("Message sent!"),
             html.button(
-                {"onClick": lambda event: set_is_sent(False)}, "Send new message?"
+                {"on_click": lambda event: set_is_sent(False)}, "Send new message?"
             ),
         )
 
@@ -20,12 +20,12 @@ def handle_submit(event):
         set_is_sent(True)
 
     return html.form(
-        {"onSubmit": handle_submit, "style": {"display": "inline-grid"}},
+        {"on_submit": handle_submit, "style": {"display": "inline-grid"}},
         html.textarea(
             {
                 "placeholder": "Your message here...",
                 "value": message,
-                "onChange": lambda event: set_message(event["target"]["value"]),
+                "on_change": lambda event: set_message(event["target"]["value"]),
             }
         ),
         html.button({"type": "submit"}, "Send"),
diff --git a/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/set_counter_3_times.py b/docs/source/guides/adding-interactivity/state-as-a-snapshot/_examples/set_counter_3_times.py
diff --git a/docs/source/guides/managing-state/sharing-component-state/_examples/filterable_list/main.py b/docs/source/guides/managing-state/sharing-component-state/_examples/filterable_list/main.py
diff --git a/docs/source/guides/managing-state/sharing-component-state/_examples/synced_inputs/main.py b/docs/source/guides/managing-state/sharing-component-state/_examples/synced_inputs/main.py
diff --git a/docs/source/reference/_examples/character_movement/main.py b/docs/source/reference/_examples/character_movement/main.py
diff --git a/src/idom/__main__.py b/src/idom/__main__.py
diff --git a/src/idom/_console/ast_utils.py b/src/idom/_console/ast_utils.py
diff --git a/src/idom/_console/rewrite_camel_case_props.py b/src/idom/_console/rewrite_camel_case_props.py
diff --git a/src/idom/_console/rewrite_keys.py b/src/idom/_console/rewrite_keys.py
diff --git a/src/idom/utils.py b/src/idom/utils.py
diff --git a/tests/test__console/test_rewrite_camel_case_props.py b/tests/test__console/test_rewrite_camel_case_props.py
diff --git a/tests/test_core/test_layout.py b/tests/test_core/test_layout.py
diff --git a/tests/test_html.py b/tests/test_html.py
diff --git a/tests/test_testing.py b/tests/test_testing.py
diff --git a/tests/test_utils.py b/tests/test_utils.py

Original file line number	Diff line number	Diff line change
`@@ -16,8 +16,8 @@ def handle_click(event):`
`16`	`16`
`17`	`17`	`return html.div(`
`18`	`18`	`html.h1("Inspiring sculptors:"),`
`19`		`- html.input({"value": artist_to_add, "onChange": handle_change}),`
`20`		`- html.button({"onClick": handle_click}, "add"),`
	`19`	`+ html.input({"value": artist_to_add, "on_change": handle_change}),`
	`20`	`+ html.button({"on_click": handle_click}, "add"),`
`21`	`21`	`html.ul([html.li({"key": name}, name) for name in artists]),`
`22`	`22`	`)`
`23`	`23`
Original file line number	Diff line number	Diff line change
`@@ -15,8 +15,8 @@ def handle_reverse_click(event):`
`15`	`15`
`16`	`16`	`return html.div(`
`17`	`17`	`html.h1("Inspiring sculptors:"),`
`18`		`- html.button({"onClick": handle_sort_click}, "sort"),`
`19`		`- html.button({"onClick": handle_reverse_click}, "reverse"),`
	`18`	`+ html.button({"on_click": handle_sort_click}, "sort"),`
	`19`	`+ html.button({"on_click": handle_reverse_click}, "reverse"),`
`20`	`20`	`html.ul([html.li({"key": name}, name) for name in artists]),`
`21`	`21`	`)`
`22`	`22`
Original file line number	Diff line number	Diff line change
`@@ -24,14 +24,16 @@ def handle_click(event):`
`24`	`24`
`25`	`25`	`return html.div(`
`26`	`26`	`html.h1("Inspiring sculptors:"),`
`27`		`- html.input({"value": artist_to_add, "onChange": handle_change}),`
`28`		`- html.button({"onClick": handle_add_click}, "add"),`
	`27`	`+ html.input({"value": artist_to_add, "on_change": handle_change}),`
	`28`	`+ html.button({"on_click": handle_add_click}, "add"),`
`29`	`29`	`html.ul(`
`30`	`30`	`[`
`31`	`31`	`html.li(`
`32`	`32`	`{"key": name},`
`33`	`33`	`name,`
`34`		`- html.button({"onClick": make_handle_delete_click(index)}, "delete"),`
	`34`	`+ html.button(`
	`35`	`+ {"on_click": make_handle_delete_click(index)}, "delete"`
	`36`	`+ ),`
`35`	`37`	`)`
`36`	`38`	`for index, name in enumerate(artists)`
`37`	`39`	`]`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ def handle_click(event):`
`17`	`17`	`html.li(`
`18`	`18`	`{"key": index},`
`19`	`19`	`count,`
`20`		`- html.button({"onClick": make_increment_click_handler(index)}, "+1"),`
	`20`	`+ html.button({"on_click": make_increment_click_handler(index)}, "+1"),`
`21`	`21`	`)`
`22`	`22`	`for index, count in enumerate(counters)`
`23`	`23`	`]`