IconsHelper can support NotStr icons

src/myfasthtml/controls/HierarchicalCanvasGraph.py

@@ -153,43 +153,6 @@ class HierarchicalCanvasGraph(MultipleInstance):
     """
     return self._state
   
-  # def get_selected_id(self) -> Optional[str]:
-  #   """Get the currently selected node ID.
-  #
-  #   Returns:
-  #       str or None: The selected node ID, or None if no selection
-  #   """
-  #   return self._state.ns_selected_id
-  #
-  # def set_collapsed(self, node_ids: set):
-  #   """Set the collapsed state of nodes.
-  #
-  #   Args:
-  #       node_ids: Set of node IDs to mark as collapsed
-  #   """
-  #   self._state.collapsed = list(node_ids)
-  #   logger.debug(f"set_collapsed: {len(node_ids)} nodes collapsed")
-  #
-  # def toggle_node(self, node_id: str):
-  #   """Toggle the collapsed state of a node.
-  #
-  #   Args:
-  #       node_id: The ID of the node to toggle
-  #
-  #   Returns:
-  #       self: For chaining
-  #   """
-  #   collapsed_set = set(self._state.collapsed)
-  #   if node_id in collapsed_set:
-  #     collapsed_set.remove(node_id)
-  #     logger.debug(f"toggle_node: expanded {node_id}")
-  #   else:
-  #     collapsed_set.add(node_id)
-  #     logger.debug(f"toggle_node: collapsed {node_id}")
-  #
-  #   self._state.collapsed = list(collapsed_set)
-  #   return self
-  
   def handle_update_view_state(self, transform: Optional[dict] = None, layout_mode: Optional[str] = None):
     """Internal handler to update view state from client.