-
Notifications
You must be signed in to change notification settings - Fork 645
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New default package img, textare padding, reserved icon #10029
New default package img, textare padding, reserved icon #10029
Conversation
@@ -974,7 +973,7 @@ | |||
else | |||
{ | |||
<td tabindex="0" class="package-icon-cell package-warning-icon" aria-label="@packageVersion.PackageWarningIconTitle"> | |||
<i class="ms-Icon ms-Icon--Warning package-icon" title="@packageVersion.PackageWarningIconTitle"></i> | |||
<i tabindex="0" class="ms-Icon ms-Icon--Warning package-icon" title="@packageVersion.PackageWarningIconTitle"></i> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added tab stop to warning icon in package versions table, this was an accesibility bug
<span class="tooltip-wrapper tooltip-with-icon popover right"> | ||
<span class="arrow"></span> | ||
<span class="popover-content"> | ||
@NuGetGallery.Strings.FrameworkFilters_Tooltip | ||
<a href="@(Model.FrameworksFilteringInformationLink)">Click here to learn more.</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved "Click here to learn more" here so it can be a link, this caused a nested anchor, so I changed it to spans
* New default package img, textare padding, reserved icon * [Dark Theme] Accessibility and UI fixes * Sort by select * default package icon
Addresses #9964
Prefix reserved and default icon
Sort by select size fixed and default icon on dark mode
Padding is the same on text area and select