Skip to content

Commit db7dc6a

Browse files
authored
Merge pull request #8226 from NuGet/learn-more-upload
Accessibility fix: Make Learn more link more descriptive
2 parents c8f727a + 7dc2f7d commit db7dc6a

6 files changed

Lines changed: 52 additions & 4 deletions

src/NuGetGallery/Services/IconUrlDeprecationValidationMessage.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,6 @@ public class IconUrlDeprecationValidationMessage : IValidationMessage
99

1010
public bool HasRawHtmlRepresentation => true;
1111

12-
public string RawHtmlMessage => $"{Strings.UploadPackage_IconUrlDeprecated.Replace("<", "&lt;").Replace(">", "&gt;")} <a href=\"https://aka.ms/deprecateIconUrl\">{Strings.UploadPackage_LearnMore}</a>.";
12+
public string RawHtmlMessage => $"{Strings.UploadPackage_IconUrlDeprecated.Replace("<", "&lt;").Replace(">", "&gt;")} <a href=\"https://aka.ms/deprecateIconUrl\" aria-label= \"{Strings.UploadPackage_LearnMore_IconUrlDeprecation}\">{Strings.UploadPackage_LearnMore}</a>.";
1313
}
1414
}

src/NuGetGallery/Services/InvalidLicenseUrlValidationMessage.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ namespace NuGetGallery
1111
/// </summary>
1212
public class InvalidLicenseUrlValidationMessage : IValidationMessage
1313
{
14-
private string DetailsLink => $"<a href=\"https://aka.ms/invalidNuGetLicenseUrl\">{Strings.UploadPackage_LearnMore}</a>.";
14+
private string DetailsLink => $"<a href=\"https://aka.ms/invalidNuGetLicenseUrl\" aria-label= \"{Strings.UploadPackage_LearMore_PackagingLicense}\">{Strings.UploadPackage_LearnMore}</a>.";
1515
private string DetailsPlainText => "https://aka.ms/invalidNuGetLicenseUrl";
1616

1717
private readonly string _baseMessage;

src/NuGetGallery/Services/InvalidUrlEncodingForLicenseUrlValidationMessage.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ namespace NuGetGallery
99
/// </summary>
1010
public class InvalidUrlEncodingForLicenseUrlValidationMessage : IValidationMessage
1111
{
12-
private string DetailsLink => $"<a href=\"https://aka.ms/malformedNuGetLicenseUrl\">{Strings.UploadPackage_LearnMore}</a>.";
12+
private string DetailsLink => $"<a href=\"https://aka.ms/malformedNuGetLicenseUrl\" aria-label= \"{Strings.UploadPackage_LearnMore_InvalidLicenseUrlEncoding}\">{Strings.UploadPackage_LearnMore}</a>.";
1313
private string DetailsPlainText => "https://aka.ms/malformedNuGetLicenseUrl";
1414

1515
private string BaseMessage => Strings.UploadPackage_MalformedLicenseUrl;

src/NuGetGallery/Services/LicenseUrlDeprecationValidationMessage.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ namespace NuGetGallery
1212
/// </summary>
1313
public class LicenseUrlDeprecationValidationMessage : IValidationMessage
1414
{
15-
private string DeprecationLink => $"<a href=\"{GalleryConstants.LicenseDeprecationUrl}\">{Strings.UploadPackage_LearnMore}</a>.";
15+
private string DeprecationLink => $"<a href=\"{GalleryConstants.LicenseDeprecationUrl}\" aria-label= \"{Strings.UploadPackage_LearnMore_LicenseUrlDreprecation}\">{Strings.UploadPackage_LearnMore}</a>.";
1616

1717
private readonly string _baseMessage;
1818

src/NuGetGallery/Strings.Designer.cs

Lines changed: 36 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/NuGetGallery/Strings.resx

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1187,4 +1187,16 @@ The {1} Team</value>
11871187
<data name="ReadmeNotEditableWithEmbeddedReadme" xml:space="preserve">
11881188
<value>The readme is not editable with the package has the embedded readme</value>
11891189
</data>
1190+
<data name="UploadPackage_LearMore_PackagingLicense" xml:space="preserve">
1191+
<value>Learn more about including a license within the package</value>
1192+
</data>
1193+
<data name="UploadPackage_LearnMore_IconUrlDeprecation" xml:space="preserve">
1194+
<value>Learn more about icon URL deprecation</value>
1195+
</data>
1196+
<data name="UploadPackage_LearnMore_InvalidLicenseUrlEncoding" xml:space="preserve">
1197+
<value>Learn more about invalid license URL encoding</value>
1198+
</data>
1199+
<data name="UploadPackage_LearnMore_LicenseUrlDreprecation" xml:space="preserve">
1200+
<value>Learn more about license URL deprecation</value>
1201+
</data>
11901202
</root>

0 commit comments

Comments
 (0)