Setting a keymap doesn't update the Footer or help panel

[davep]

Given this code:

from textual.app import App, ComposeResult
from textual.binding import Binding
from textual.widgets import Footer


class RemapIssueApp(App[None]):

    BINDINGS = [
        Binding("f2", "change", description="Change my own key", id="change"),
    ]

    def compose(self) -> ComposeResult:
        yield Footer()

    def on_mount(self) -> None:
        self.app.action_show_help_panel()

    def action_change(self) -> None:
        self.set_keymap({"change": "f9"})

if __name__ == "__main__":
    RemapIssueApp().run()

Pressing f2 remaps the binding to f9 but the Footer and help panel both show the old mapping. If you pull up the command palette, or tab away from and back to the terminal window (actions that would cause a refresh) then they both catch up.

[Forgen]

Widgets cannot get automatic notification that the parent bindings change I believe. Try calling a manual refresh in action_change?

[davep]

@Forgen The fact that both the footer and the key panel subscribe to bindings_updated_signal suggests that the intent is that that should not be necessary.

[Forgen]

I thought in the Textual signal architecture that signals only bubble upward?

…

On Tue, Apr 1, 2025, 11:04 AM Dave Pearson ***@***.***> wrote: @Forgen <https://github.com/Forgen> The fact that both the footer and the key panel subscribe to bindings_updated_signal <https://textual.textualize.io/api/screen/#textual.screen.Screen.bindings_updated_signal> suggests that the intent is that that should not be necessary. — Reply to this email directly, view it on GitHub <#5695 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFSM646LA6XUU76ULH6HGD2XK2IRAVCNFSM6AAAAAB2GYPVNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONRZHA3DQOBTGU> . You are receiving this because you were mentioned.Message ID: ***@***.***> [image: davep]*davep* left a comment (Textualize/textual#5695) <#5695 (comment)> @Forgen <https://github.com/Forgen> The fact that both the footer and the key panel subscribe to bindings_updated_signal <https://textual.textualize.io/api/screen/#textual.screen.Screen.bindings_updated_signal> suggests that the intent is that that should not be necessary. — Reply to this email directly, view it on GitHub <#5695 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFSM646LA6XUU76ULH6HGD2XK2IRAVCNFSM6AAAAAB2GYPVNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONRZHA3DQOBTGU> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[davep]

@Forgen I suspect you're confusing signals and messages. Messages bubble, signals are pub/sub.

[Forgen]

I was certainly confusing the two, thank you!

…

On Tue, Apr 1, 2025, 4:58 PM Dave Pearson ***@***.***> wrote: @Forgen <https://github.com/Forgen> I suspect you're confusing signals and messages. Messages bubble <https://textual.textualize.io/guide/events/#bubbling>, signals are pub/sub <https://textual.textualize.io/api/signal/>. — Reply to this email directly, view it on GitHub <#5695 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFSM67HMMQ3Q7WOMW66ELD2XMDYFAVCNFSM6AAAAAB2GYPVNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZQG43TQNBTG4> . You are receiving this because you were mentioned.Message ID: ***@***.***> [image: davep]*davep* left a comment (Textualize/textual#5695) <#5695 (comment)> @Forgen <https://github.com/Forgen> I suspect you're confusing signals and messages. Messages bubble <https://textual.textualize.io/guide/events/#bubbling>, signals are pub/sub <https://textual.textualize.io/api/signal/>. — Reply to this email directly, view it on GitHub <#5695 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFSM67HMMQ3Q7WOMW66ELD2XMDYFAVCNFSM6AAAAAB2GYPVNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDONZQG43TQNBTG4> . You are receiving this because you were mentioned.Message ID: ***@***.***>