Skip to content

Commit b387e4b

Browse files
committed
by default hide notification bell
Signed-off-by: alperozturk96 <alper_ozturk@proton.me>
1 parent 7e21f63 commit b387e4b

2 files changed

Lines changed: 7 additions & 5 deletions

File tree

app/src/main/java/com/owncloud/android/ui/activity/FileDisplayActivity.kt

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ import com.nextcloud.utils.extensions.lastFragment
8484
import com.nextcloud.utils.extensions.logFileSize
8585
import com.nextcloud.utils.extensions.navigateToAllFiles
8686
import com.nextcloud.utils.extensions.observeWorker
87+
import com.nextcloud.utils.extensions.setVisibleIf
8788
import com.nextcloud.utils.fileNameValidator.FileNameValidator.checkFolderPath
8889
import com.nextcloud.utils.view.FastScrollUtils
8990
import com.owncloud.android.MainApp
@@ -481,10 +482,9 @@ class FileDisplayActivity :
481482
val result = GetNotificationsRemoteOperation()
482483
.execute(clientFactory.createNextcloudClient(accountManager.user))
483484

484-
if (result.isSuccess && result.getResultData()?.isEmpty() == false) {
485-
runOnUiThread { mNotificationButton.visibility = View.VISIBLE }
486-
} else {
487-
runOnUiThread { mNotificationButton.visibility = View.GONE }
485+
val isVisible = (result.isSuccess && result.getResultData()?.isEmpty() == false)
486+
withContext(Dispatchers.Main) {
487+
mNotificationButton.setVisibleIf(isVisible)
488488
}
489489
} catch (_: CreationException) {
490490
Log_OC.e(TAG, "Could not fetch notifications!")

app/src/main/res/layout/toolbar_standard.xml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,9 @@
221221
app:iconTint="@color/fontAppbar"
222222
app:layout_constraintBottom_toBottomOf="parent"
223223
app:layout_constraintStart_toEndOf="@id/switch_account_button"
224-
app:layout_constraintTop_toTopOf="parent" />
224+
app:layout_constraintTop_toTopOf="parent"
225+
android:visibility="gone"
226+
tools:visibility="visible" />
225227

226228
</LinearLayout>
227229

0 commit comments

Comments
 (0)