Skip to content

Commit af3b5b0

Browse files
committed
More precise names.
1 parent 06c71d8 commit af3b5b0

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

app/src/main/java/org/wikipedia/createaccount/CreateAccountActivity.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,7 @@ class CreateAccountActivity : BaseActivity() {
213213
binding.viewCreateAccountError.visibility = View.GONE
214214
}
215215
binding.createAccountSubmitButton.setOnClickListener {
216-
instrument?.submitInteraction("click", elementId = "create_account")
216+
instrument?.submitInteraction("click", elementId = "create_account_button")
217217
if (requestSource == LoginActivity.SOURCE_YEAR_IN_REVIEW) {
218218
YearInReviewEvent.submit(action = "create_account_click", slide = "explore_prompt")
219219
}

app/src/main/java/org/wikipedia/login/LoginActivity.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ class LoginActivity : BaseActivity() {
155155

156156
private fun setAllViewsClickListener() {
157157
binding.loginButton.setOnClickListener {
158-
instrument?.submitInteraction("click", elementId = "login")
158+
instrument?.submitInteraction("click", elementId = "login_button")
159159
validateThenLogin()
160160
}
161161
binding.loginCreateAccountButton.setOnClickListener {

app/src/main/java/org/wikipedia/settings/LogoutPreference.kt

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -49,17 +49,17 @@ class LogoutPreference : Preference {
4949
setOnClickListener {
5050
val instrument = TestKitchenAdapter.client.getInstrument("apps-authentication")
5151
.startFunnel("logout_account")
52-
instrument.submitInteraction("click", actionSource = "settings", elementId = "logout")
52+
instrument.submitInteraction("click", actionSource = "settings", elementId = "logout_button")
5353

5454
activity?.let {
5555
instrument.submitInteraction("impression", actionSource = "logout_warning")
5656
MaterialAlertDialogBuilder(it)
5757
.setMessage(if (AccountUtil.isTemporaryAccount) R.string.temp_account_end_session_confirm else R.string.logout_prompt)
5858
.setNegativeButton(R.string.logout_dialog_cancel_button_text) { dialog, which ->
59-
instrument.submitInteraction("click", actionSource = "logout_warning", elementId = "cancel")
59+
instrument.submitInteraction("click", actionSource = "logout_warning", elementId = "cancel_button")
6060
}
6161
.setPositiveButton(if (AccountUtil.isTemporaryAccount) R.string.temp_account_end_session else R.string.preference_title_logout) { _, _ ->
62-
instrument.submitInteraction("click", actionSource = "logout_warning", elementId = "logout_confirm")
62+
instrument.submitInteraction("click", actionSource = "logout_warning", elementId = "confirm_button")
6363
WikipediaApp.instance.logOut()
6464
Prefs.readingListsLastSyncTime = null
6565
Prefs.isReadingListSyncEnabled = false
@@ -73,7 +73,7 @@ class LogoutPreference : Preference {
7373
holder.itemView.findViewById<View>(R.id.accountVanishButton).setOnClickListener {
7474
val instrument = TestKitchenAdapter.client.getInstrument("apps-authentication")
7575
.startFunnel("vanish_account")
76-
instrument.submitInteraction("click", actionSource = "settings", elementId = "vanish")
76+
instrument.submitInteraction("click", actionSource = "settings", elementId = "vanish_button")
7777

7878
activity?.let {
7979
instrument.submitInteraction("impression", actionSource = "vanish_warning")
@@ -82,10 +82,10 @@ class LogoutPreference : Preference {
8282
.setTitle(R.string.account_vanish_request_confirm_title)
8383
.setMessage(StringUtil.fromHtml(it.getString(R.string.account_vanish_request_confirm)))
8484
.setNegativeButton(android.R.string.cancel) { dialog, which ->
85-
instrument.submitInteraction("click", actionSource = "vanish_warning", elementId = "cancel")
85+
instrument.submitInteraction("click", actionSource = "vanish_warning", elementId = "cancel_button")
8686
}
8787
.setPositiveButton(R.string.account_vanish_request_title) { _, _ ->
88-
instrument.submitInteraction("click", actionSource = "vanish_warning", elementId = "vanish_confirm")
88+
instrument.submitInteraction("click", actionSource = "vanish_warning", elementId = "confirm_button")
8989
it.finish()
9090
it.startActivity(SingleWebViewActivity.newIntent(it, it.getString(R.string.account_vanish_url), isWebForm = true))
9191
}.show()

0 commit comments

Comments
 (0)