Skip to content
This repository was archived by the owner on Jan 16, 2021. It is now read-only.

Commit 7a20f40

Browse files
committed
Accessibility identifiers: fixing typo, updating style to match code guidelines, improving uniqueness of accessibility identifiers
1 parent 17b58af commit 7a20f40

File tree

4 files changed

+36
-32
lines changed

4 files changed

+36
-32
lines changed

ParseUI/Classes/LogInViewController/PFLogInView.h

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -62,14 +62,14 @@ typedef NS_OPTIONS(NSInteger, PFLogInFields) {
6262
6363
@see PFLogInView
6464
*/
65-
extern NSString * const PFLogInViewUsernameFieldAccessibilityIdentifier;
66-
extern NSString * const PFLogInViewPasswordFieldAccessibilityIdentifier;
67-
extern NSString * const PFLogInViewLogInButtonAccessibilityIdentifier;
68-
extern NSString * const PFLogInViewSignUpButtonAccessibilityIdentifier;
69-
extern NSString * const PFLogInViewPasswordForgottenButotnAccessibilityIdentifier;
70-
extern NSString * const PFLogInViewTwitterButtonAccessibilityIdentifier;
71-
extern NSString * const PFLogInViewFacebookButtonAccessibilityIdentifier;
72-
extern NSString * const PFLogInViewDismissButtonAccessibilityIdentifier;
65+
extern NSString *const PFLogInViewUsernameFieldAccessibilityIdentifier;
66+
extern NSString *const PFLogInViewPasswordFieldAccessibilityIdentifier;
67+
extern NSString *const PFLogInViewLogInButtonAccessibilityIdentifier;
68+
extern NSString *const PFLogInViewSignUpButtonAccessibilityIdentifier;
69+
extern NSString *const PFLogInViewPasswordForgottenButtonAccessibilityIdentifier;
70+
extern NSString *const PFLogInViewTwitterButtonAccessibilityIdentifier;
71+
extern NSString *const PFLogInViewFacebookButtonAccessibilityIdentifier;
72+
extern NSString *const PFLogInViewDismissButtonAccessibilityIdentifier;
7373

7474
@class PFTextField;
7575

ParseUI/Classes/LogInViewController/PFLogInView.m

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,17 @@
3535
static NSString *const PFLogInViewDefaultFacebookButtonImageName = @"facebook_icon.png";
3636
static NSString *const PFLogInViewDefaultTwitterButtonImageName = @"twitter_icon.png";
3737

38-
#pragma mark - Accessibility labels identifiers
39-
NSString * const PFLogInViewUsernameFieldAccessibilityIdentifier = @"username";
40-
NSString * const PFLogInViewPasswordFieldAccessibilityIdentifier = @"password";
41-
NSString * const PFLogInViewLogInButtonAccessibilityIdentifier = @"logIn";
42-
NSString * const PFLogInViewSignUpButtonAccessibilityIdentifier = @"signUp";
43-
NSString * const PFLogInViewPasswordForgottenButotnAccessibilityIdentifier = @"passwordForgotten";
44-
NSString * const PFLogInViewTwitterButtonAccessibilityIdentifier = @"twitter";
45-
NSString * const PFLogInViewFacebookButtonAccessibilityIdentifier = @"facebook";
46-
NSString * const PFLogInViewDismissButtonAccessibilityIdentifier = @"dismiss";
38+
///--------------------------------------
39+
#pragma mark - Accessibility Identifiers
40+
///--------------------------------------
41+
NSString *const PFLogInViewUsernameFieldAccessibilityIdentifier = @"PFLogInViewUsernameFieldAccessibilityIdentifier";
42+
NSString *const PFLogInViewPasswordFieldAccessibilityIdentifier = @"PFLogInViewPasswordFieldAccessibilityIdentifier";
43+
NSString *const PFLogInViewLogInButtonAccessibilityIdentifier = @"PFLogInViewLogInButtonAccessibilityIdentifier";
44+
NSString *const PFLogInViewSignUpButtonAccessibilityIdentifier = @"PFLogInViewSignUpButtonAccessibilityIdentifier";
45+
NSString *const PFLogInViewPasswordForgottenButtonAccessibilityIdentifier = @"PFLogInViewPasswordForgottenButtonAccessibilityIdentifier";
46+
NSString *const PFLogInViewTwitterButtonAccessibilityIdentifier = @"PFLogInViewTwitterButtonAccessibilityIdentifier";
47+
NSString *const PFLogInViewFacebookButtonAccessibilityIdentifier = @"PFLogInViewFacebookButtonAccessibilityIdentifier";
48+
NSString *const PFLogInViewDismissButtonAccessibilityIdentifier = @"PFLogInViewDismissButtonAccessibilityIdentifier";
4749

4850
@implementation PFLogInView
4951

@@ -174,7 +176,7 @@ - (void)_updateAllFields {
174176
if (_fields & PFLogInFieldsPasswordForgotten) {
175177
if (!_passwordForgottenButton) {
176178
_passwordForgottenButton = [[PFTextButton alloc] initWithFrame:CGRectZero];
177-
_passwordForgottenButton.accessibilityIdentifier = PFLogInViewPasswordForgottenButotnAccessibilityIdentifier;
179+
_passwordForgottenButton.accessibilityIdentifier = PFLogInViewPasswordForgottenButtonAccessibilityIdentifier;
178180
[_passwordForgottenButton setTitle:PFLocalizedString(@"Forgot Password?", "Forgot Password?")
179181
forState:UIControlStateNormal];
180182
[self addSubview:_passwordForgottenButton];

ParseUI/Classes/SignUpViewController/PFSignUpView.h

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -51,16 +51,16 @@ typedef NS_OPTIONS(NSInteger, PFSignUpFields) {
5151

5252

5353
/**
54-
`PFSignUpFields`'s accessibilities labels
54+
`PFSignUpFields`'s accessibity identifiers
5555
5656
@see PFSignUpView
5757
*/
58-
extern NSString * const PFSignUpViewUsernameFieldAccessibilityIdentifier;
59-
extern NSString * const PFSignUpViewEmailFieldAccessibilityIdentifier;
60-
extern NSString * const PFSignUpViewPasswordFieldAccessibilityIdentifier;
61-
extern NSString * const PFSignUpViewAdditionalFieldAccessibilityIdentifier;
62-
extern NSString * const PFSignUpViewSignUpButtonAccessibilityIdentifier;
63-
extern NSString * const PFSignUpViewDismissButtonAccessibilityIdentifier;
58+
extern NSString *const PFSignUpViewUsernameFieldAccessibilityIdentifier;
59+
extern NSString *const PFSignUpViewEmailFieldAccessibilityIdentifier;
60+
extern NSString *const PFSignUpViewPasswordFieldAccessibilityIdentifier;
61+
extern NSString *const PFSignUpViewAdditionalFieldAccessibilityIdentifier;
62+
extern NSString *const PFSignUpViewSignUpButtonAccessibilityIdentifier;
63+
extern NSString *const PFSignUpViewDismissButtonAccessibilityIdentifier;
6464

6565

6666
@class PFTextField;

ParseUI/Classes/SignUpViewController/PFSignUpView.m

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -32,13 +32,15 @@
3232

3333
static NSString *const PFSignUpViewDefaultLogoImageName = @"parse_logo.png";
3434

35-
#pragma mark - Accessibility labels
36-
NSString * const PFSignUpViewUsernameFieldAccessibilityIdentifier = @"username";
37-
NSString * const PFSignUpViewEmailFieldAccessibilityIdentifier = @"email";
38-
NSString * const PFSignUpViewPasswordFieldAccessibilityIdentifier = @"password";
39-
NSString * const PFSignUpViewAdditionalFieldAccessibilityIdentifier = @"additional";
40-
NSString * const PFSignUpViewSignUpButtonAccessibilityIdentifier = @"sign up";
41-
NSString * const PFSignUpViewDismissButtonAccessibilityIdentifier = @"dismiss";
35+
///--------------------------------------
36+
#pragma mark - Accessibility Identifiers
37+
///--------------------------------------
38+
NSString *const PFSignUpViewUsernameFieldAccessibilityIdentifier = @"PFSignUpViewUsernameFieldAccessibilityIdentifier";
39+
NSString *const PFSignUpViewEmailFieldAccessibilityIdentifier = @"PFSignUpViewEmailFieldAccessibilityIdentifier";
40+
NSString *const PFSignUpViewPasswordFieldAccessibilityIdentifier = @"PFSignUpViewPasswordFieldAccessibilityIdentifier";
41+
NSString *const PFSignUpViewAdditionalFieldAccessibilityIdentifier = @"PFSignUpViewAdditionalFieldAccessibilityIdentifier";
42+
NSString *const PFSignUpViewSignUpButtonAccessibilityIdentifier = @"PFSignUpViewSignUpButtonAccessibilityIdentifier";
43+
NSString *const PFSignUpViewDismissButtonAccessibilityIdentifier = @"PFSignUpViewDismissButtonAccessibilityIdentifier";
4244

4345
@implementation PFSignUpView
4446

0 commit comments

Comments
 (0)