-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Accessibility fixes for Naming Styles #17973
Conversation
Tagging for review @dotnet/roslyn-ide |
<value>Add a naming rule.</value> | ||
</data> | ||
<data name="Remove_naming_rule" xml:space="preserve"> | ||
<value>Remove naming rule.</value> |
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.
are all the trailing periods intentional/desirable here?
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.
I completely don't care. It doesn't change how anything gets read.
@@ -24,6 +24,11 @@ internal partial class NamingStyleOptionPageControl : AbstractOptionPageControl | |||
public static string RequiredStyleHeader => ServicesVSResources.Required_Style; | |||
public static string SeverityHeader => ServicesVSResources.Severity; | |||
public static string ExplanatoryText => ServicesVSResources.For_a_given_symbol_only_the_topmost_rule_with_a_matching_Specification_will_be_applied_Violation_of_that_rules_Required_Style_will_be_reported_at_the_chosen_Severity_level; | |||
public static string AddRuleAutomationText => ServicesVSResources.Add_a_naming_rule; | |||
public static string RemoveAutomationText => ServicesVSResources.Remove_naming_rule; | |||
public static string SymbolSpecification => ServicesVSResources.Symbol_Specification; |
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.
are the last tree also AutomationText?
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.
I'll fix up the naming
47d0704
to
0b0cea2
Compare
Fixes https://devdiv.visualstudio.com/DevDiv/_workitems?_a=edit&id=391338&triage=true & https://devdiv.visualstudio.com/DevDiv/_workitems?_a=edit&id=391337&triage=true