From 96d35daeec187cd6a59561141d2cc002fedab09c Mon Sep 17 00:00:00 2001 From: Carlos Zamora Date: Wed, 10 Dec 2025 12:05:49 -0800 Subject: [PATCH] Remove 'New' Badge from Extensions page (#19637) It's no longer "new". --- src/cascadia/TerminalSettingsEditor/Extensions.cpp | 12 ------------ src/cascadia/TerminalSettingsEditor/Extensions.h | 2 -- src/cascadia/TerminalSettingsEditor/Extensions.idl | 1 - src/cascadia/TerminalSettingsEditor/MainPage.xaml | 4 ---- 4 files changed, 19 deletions(-) diff --git a/src/cascadia/TerminalSettingsEditor/Extensions.cpp b/src/cascadia/TerminalSettingsEditor/Extensions.cpp index 5db1ab4db6..9557eff7ad 100644 --- a/src/cascadia/TerminalSettingsEditor/Extensions.cpp +++ b/src/cascadia/TerminalSettingsEditor/Extensions.cpp @@ -37,7 +37,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation auto vmImpl = get_self(_ViewModel); vmImpl->ExtensionPackageIdentifierTemplateSelector(_extensionPackageIdentifierTemplateSelector); vmImpl->LazyLoadExtensions(); - vmImpl->MarkAsVisited(); if (vmImpl->IsExtensionView()) { @@ -363,11 +362,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation return _ExtensionPackageIdentifierTemplateSelector.SelectTemplate(CurrentExtensionPackage()); } - bool ExtensionsViewModel::DisplayBadge() const noexcept - { - return !Model::ApplicationState::SharedInstance().BadgeDismissed(L"page.extensions"); - } - // Returns true if the extension is enabled, false otherwise bool ExtensionsViewModel::GetExtensionState(hstring extensionSource, const Model::CascadiaSettings& settings) { @@ -436,12 +430,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation NavigateToColorSchemeRequested.raise(*this, nullptr); } - void ExtensionsViewModel::MarkAsVisited() - { - Model::ApplicationState::SharedInstance().DismissBadge(L"page.extensions"); - _NotifyChanges(L"DisplayBadge"); - } - bool ExtensionPackageViewModel::SortAscending(const Editor::ExtensionPackageViewModel& lhs, const Editor::ExtensionPackageViewModel& rhs) { auto getKey = [&](const Editor::ExtensionPackageViewModel& pkgVM) { diff --git a/src/cascadia/TerminalSettingsEditor/Extensions.h b/src/cascadia/TerminalSettingsEditor/Extensions.h index a9379829d2..cbe3d6c885 100644 --- a/src/cascadia/TerminalSettingsEditor/Extensions.h +++ b/src/cascadia/TerminalSettingsEditor/Extensions.h @@ -46,7 +46,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation bool NoProfilesModified() const noexcept { return _profilesModifiedView.Size() == 0; } bool NoProfilesAdded() const noexcept { return _profilesAddedView.Size() == 0; } bool NoSchemesAdded() const noexcept { return _colorSchemesAddedView.Size() == 0; } - bool DisplayBadge() const noexcept; // Views Windows::Foundation::Collections::IObservableVector ExtensionPackages() const noexcept { return _extensionPackages; } @@ -59,7 +58,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation void UpdateSettings(const Model::CascadiaSettings& settings, const Editor::ColorSchemesPageViewModel& colorSchemesPageVM); void NavigateToProfile(const guid profileGuid); void NavigateToColorScheme(const Editor::ColorSchemeViewModel& schemeVM); - void MarkAsVisited(); static bool GetExtensionState(hstring extensionSource, const Model::CascadiaSettings& settings); static void SetExtensionState(hstring extensionSource, const Model::CascadiaSettings& settings, bool enableExt); diff --git a/src/cascadia/TerminalSettingsEditor/Extensions.idl b/src/cascadia/TerminalSettingsEditor/Extensions.idl index 6093471014..349d0fe225 100644 --- a/src/cascadia/TerminalSettingsEditor/Extensions.idl +++ b/src/cascadia/TerminalSettingsEditor/Extensions.idl @@ -23,7 +23,6 @@ namespace Microsoft.Terminal.Settings.Editor Boolean NoProfilesModified { get; }; Boolean NoProfilesAdded { get; }; Boolean NoSchemesAdded { get; }; - Boolean DisplayBadge { get; }; // Views IVector ExtensionPackages { get; }; diff --git a/src/cascadia/TerminalSettingsEditor/MainPage.xaml b/src/cascadia/TerminalSettingsEditor/MainPage.xaml index 3601358ef3..bff716aff7 100644 --- a/src/cascadia/TerminalSettingsEditor/MainPage.xaml +++ b/src/cascadia/TerminalSettingsEditor/MainPage.xaml @@ -167,10 +167,6 @@ - - -