snapshot

src/textual/_compositor.py

@@ -486,9 +486,9 @@ def full_map(self) -> CompositorMap:
             return {}
         if self._full_map_invalidated:
             self._full_map_invalidated = False
-            old = self._full_map.keys()
+            old_widgets = self._full_map.keys()
             map, widgets = self._arrange_root(self.root, self.size, visible_only=False)
-            self._new_widgets.update(map.keys() - old)
+            self._new_widgets.update(map.keys() - old_widgets)
             self._full_map = map
             self._visible_widgets = None
             self._visible_map = None

tests/snapshot_tests/test_snapshots.py

@@ -9,7 +9,7 @@
 from textual import events
 from textual.app import App, ComposeResult
 from textual.binding import Binding
-from textual.containers import Vertical
+from textual.containers import Vertical, VerticalScroll
 from textual.pilot import Pilot
 from textual.screen import Screen
 from textual.widgets import (
@@ -24,8 +24,7 @@
     OptionList,
     SelectionList,
 )
-from textual.widgets import Switch
-from textual.widgets import Label
+from textual.widgets import ProgressBar, Label, Switch
 from textual.widgets.text_area import BUILTIN_LANGUAGES, Selection, TextAreaTheme
 
 # These paths should be relative to THIS directory.
@@ -1992,3 +1991,31 @@ def on_mount(self) -> None:
 
     app = DisabledApp()
     assert snap_compare(app)
+
+
+def test_missing_new_widgets(snap_compare):
+    """Regression test for https://github.com/Textualize/textual/issues/5024"""
+
+    class MRE(App):
+        BINDINGS = [("z", "toggle_console", "Console")]
+        CSS = """
+        RichLog { border-top: dashed blue; height: 6; }
+        .hidden { display: none; }
+        """
+
+        def compose(self):
+            yield VerticalScroll()
+            yield ProgressBar(
+                classes="hidden"
+            )  # removing or displaying this widget prevents the bug
+            yield Footer()  # clicking "Console" in the footer prevents the bug
+            yield RichLog(classes="hidden")
+
+        def on_ready(self) -> None:
+            self.query_one(RichLog).write("\n".join(f"line #{i}" for i in range(5)))
+
+        def action_toggle_console(self) -> None:
+            self.query_one(RichLog).toggle_class("hidden")
+
+    app = MRE()
+    assert snap_compare(app, press=["space", "space", "z"])