Materialize literal-blocks list before modifying nodes

Author: kwankyu

src/sage_docbuild/conf.py

@@ -954,7 +954,7 @@ class SagecodeTransform(SphinxTransform):
 
     def apply(self):
         if self.app.builder.tags.has('html') or self.app.builder.tags.has('inventory'):
-            for node in self.document.findall(nodes.literal_block):
+            for node in list(self.document.findall(nodes.literal_block)):
                 if node.get('language') is None and node.astext().startswith('sage:'):
                     from docutils.nodes import container as Container, label as Label, literal_block as LiteralBlock, Text
                     from sphinx_inline_tabs._impl import TabContainer
@@ -963,7 +963,8 @@ def apply(self):
                     prev_node = node.previous_sibling()
                     if isinstance(node.previous_sibling(), TabContainer):
                         # Make sure not to merge inline tabs for adjacent literal blocks
-                        parent.insert(index, Text(''))
+                        parent.insert(index, nodes.paragraph())
+                        prev_node = parent[index]
                         index += 1
                     parent.remove(node)
                     # Tab for Sage code
@@ -1102,3 +1103,4 @@ def feature_tags():
     for feature in all_features():
         if feature.is_present():
             yield 'feature_' + feature.name.replace('.', '_')
+