simplify types

src/textual/app.py

@@ -190,8 +190,7 @@ DEFAULT_COLORS = {
 
 ComposeResult = Iterable[Widget]
 RenderResult = RenderableType | Visual | SupportsTextualize
-ContentType = RenderableType | Visual | SupportsTextualize
-"""Any content that can be rendered."""
+"""Result of Widget.render()"""
 
 AutopilotCallbackType: TypeAlias = (
     "Callable[[Pilot[object]], Coroutine[Any, Any, None]]"

src/textual/content.py

@@ -238,10 +238,6 @@ class Content(Visual):
         lines = self.without_spans.split("\n")
         return max(line.expand_tabs(8).cell_length for line in lines)
 
-    # def textualize(self) -> Content:
-    #     ""
-    #     return self
-
     def render_strips(
         self,
         widget: Widget,

src/textual/visual.py

@@ -61,10 +61,12 @@ def visualize(widget: Widget, obj: object) -> Visual:
     if isinstance(obj, Visual):
         # Already a visual
         return obj
+    # The visualize method should return a Visual if present.
     visualize = getattr(obj, "visualize", None)
     if visualize is None:
         # Doesn't expose the textualize protocol
         if is_renderable(obj):
+            # If it is a string, render it to Text
             if isinstance(obj, str):
                 obj = widget.render_str(obj)
 

src/textual/widgets/_option_list.py

@@ -22,7 +22,7 @@ from textual.visual import Padding, Visual, visualize
 if TYPE_CHECKING:
     from typing_extensions import Self, TypeAlias
 
-    from textual.app import ContentType
+    from textual.app import RenderResult
 
 
 class DuplicateID(Exception):
@@ -42,7 +42,7 @@ class Option:
     """Class that holds the details of an individual option."""
 
     def __init__(
-        self, prompt: ContentType, id: str | None = None, disabled: bool = False
+        self, prompt: RenderResult, id: str | None = None, disabled: bool = False
     ) -> None:
         """Initialise the option.
 
@@ -56,11 +56,11 @@ class Option:
         self.disabled = disabled
 
     @property
-    def prompt(self) -> ContentType:
+    def prompt(self) -> RenderResult:
         """The prompt for the option."""
         return self._prompt
 
-    def set_prompt(self, prompt: ContentType) -> None:
+    def set_prompt(self, prompt: RenderResult) -> None:
         """Set the prompt for the option.
 
         Args:
@@ -81,7 +81,7 @@ class Option:
         yield "id", self.id, None
         yield "disabled", self.disabled, False
 
-    def __rich__(self) -> ContentType:
+    def __rich__(self) -> RenderResult:
         return self._prompt
 
 
@@ -459,7 +459,7 @@ class OptionList(ScrollView, can_focus=True):
         return Option(content)
 
     def _render_option_content(
-        self, option_index: int, content: ContentType, component_class: str, width: int
+        self, option_index: int, content: RenderResult, component_class: str, width: int
     ) -> list[Strip]:
         """Render content for option and style.
 
@@ -874,12 +874,6 @@ class OptionList(ScrollView, can_focus=True):
                 elif mouse_over:
                     component_class = "option-list--option-hover"
 
-        # style = (
-        #     self.get_component_rich_style(component_class)
-        #     if component_class
-        #     else self.rich_style
-        # )
-
         strips = self._render_option_content(
             option_index,
             renderable,