refactor: enhance _BaseSlide class with background and name properties; update Slide class for improved functionality

Author: ya7010

src/tppt/pptx/snotes_slide.py renamed to src/tppt/pptx/notes_slide.py

@@ -19,7 +19,7 @@
 from tppt.types._length import Length, LiteralLength
 
 from .converter import PptxConvertible, to_pptx_length, to_tppt_length
-from .slide import SlideBuilder
+from .slide import SlideBuilder, _BaseSlide
 
 if TYPE_CHECKING:
     from tppt.pptx.shape import BaseShape
@@ -187,7 +187,7 @@ def save(self, file: FilePath | IO[bytes]) -> None:
         self.build().save(file)
 
 
-class _BaseMaster(PptxConvertible[_PptxBaseMaster]):
+class _BaseMaster(_BaseSlide[_PptxBaseMaster]):
     @property
     def placeholders(self) -> "list[MasterPlaceholder]":
         """Get the placeholders."""

src/tppt/pptx/presentation.py

@@ -32,40 +32,49 @@
 from .shape.placeholder import SlidePlaceholder
 from .shape.text import Text, TextData, TextProps
 from .slide_layout import SlideLayout
-from .snotes_slide import NotesSlide
 from .table.table import DataFrame, Table, TableData, TableProps, dataframe2list
 
 if TYPE_CHECKING:
-    pass
+    from tppt.pptx.shape.background import Background
+
+    from .notes_slide import NotesSlide
 
 _GenericPptxBaseSlide = TypeVar("_GenericPptxBaseSlide", bound=_PptxBaseSlide)
 
 
-class _BaseSlide(PptxConvertible[_GenericPptxBaseSlide]): ...
+class _BaseSlide(PptxConvertible[_GenericPptxBaseSlide]):
+    @property
+    def background(self) -> "Background":
+        """Background of the slide."""
+        from tppt.pptx.shape.background import Background
+
+        return Background(self._pptx.background)
+
+    @property
+    def name(self) -> str:
+        return self._pptx.name
+
+    @name.setter
+    def name(self, value: str) -> None:
+        self._pptx.name = value
 
 
 class Slide(_BaseSlide[PptxSlide]):
     """Slide wrapper with type safety."""
 
     @property
-    def name(self) -> str | None:
-        """String representing the internal name of this slide.
-
-        Returns an empty string if no name is assigned.
-        """
-        if name := self._pptx.name:
-            return name
-        return None
+    def follow_master_background(self) -> bool:
+        return self._pptx.follow_master_background
 
     @property
-    def slide_id(self) -> int:
-        """Get the slide id."""
-        return self._pptx.slide_id
+    def notes_slide(self) -> "NotesSlide | None":
+        """Get the notes slide."""
+        if not self._pptx.has_notes_slide:
+            return None
 
-    @property
-    def shapes(self) -> list[BaseShape]:
-        """Get all shapes in the slide."""
-        return [BaseShape(shape) for shape in self._pptx.shapes]
+        from .notes_slide import NotesSlide
+
+        return NotesSlide(self._pptx.notes_slide)
 
     @property
     def placeholders(self) -> list[SlidePlaceholder]:
@@ -77,18 +86,21 @@ def placeholders(self) -> list[SlidePlaceholder]:
             for placeholder in self._pptx.placeholders
         ]
 
+    @property
+    def shapes(self) -> list[BaseShape]:
+        """Get all shapes in the slide."""
+        return [BaseShape(shape) for shape in self._pptx.shapes]
+
+    @property
+    def slide_id(self) -> int:
+        """Get the slide id."""
+        return self._pptx.slide_id
+
     @property
     def slide_layout(self) -> SlideLayout:
         """Get the slide layout."""
         return SlideLayout.from_pptx(self._pptx.slide_layout)
 
-    @property
-    def notes_slide(self) -> NotesSlide | None:
-        """Get the notes slide."""
-        if not self._pptx.has_notes_slide:
-            return None
-        return NotesSlide.from_pptx(self._pptx.notes_slide)
-
 
 class SlideBuilder:
     """Slide builder."""