Skip to content

Commit abd7f32

Browse files
authored
Move search filter constants to GalleryConstants (#10633)
* Move search filter constants to GalleryConstants * Fix build errors * Fix build errors
1 parent 23d824a commit abd7f32

6 files changed

Lines changed: 20 additions & 17 deletions

File tree

src/NuGetGallery/GalleryConstants.cs

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
// Copyright (c) .NET Foundation. All rights reserved.
1+
// Copyright (c) .NET Foundation. All rights reserved.
22
// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
33

44
using NuGet.Versioning;
@@ -109,5 +109,18 @@ public static class SearchSortNames
109109
public const string TotalDownloadsAsc = "totalDownloads-asc";
110110
public const string TotalDownloadsDesc = "totalDownloads-desc";
111111
}
112+
113+
public static class SearchFilterIds
114+
{
115+
/// <summary>
116+
/// MUST MATCH common.js. Used to add localy stored search params to a element.
117+
/// </summary>
118+
public const string PackagesLinkClass = "link-to-add-local-search-filters";
119+
120+
/// <summary>
121+
/// MUST MATCH common.js. Used to add localy stored search params to form element.
122+
/// </summary>
123+
public const string SimpleSearchFormId = "form-to-add-local-search-filters";
124+
}
112125
}
113126
}

src/NuGetGallery/UrlHelperExtensions.cs

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19,16 +19,6 @@ public static class UrlHelperExtensions
1919
private const string Area = "area";
2020
private static IGalleryConfigurationService _configuration;
2121

22-
/// <summary>
23-
/// MUST MATCH common.js. Used to add localy stored search params to a element.
24-
/// </summary>
25-
public const string PackagesLinkClass = "link-to-add-local-search-filters";
26-
27-
/// <summary>
28-
/// MUST MATCH common.js. Used to add localy stored search params to form element.
29-
/// </summary>
30-
public const string SimpleSearchFormId = "form-to-add-local-search-filters";
31-
3222
public static class Fragments
3323
{
3424
public static class ManagePage

src/NuGetGallery/Views/Experiments/SearchSideBySide.cshtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
The side-by-side search page is now <b>closed</b>.
1818
<br /><br />
1919
Thank you for trying out the side-by-side preview. The new and improved NuGet search
20-
is now <a href="@Url.PackageList()" class="@UrlHelperExtensions.PackagesLinkClass">live</a>! For details about the initial release, check out the
20+
is now <a href="@Url.PackageList()" class="@GalleryConstants.SearchFilterIds.PackagesLinkClass">live</a>! For details about the initial release, check out the
2121
<a href="https://devblogs.microsoft.com/nuget/new-and-improved-nuget-search/">blog post</a>.
2222
The old search results are no longer available.
2323
<br /><br />

src/NuGetGallery/Views/Pages/Home.cshtml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ else if (AskUserToEnable2FA)
4848
</div>
4949
<div class="row">
5050
<div class="col-sm-8 col-sm-offset-2">
51-
<form id="@UrlHelperExtensions.SimpleSearchFormId" action="@Url.PackageList()" method="get">
51+
<form id="@GalleryConstants.SearchFilterIds.SimpleSearchFormId" action="@Url.PackageList()" method="get">
5252
@Html.Partial("_SearchBar")
5353
@Html.Partial("_AutocompleteTemplate")
5454
</form>
@@ -96,7 +96,7 @@ else if (AskUserToEnable2FA)
9696
</p>
9797
</div>
9898
<div class="col-sm-4">
99-
<a href="@Url.PackageList()" class="@UrlHelperExtensions.PackagesLinkClass"
99+
<a href="@Url.PackageList()" class="@GalleryConstants.SearchFilterIds.PackagesLinkClass"
100100
title="Explore packages available on NuGet.org">
101101
<div class="home-icons">
102102
<img src="@Url.Absolute("~/Content/gallery/img/search-icon.svg")" alt="Find packages" width="96" height="96" />

src/NuGetGallery/Views/Shared/Gallery/Header.cshtml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@
7474
</div>
7575
<div id="navbar" class="navbar-collapse collapse">
7676
<ul class="nav navbar-nav" role="tablist">
77-
@DisplayNavigationItem("Packages", Url.PackageList(), className: UrlHelperExtensions.PackagesLinkClass)
77+
@DisplayNavigationItem("Packages", Url.PackageList(), className: GalleryConstants.SearchFilterIds.PackagesLinkClass)
7878
@DisplayNavigationItem("Upload", Url.UploadPackage())
7979
@if (StatisticsHelper.IsStatisticsPageAvailable)
8080
{
@@ -176,7 +176,7 @@
176176
{
177177
<div id="search-bar-header" class="container search-container">
178178
<div class="row">
179-
<form aria-label="Package search bar" class="col-sm-12" id="@UrlHelperExtensions.SimpleSearchFormId" action="@Url.PackageList()" method="get">
179+
<form aria-label="Package search bar" class="col-sm-12" id="@GalleryConstants.SearchFilterIds.SimpleSearchFormId" action="@Url.PackageList()" method="get">
180180
@Html.Partial("_SearchBar")
181181
@Html.Partial("_AutocompleteTemplate")
182182
</form>

src/NuGetGallery/Views/Shared/SiteMenu.cshtml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
}
99
}
1010
<li class="@classes["Home"]"><a href="@Url.Home()">Home</a></li>
11-
<li class="@classes["Packages"]"><a href="@Url.PackageList()" class="@UrlHelperExtensions.PackagesLinkClass">Packages</a></li>
11+
<li class="@classes["Packages"]"><a href="@Url.PackageList()" class="@GalleryConstants.SearchFilterIds.PackagesLinkClass">Packages</a></li>
1212
<li class="@classes["Upload"]"><a href="@Url.UploadPackage()" class="upload">Upload Package</a></li>
1313
@if (StatisticsHelper.IsStatisticsPageAvailable)
1414
{

0 commit comments

Comments
 (0)