Skip to content

Commit

Permalink
Removing privacy from root Settings menu (#456)
Browse files Browse the repository at this point in the history
  • Loading branch information
eoji committed Feb 1, 2019
1 parent 397e999 commit 2868e60
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 23 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,18 +56,18 @@ class SettingsActivity : BaseActivity<SettingsViewModel.ViewModel>() {
this.viewModel.outputs.showConfirmLogoutPrompt()
.compose(bindToLifecycle())
.observeOn(AndroidSchedulers.mainThread())
.subscribe({ show ->
.subscribe { show ->
if (show) {
lazyLogoutConfirmationDialog().show()
} else {
lazyLogoutConfirmationDialog().dismiss()
}
})
}

this.viewModel.outputs.userNameTextViewText()
.compose(bindToLifecycle())
.compose(Transformers.observeForUI())
.subscribe({ name_text_view.text = it })
.subscribe { name_text_view.text = it }

account_row.setOnClickListener {
startActivityWithSlideUpTransition(Intent(this, AccountActivity::class.java))
Expand All @@ -93,10 +93,6 @@ class SettingsActivity : BaseActivity<SettingsViewModel.ViewModel>() {
startActivityWithSlideUpTransition(Intent(this, NotificationsActivity::class.java))
}

privacy_row.setOnClickListener {
startActivityWithSlideUpTransition(Intent(this, PrivacyActivity::class.java))
}

rate_us_row.setOnClickListener { ViewUtils.openStoreRating(this, this.packageName) }
}

Expand Down
22 changes: 6 additions & 16 deletions app/src/main/res/layout/settings_layout.xml
Original file line number Diff line number Diff line change
Expand Up @@ -107,21 +107,10 @@
android:layout_marginBottom="@dimen/grid_3"
android:layout_marginTop="@dimen/grid_3">

<LinearLayout
android:layout_width="match_parent"
android:layout_height="wrap_content"
android:orientation="vertical">

<TextView
android:id="@+id/help_row"
style="@style/SettingsSingleRow"
android:text="@string/Help" />

<TextView
android:id="@+id/privacy_row"
style="@style/SettingsSingleRow"
android:text="@string/Privacy" />
</LinearLayout>
<TextView
android:id="@+id/help_row"
style="@style/SettingsSingleRow"
android:text="@string/Help" />

</LinearLayout>

Expand Down Expand Up @@ -156,7 +145,8 @@
<TextView
android:id="@+id/version_name_text_view"
android:layout_width="match_parent"
android:layout_height="wrap_content" />
android:layout_height="wrap_content"
tools:text="@string/App_version" />
</LinearLayout>

</LinearLayout>
Expand Down

0 comments on commit 2868e60

Please sign in to comment.