Skip to content
This repository was archived by the owner on Feb 5, 2025. It is now read-only.

Commit 2c933ff

Browse files
authored
Merge pull request #53 from wordpress-mobile/fix/unique-accessibility-identifiers
Update accessibility identifiers to unique values
2 parents f6dab0d + 90a025c commit 2c933ff

File tree

5 files changed

+7
-7
lines changed

5 files changed

+7
-7
lines changed

WordPressAuthenticator/Signin/LoginEmailViewController.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,14 +110,14 @@ open class LoginEmailViewController: LoginViewController, NUXKeyboardResponder {
110110
instructionLabel?.text = WordPressAuthenticator.shared.displayStrings.emailLoginInstructions
111111
}
112112
emailTextField.placeholder = NSLocalizedString("Email address", comment: "Placeholder for a textfield. The user may enter their email address.")
113-
emailTextField.accessibilityIdentifier = "Email address"
113+
emailTextField.accessibilityIdentifier = "Login Email Address"
114114

115115
alternativeLoginLabel?.text = NSLocalizedString("Alternatively:", comment: "String displayed before offering alternative login methods")
116116

117117
let submitButtonTitle = NSLocalizedString("Next", comment: "Title of a button. The text should be capitalized.").localizedCapitalized
118118
submitButton?.setTitle(submitButtonTitle, for: .normal)
119119
submitButton?.setTitle(submitButtonTitle, for: .highlighted)
120-
submitButton?.accessibilityIdentifier = "Next Button"
120+
submitButton?.accessibilityIdentifier = "Login Email Next Button"
121121
}
122122

123123

WordPressAuthenticator/Signin/LoginPrologueViewController.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ class LoginPrologueViewController: LoginViewController {
6363
let loginTitle = NSLocalizedString("Log In", comment: "Button title. Tapping takes the user to the login form.")
6464
let createTitle = NSLocalizedString("Sign up for WordPress.com", comment: "Button title. Tapping begins the process of creating a WordPress.com account.")
6565

66-
buttonViewController.setupTopButton(title: loginTitle, isPrimary: true, accessibilityIdentifier: "Log In Button") { [weak self] in
66+
buttonViewController.setupTopButton(title: loginTitle, isPrimary: true, accessibilityIdentifier: "Prologue Log In Button") { [weak self] in
6767
self?.loginTapped()
6868
}
6969
buttonViewController.setupBottomButton(title: createTitle, isPrimary: false) { [weak self] in

WordPressAuthenticator/Signin/LoginSiteAddressViewController.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ class LoginSiteAddressViewController: LoginViewController, NUXKeyboardResponder
7474
let submitButtonTitle = NSLocalizedString("Next", comment: "Title of a button. The text should be capitalized.").localizedCapitalized
7575
submitButton?.setTitle(submitButtonTitle, for: .normal)
7676
submitButton?.setTitle(submitButtonTitle, for: .highlighted)
77-
submitButton?.accessibilityIdentifier = "Next Button"
77+
submitButton?.accessibilityIdentifier = "Site Address Next Button"
7878

7979
let siteAddressHelpTitle = NSLocalizedString("Need help finding your site address?", comment: "A button title.")
8080
siteAddressHelpButton.setTitle(siteAddressHelpTitle, for: .normal)

WordPressAuthenticator/Signin/LoginWPComViewController.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ class LoginWPComViewController: LoginViewController, NUXKeyboardResponder {
131131
let submitButtonTitle = NSLocalizedString("Next", comment: "Title of a button. The text should be capitalized.").localizedCapitalized
132132
submitButton?.setTitle(submitButtonTitle, for: .normal)
133133
submitButton?.setTitle(submitButtonTitle, for: .highlighted)
134-
submitButton?.accessibilityIdentifier = "Log In Button"
134+
submitButton?.accessibilityIdentifier = "Password Next Button"
135135

136136
let forgotPasswordTitle = NSLocalizedString("Lost your password?", comment: "Title of a button. ")
137137
forgotPasswordButton?.setTitle(forgotPasswordTitle, for: .normal)

WordPressAuthenticator/Signup/SignupEmailViewController.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -74,13 +74,13 @@ class SignupEmailViewController: LoginViewController, NUXKeyboardResponder {
7474
instructionLabel?.text = NSLocalizedString("To create your new WordPress.com account, please enter your email address.", comment: "Text instructing the user to enter their email address.")
7575

7676
emailField.placeholder = NSLocalizedString("Email address", comment: "Placeholder for a textfield. The user may enter their email address.")
77-
emailField.accessibilityIdentifier = "Email address"
77+
emailField.accessibilityIdentifier = "Signup Email Address"
7878
emailField.contentInsets = WPStyleGuide.edgeInsetForLoginTextFields()
7979

8080
let submitButtonTitle = NSLocalizedString("Next", comment: "Title of a button. The text should be capitalized.").localizedCapitalized
8181
submitButton?.setTitle(submitButtonTitle, for: .normal)
8282
submitButton?.setTitle(submitButtonTitle, for: .highlighted)
83-
submitButton?.accessibilityIdentifier = "Next Button"
83+
submitButton?.accessibilityIdentifier = "Signup Email Next Button"
8484
}
8585

8686
/// Configure the view for an editing state. Should only be called from viewWillAppear

0 commit comments

Comments
 (0)