Skip to content

Commit 96d35da

Browse files
Remove 'New' Badge from Extensions page (#19637)
It's no longer "new".
1 parent 297703d commit 96d35da

File tree

4 files changed

+0
-19
lines changed

4 files changed

+0
-19
lines changed

src/cascadia/TerminalSettingsEditor/Extensions.cpp

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
3737
auto vmImpl = get_self<ExtensionsViewModel>(_ViewModel);
3838
vmImpl->ExtensionPackageIdentifierTemplateSelector(_extensionPackageIdentifierTemplateSelector);
3939
vmImpl->LazyLoadExtensions();
40-
vmImpl->MarkAsVisited();
4140

4241
if (vmImpl->IsExtensionView())
4342
{
@@ -363,11 +362,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
363362
return _ExtensionPackageIdentifierTemplateSelector.SelectTemplate(CurrentExtensionPackage());
364363
}
365364

366-
bool ExtensionsViewModel::DisplayBadge() const noexcept
367-
{
368-
return !Model::ApplicationState::SharedInstance().BadgeDismissed(L"page.extensions");
369-
}
370-
371365
// Returns true if the extension is enabled, false otherwise
372366
bool ExtensionsViewModel::GetExtensionState(hstring extensionSource, const Model::CascadiaSettings& settings)
373367
{
@@ -436,12 +430,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
436430
NavigateToColorSchemeRequested.raise(*this, nullptr);
437431
}
438432

439-
void ExtensionsViewModel::MarkAsVisited()
440-
{
441-
Model::ApplicationState::SharedInstance().DismissBadge(L"page.extensions");
442-
_NotifyChanges(L"DisplayBadge");
443-
}
444-
445433
bool ExtensionPackageViewModel::SortAscending(const Editor::ExtensionPackageViewModel& lhs, const Editor::ExtensionPackageViewModel& rhs)
446434
{
447435
auto getKey = [&](const Editor::ExtensionPackageViewModel& pkgVM) {

src/cascadia/TerminalSettingsEditor/Extensions.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
4646
bool NoProfilesModified() const noexcept { return _profilesModifiedView.Size() == 0; }
4747
bool NoProfilesAdded() const noexcept { return _profilesAddedView.Size() == 0; }
4848
bool NoSchemesAdded() const noexcept { return _colorSchemesAddedView.Size() == 0; }
49-
bool DisplayBadge() const noexcept;
5049

5150
// Views
5251
Windows::Foundation::Collections::IObservableVector<Editor::ExtensionPackageViewModel> ExtensionPackages() const noexcept { return _extensionPackages; }
@@ -59,7 +58,6 @@ namespace winrt::Microsoft::Terminal::Settings::Editor::implementation
5958
void UpdateSettings(const Model::CascadiaSettings& settings, const Editor::ColorSchemesPageViewModel& colorSchemesPageVM);
6059
void NavigateToProfile(const guid profileGuid);
6160
void NavigateToColorScheme(const Editor::ColorSchemeViewModel& schemeVM);
62-
void MarkAsVisited();
6361

6462
static bool GetExtensionState(hstring extensionSource, const Model::CascadiaSettings& settings);
6563
static void SetExtensionState(hstring extensionSource, const Model::CascadiaSettings& settings, bool enableExt);

src/cascadia/TerminalSettingsEditor/Extensions.idl

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@ namespace Microsoft.Terminal.Settings.Editor
2323
Boolean NoProfilesModified { get; };
2424
Boolean NoProfilesAdded { get; };
2525
Boolean NoSchemesAdded { get; };
26-
Boolean DisplayBadge { get; };
2726

2827
// Views
2928
IVector<ExtensionPackageViewModel> ExtensionPackages { get; };

src/cascadia/TerminalSettingsEditor/MainPage.xaml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -167,10 +167,6 @@
167167
<muxc:NavigationViewItem.Icon>
168168
<FontIcon Glyph="&#xEA86;" />
169169
</muxc:NavigationViewItem.Icon>
170-
<muxc:NavigationViewItem.InfoBadge>
171-
<muxc:InfoBadge Style="{StaticResource NewInfoBadge}"
172-
Visibility="{x:Bind ExtensionsVM.DisplayBadge, Mode=OneWay}" />
173-
</muxc:NavigationViewItem.InfoBadge>
174170
</muxc:NavigationViewItem>
175171

176172
<muxc:NavigationViewItemHeader x:Uid="Nav_Profiles" />

0 commit comments

Comments
 (0)