Skip to content

Commit 950b121

Browse files
Merge pull request #239 from adamtheturtle/video-simple
Simplify handling of video nodes
2 parents 7e56237 + ca74bae commit 950b121

File tree

1 file changed

+8
-13
lines changed

1 file changed

+8
-13
lines changed

src/sphinx_notion/__init__.py

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1354,6 +1354,13 @@ def depart_PdfNode(node: nodes.Element) -> None: # pylint: disable=invalid-name
13541354
"""
13551355
del node
13561356

1357+
@staticmethod
1358+
def depart_video_node(node: nodes.Element) -> None:
1359+
"""
1360+
Depart from video nodes.
1361+
"""
1362+
del node
1363+
13571364
@beartype
13581365
def _serialize_blocks(
13591366
self,
@@ -1411,18 +1418,6 @@ def _visit_video_node_notion(
14111418
translator.visit_video(node=node)
14121419

14131420

1414-
@beartype
1415-
def _depart_video_node_notion(
1416-
translator: NotionTranslator,
1417-
node: video_node,
1418-
) -> None:
1419-
"""
1420-
Depart from a video node (no action needed).
1421-
"""
1422-
del translator
1423-
del node
1424-
1425-
14261421
@beartype
14271422
def _notion_register_pdf_include_directive(
14281423
app: Sphinx,
@@ -1451,7 +1446,7 @@ def setup(app: Sphinx) -> ExtensionMetadata:
14511446

14521447
app.add_node(
14531448
node=video_node,
1454-
notion=(_visit_video_node_notion, _depart_video_node_notion),
1449+
notion=(_visit_video_node_notion, None),
14551450
override=True,
14561451
)
14571452
app.connect(

0 commit comments

Comments
 (0)