From d10a484afbd95ecb65c2142fb5e7dddc22059e92 Mon Sep 17 00:00:00 2001 From: Jonas Date: Mon, 16 Sep 2024 00:44:09 +0200 Subject: [PATCH] Use `isAllDay` instead of `isSingleDayAllDayEvent` to decide top bar placement This should fix #82. In my opinion, it makes much more sense to show multi-day all-day events always in the top bar (like single-day all-day events), independent of this config setting. --- .../main/kotlin/org/fossify/calendar/fragments/WeekFragment.kt | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/app/src/main/kotlin/org/fossify/calendar/fragments/WeekFragment.kt b/app/src/main/kotlin/org/fossify/calendar/fragments/WeekFragment.kt index 53bcb4424..5335107ff 100644 --- a/app/src/main/kotlin/org/fossify/calendar/fragments/WeekFragment.kt +++ b/app/src/main/kotlin/org/fossify/calendar/fragments/WeekFragment.kt @@ -752,8 +752,7 @@ class WeekFragment : Fragment(), WeeklyCalendar { private fun shouldAddEventOnTopBar(isAllDay: Boolean, startDayCode: String, endDayCode: String): Boolean { val spansMultipleDays = startDayCode != endDayCode - val isSingleDayAllDayEvent = isAllDay && !spansMultipleDays - return isSingleDayAllDayEvent || (spansMultipleDays && config.showMidnightSpanningEventsAtTop) + return isAllDay || (spansMultipleDays && config.showMidnightSpanningEventsAtTop) } @SuppressLint("NewApi")