Skip to content
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.

Sign up for GitHub

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

Navigation | New Home tab to priorize all unread - BLOCKED #4010

Closed
wants to merge 17 commits into from
Closed

Navigation | New Home tab to priorize all unread - BLOCKED #4010

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
0d8a190
Navigation | New Home tab to priorize all unread
BillCarsonFr Sep 14, 2021
e21c25c
Add support for simple vs pro layout
BillCarsonFr Sep 16, 2021
a0c3541
Quick legacy community layout support
BillCarsonFr Sep 16, 2021
c6931d7
Show Fab in all in one mode
BillCarsonFr Sep 16, 2021
80cf904
Fix wrong highlight
BillCarsonFr Sep 17, 2021
bf41085
Fix low priority section in home and bad notification highlight
BillCarsonFr Sep 17, 2021
250c7df
Quick design update
BillCarsonFr Sep 22, 2021
6423459
Update layout mode names
BillCarsonFr Sep 23, 2021
37486b3
Add layout mode test
BillCarsonFr Oct 4, 2021
7498c22
cleanup
BillCarsonFr Oct 13, 2021
83229fa
Fix rebase
BillCarsonFr Oct 13, 2021
fe9b4ef
ktlint
BillCarsonFr Oct 13, 2021
8b61c28
Use old labNotif tab to decide on best default layout mode
BillCarsonFr Oct 13, 2021
defd423
Change visibility of legacy pref
BillCarsonFr Oct 13, 2021
f700330
FIx empty state + proper dark mode
BillCarsonFr Oct 13, 2021
3b90446
Clean up
bmarty Oct 13, 2021
70d0e5f
Code quality
bmarty Oct 13, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion vector/src/main/java/im/vector/app/features/settings/VectorPreferences.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,7 @@ class VectorPreferences @Inject constructor(private val context: Context) {

// SETTINGS_LABS_HIDE_TECHNICAL_E2E_ERRORS
private const val SETTINGS_LABS_SHOW_COMPLETE_HISTORY_IN_ENCRYPTED_ROOM = "SETTINGS_LABS_SHOW_COMPLETE_HISTORY_IN_ENCRYPTED_ROOM"
const val SETTINGS_LABS_UNREAD_NOTIFICATIONS_AS_TAB = "SETTINGS_LABS_UNREAD_NOTIFICATIONS_AS_TAB"

// analytics
const val SETTINGS_USE_ANALYTICS_KEY = "SETTINGS_USE_ANALYTICS_KEY"
Expand Down Expand Up @@ -335,6 +336,11 @@ class VectorPreferences @Inject constructor(private val context: Context) {
return developerMode() && defaultPrefs.getBoolean(SETTINGS_LABS_ALLOW_EXTENDED_LOGS, false)
}

// Legacy lab option kept for migration to layout mode settings
fun labAddNotificationTab(): Boolean {
return defaultPrefs.getBoolean(SETTINGS_LABS_UNREAD_NOTIFICATIONS_AS_TAB, false)
}

fun failFast(): Boolean {
return BuildConfig.DEBUG || (developerMode() && defaultPrefs.getBoolean(SETTINGS_DEVELOPER_MODE_FAIL_FAST_PREFERENCE_KEY, false))
}
Expand Down Expand Up @@ -1012,7 +1018,12 @@ class VectorPreferences @Inject constructor(private val context: Context) {
// Layout

fun getLayoutMode(): LayoutMode {
return defaultPrefs.getString(SETTINGS_LAYOUT_MODE, LayoutMode.PRODUCTIVITY.name).let {
val defaultLayoutMode = if (labAddNotificationTab()) {
LayoutMode.PRODUCTIVITY.name
} else {
LayoutMode.SIMPLE.name
}
return defaultPrefs.getString(SETTINGS_LAYOUT_MODE, defaultLayoutMode).let {
when (it) {
LayoutMode.SIMPLE.name -> LayoutMode.SIMPLE
else -> LayoutMode.PRODUCTIVITY
Expand Down