Untie _UpdateCommandsForPalette and _RefreshUIForSettingsReload #12697

Open
opened 2026-01-31 03:22:25 +00:00 by claunia · 0 comments
Owner

Originally created by @zadjii-msft on GitHub (Feb 23, 2021).

            // Make sure to _UpdateCommandsForPalette before
            // _RefreshUIForSettingsReload. _UpdateCommandsForPalette will make
            // sure the KeyChordText of Commands is updated, which needs to

Huh. Really. That's a weird dependency.

Please file a followup task to .. i dunno, make the settings model standalone or something? It seems like it still can't work without Page around :|

Originally posted by @DHowett in https://github.com/microsoft/terminal/pull/9253#discussion_r581332862

Originally created by @zadjii-msft on GitHub (Feb 23, 2021). ```c++ // Make sure to _UpdateCommandsForPalette before // _RefreshUIForSettingsReload. _UpdateCommandsForPalette will make // sure the KeyChordText of Commands is updated, which needs to ``` Huh. Really. That's a weird dependency. Please file a followup task to .. i dunno, make the settings model standalone or something? It seems like it still can't work without Page around :| _Originally posted by @DHowett in https://github.com/microsoft/terminal/pull/9253#discussion_r581332862_
claunia added the Issue-TaskProduct-TerminalArea-CodeHealthArea-SettingsUI labels 2026-01-31 03:22:26 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#12697