Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| b62f097392 | |||
| 210ddff8d8 |
@@ -126,6 +126,23 @@ Deliberately deferred (add only if needed):
|
|||||||
- Shared `InlineTextField` extracted to `ui.common` (event form + calendar
|
- Shared `InlineTextField` extracted to `ui.common` (event form + calendar
|
||||||
editor share one input style)
|
editor share one input style)
|
||||||
|
|
||||||
|
## v2.3 — Material 3 grouped-list redesign (shipped 2026-06-16)
|
||||||
|
|
||||||
|
A structural + visual pass adopting one shared blueprint (modelled on the ReFra
|
||||||
|
gallery app) across Settings, the calendar manager and the navigation drawer.
|
||||||
|
|
||||||
|
- Shared `ui/common/GroupedList.kt`: `CollapsingScaffold` (a `LargeTopAppBar`
|
||||||
|
whose title collapses on scroll) + `GroupedRow` (Position-based corner
|
||||||
|
grouping, press-animated corners, `selected` + `minHeight` knobs).
|
||||||
|
- Settings: category hub with About card on top and sliding sub-pages
|
||||||
|
(Appearance / New event form / Notifications); theme/week-start/language
|
||||||
|
pickers moved from `DropdownMenu` to OptionCard dialogs; token-based icon
|
||||||
|
chips; `ic_gitea.xml` for the About "Source" button.
|
||||||
|
- Calendar manager + drawer restyled to match; shared `CalendarColorChip`;
|
||||||
|
drawer scrolls as one with the active view highlighted.
|
||||||
|
- Cards use `surfaceContainerHigh` for readable contrast.
|
||||||
|
- Donate button on the About card deferred (target TBD).
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
# Backlog (theme-based, post-v2.1)
|
# Backlog (theme-based, post-v2.1)
|
||||||
@@ -146,13 +163,20 @@ unblock a later item. Order is a plan, not a contract — revisit after each lan
|
|||||||
**Tier 1 — finish the current arc (create/edit + calendars)**
|
**Tier 1 — finish the current arc (create/edit + calendars)**
|
||||||
1. Tap-to-create in day/week *(shipped v2.2.0)* — prefilled create from an empty slot
|
1. Tap-to-create in day/week *(shipped v2.2.0)* — prefilled create from an empty slot
|
||||||
2. Local calendar management + "manage in source app" deep-links *(shipped v2.2.0)*
|
2. Local calendar management + "manage in source app" deep-links *(shipped v2.2.0)*
|
||||||
3. **Settings redesign & restructure** *(next, high prio)* — see scope below
|
3. ~~Settings redesign & restructure~~ *(shipped v2.3.0 — grew into the full
|
||||||
4. Per-event color — reuses the calendar color picker/palette; closes the create/edit theme
|
grouped-list blueprint across Settings + calendars + drawer; see "v2.3"
|
||||||
5. Duplicate event — detail action → prefilled create form; near-free on the tap-to-create prefill infra
|
above)*
|
||||||
|
4. ~~Per-event color~~ *(shipped v2.4.0)* — palette calendars write
|
||||||
|
`EVENT_COLOR_KEY` (sync-safe); local/opted-in calendars write a raw
|
||||||
|
`EVENT_COLOR`; off-by-default setting for no-palette synced calendars
|
||||||
|
5. **Duplicate event** *(next)* — detail action → prefilled create form; near-free on the tap-to-create prefill infra
|
||||||
|
|
||||||
(Tier 2+ numbering below shifts accordingly; ranking unchanged.)
|
(Tier 2+ numbering below shifts accordingly; ranking unchanged.)
|
||||||
|
|
||||||
### Settings redesign & restructure *(next, high prio)*
|
### Settings redesign & restructure *(shipped v2.3.0)*
|
||||||
|
|
||||||
|
The original scope below is kept as a record; the implementation expanded from a
|
||||||
|
sub-screen restructure into the shared grouped-list blueprint (see "v2.3" above).
|
||||||
|
|
||||||
The settings screen has grown into a flat vertical scroll of divider-separated
|
The settings screen has grown into a flat vertical scroll of divider-separated
|
||||||
sections (Appearance, Event form, Notifications, Calendars, Language, About) and
|
sections (Appearance, Event form, Notifications, Calendars, Language, About) and
|
||||||
|
|||||||
@@ -1,14 +1,15 @@
|
|||||||
# Calendula — Current State
|
# Calendula — Current State
|
||||||
|
|
||||||
*Last updated: 2026-06-16*
|
*Last updated: 2026-06-17*
|
||||||
|
|
||||||
## Status
|
## Status
|
||||||
|
|
||||||
**Milestone:** 2 (write support) **complete** — v2.0.0 shipped 2026-06-11;
|
**Milestone:** 2 (write support) **complete** — v2.0.0 shipped 2026-06-11;
|
||||||
v2.1.0 (month event grid, drawer view tabs, cursor fix) shipped 2026-06-15.
|
v2.1.0 (month event grid, drawer view tabs, cursor fix) shipped 2026-06-15.
|
||||||
**Phase:** post-2.1 backlog work. v2.2.0 (tap-to-create in day/week + local
|
**Phase:** post-2.1 backlog work. v2.2.0 (tap-to-create in day/week + local
|
||||||
calendar management with per-calendar "manage in source app" deep-links)
|
calendar management) and v2.3.0 (Material 3 grouped-list redesign of Settings,
|
||||||
shipped 2026-06-16. The backlog is now organised by theme in `ROADMAP.md`.
|
the calendar manager and the navigation drawer) both shipped 2026-06-16. The
|
||||||
|
backlog is now organised by theme in `ROADMAP.md`.
|
||||||
|
|
||||||
## Progress
|
## Progress
|
||||||
|
|
||||||
@@ -86,18 +87,42 @@ shipped 2026-06-16. The backlog is now organised by theme in `ROADMAP.md`.
|
|||||||
deep-link (resolved from the account's authenticator: DAVx5/ICSx5/…) and
|
deep-link (resolved from the account's authenticator: DAVx5/ICSx5/…) and
|
||||||
an add-account shortcut. Shared `InlineTextField` extracted to `ui.common`
|
an add-account shortcut. Shared `InlineTextField` extracted to `ui.common`
|
||||||
|
|
||||||
|
- [x] v2.3 settings/calendars/drawer redesign (shipped 2026-06-16) — adopted a
|
||||||
|
shared Material 3 grouped-list blueprint, modelled on the ReFra gallery app
|
||||||
|
and extracted to `ui/common/GroupedList.kt` (`CollapsingScaffold` with a
|
||||||
|
`LargeTopAppBar` exit-until-collapsed title; `GroupedRow` with Position-based
|
||||||
|
corner grouping, press-animated corners, `selected` + `minHeight` knobs).
|
||||||
|
- Settings: category hub (About card on top → version mark at the foot) with
|
||||||
|
sliding sub-pages (Appearance / New event form / Notifications); token-
|
||||||
|
based icon chips; theme/week-start/language pickers migrated from
|
||||||
|
`DropdownMenu` to OptionCard dialogs. New `ic_gitea.xml` (Simple Icons,
|
||||||
|
verbatim path) for the About "Source" button; en+de strings.
|
||||||
|
- Calendar manager: same collapsing scaffold + grouped rows; shared
|
||||||
|
`CalendarColorChip` (neutral chip, pastelised calendar glyph).
|
||||||
|
- Navigation drawer: branded header, grouped View switcher (active view
|
||||||
|
highlighted via `secondaryContainer`), the filter list restyled to
|
||||||
|
grouped rows with a trailing checkbox; the whole drawer scrolls as one.
|
||||||
|
- Cards use `surfaceContainerHigh` for readable contrast against `surface`.
|
||||||
|
- Donate button on the About card deferred (target still TBD).
|
||||||
|
|
||||||
|
- [x] v2.4 per-event color (shipped 2026-06-17) — an optional "Color" field in
|
||||||
|
the event form. Read/render already resolved `EVENT_COLOR` with a calendar
|
||||||
|
fallback; this adds the write side and the picker. Palette-backed calendars
|
||||||
|
(Google, some CalDAV) pick from the account's `Colors` (`TYPE_EVENT`) and
|
||||||
|
write `EVENT_COLOR_KEY` so the color round-trips through sync; local
|
||||||
|
calendars write a raw `EVENT_COLOR` from the shared `CALENDAR_COLOR_PALETTE`
|
||||||
|
(extracted with the swatch row to `ui/common/ColorSwatchRow.kt`). Switching
|
||||||
|
calendars resets the choice (a key is account-scoped). A settings toggle
|
||||||
|
("Allow colors on unsupported calendars", off by default) extends the raw
|
||||||
|
path to synced calendars with no palette, with an honest "may not survive
|
||||||
|
sync" warning on the picker and in Settings. Color writes flow through
|
||||||
|
insert / dirty-checked update / occurrence-exception; mapper + form tests.
|
||||||
|
|
||||||
## Next
|
## Next
|
||||||
|
|
||||||
1. Monitor the F-Droid build/publish for the v2.2.0 tag
|
1. Monitor the F-Droid build/publish for the v2.4.0 tag
|
||||||
2. Decide the "Locations & People" and "remote calendar create/edit"
|
2. Decide the "Locations & People" and "remote calendar create/edit"
|
||||||
go/no-go calls (both hinge on the INTERNET permission) — see `ROADMAP.md`
|
go/no-go calls (both hinge on the INTERNET permission) — see `ROADMAP.md`
|
||||||
3. **Settings redesign & restructure** is the agreed high-prio next item
|
3. **Duplicate event** and **jump-to-date** are the cheap follow-ups; then
|
||||||
(2026-06-16) — group into M3 cards / sub-screens, and migrate the
|
agenda view (strategic, backs a future widget). Full ranked sequence in
|
||||||
theme/week-start/language `DropdownMenu` selectors to the OptionCard modal
|
|
||||||
default (current dropdowns violate `option-card-modal-style-default`).
|
|
||||||
Structure + style pass only, no new settings features.
|
|
||||||
4. **Per-event color** follows — reuses the color picker + palette plumbing
|
|
||||||
from local calendar management; finishes the create/edit theme.
|
|
||||||
5. Then agenda view (strategic, backs a future widget); jump-to-date and
|
|
||||||
duplicate event remain cheap follow-ups. Full ranked sequence in
|
|
||||||
`ROADMAP.md` → "Near-term sequence".
|
`ROADMAP.md` → "Near-term sequence".
|
||||||
|
|||||||
35
CHANGELOG.md
35
CHANGELOG.md
@@ -7,6 +7,41 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|||||||
|
|
||||||
## [Unreleased]
|
## [Unreleased]
|
||||||
|
|
||||||
|
## [2.4.0] — 2026-06-17
|
||||||
|
|
||||||
|
### Added
|
||||||
|
- Per-event colors: give a single event its own color, instead of always
|
||||||
|
inheriting its calendar's. Add the new "Color" field from "More fields" in
|
||||||
|
the event form. On calendars that publish their own color set — such as
|
||||||
|
Google — you pick from that calendar's palette, so the color is stored
|
||||||
|
with the event and shows correctly on every synced device. On local
|
||||||
|
calendars you pick from Calendula's palette. "Reset" returns an event to
|
||||||
|
its calendar's color
|
||||||
|
- A new "Allow colors on unsupported calendars" setting (New event form,
|
||||||
|
off by default) extends per-event colors to calendars that publish no
|
||||||
|
color set of their own (some CalDAV). Such a color is kept on the device
|
||||||
|
and may be dropped or overwritten on that calendar's next sync — a
|
||||||
|
limitation of those calendars, called out plainly in the setting and on
|
||||||
|
the color picker
|
||||||
|
|
||||||
|
## [2.3.0] — 2026-06-16
|
||||||
|
|
||||||
|
### Changed
|
||||||
|
- Redesigned Settings around the Material 3 grouped-list pattern: a large
|
||||||
|
title that collapses into the toolbar as you scroll, category cards on the
|
||||||
|
main screen, and dedicated sub-pages for Appearance, the new-event form, and
|
||||||
|
Notifications. The theme, week-start and language pickers now use the app's
|
||||||
|
standard option-card dialogs instead of dropdown menus
|
||||||
|
- About moved to the top of Settings as a card — app icon, author, and quick
|
||||||
|
links to the source code and licence — with the version shown plainly at the
|
||||||
|
foot of the list
|
||||||
|
- The Calendars screen now uses the same grouped-card layout and collapsing
|
||||||
|
title, and each calendar shows a soft pastel-tinted calendar glyph rather
|
||||||
|
than a plain colour swatch
|
||||||
|
- Redesigned the navigation drawer to match: a branded header, the
|
||||||
|
Month / Week / Day switch and your calendars as grouped cards (with the
|
||||||
|
active view highlighted), and the whole drawer now scrolls as one
|
||||||
|
|
||||||
## [2.2.0] — 2026-06-16
|
## [2.2.0] — 2026-06-16
|
||||||
|
|
||||||
### Added
|
### Added
|
||||||
|
|||||||
@@ -28,8 +28,8 @@ android {
|
|||||||
// the tag, with versionCode = MAJOR*10000 + MINOR*100 + PATCH
|
// the tag, with versionCode = MAJOR*10000 + MINOR*100 + PATCH
|
||||||
// (e.g. v2.0.0 -> 20000). These committed values are the dev/local
|
// (e.g. v2.0.0 -> 20000). These committed values are the dev/local
|
||||||
// default; keep them matching the latest released tag. See docs/RELEASING.md.
|
// default; keep them matching the latest released tag. See docs/RELEASING.md.
|
||||||
versionCode = 20200
|
versionCode = 20400
|
||||||
versionName = "2.2.0"
|
versionName = "2.4.0"
|
||||||
|
|
||||||
testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
|
testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -14,6 +14,7 @@ import android.util.Log
|
|||||||
import dagger.hilt.android.qualifiers.ApplicationContext
|
import dagger.hilt.android.qualifiers.ApplicationContext
|
||||||
import de.jeanlucmakiola.calendula.domain.Attendee
|
import de.jeanlucmakiola.calendula.domain.Attendee
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
|
import de.jeanlucmakiola.calendula.domain.EventColorOption
|
||||||
import de.jeanlucmakiola.calendula.domain.EventDetail
|
import de.jeanlucmakiola.calendula.domain.EventDetail
|
||||||
import de.jeanlucmakiola.calendula.domain.EventForm
|
import de.jeanlucmakiola.calendula.domain.EventForm
|
||||||
import de.jeanlucmakiola.calendula.domain.EventInstance
|
import de.jeanlucmakiola.calendula.domain.EventInstance
|
||||||
@@ -37,6 +38,15 @@ interface CalendarDataSource {
|
|||||||
fun instances(beginMillis: Long, endMillis: Long): List<EventInstance>
|
fun instances(beginMillis: Long, endMillis: Long): List<EventInstance>
|
||||||
fun eventDetail(eventId: Long): EventDetail?
|
fun eventDetail(eventId: Long): EventDetail?
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The event-colour palette the calendar's account publishes
|
||||||
|
* (`CalendarContract.Colors`, `TYPE_EVENT`), sorted by key. Empty when the
|
||||||
|
* account exposes no palette (most local calendars, some CalDAV) — the
|
||||||
|
* signal that a custom colour can only be written as a raw `EVENT_COLOR`,
|
||||||
|
* which a synced calendar may drop on its next sync.
|
||||||
|
*/
|
||||||
|
fun eventColorPalette(calendarId: Long): List<EventColorOption>
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Create a new device-only (`ACCOUNT_TYPE_LOCAL`) calendar the app owns;
|
* Create a new device-only (`ACCOUNT_TYPE_LOCAL`) calendar the app owns;
|
||||||
* returns its `Calendars._ID`. Inserted through the sync-adapter URI so the
|
* returns its `Calendars._ID`. Inserted through the sync-adapter URI so the
|
||||||
@@ -215,6 +225,46 @@ class AndroidCalendarDataSource @Inject constructor(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
override fun eventColorPalette(calendarId: Long): List<EventColorOption> {
|
||||||
|
val account = calendarAccount(calendarId) ?: return emptyList()
|
||||||
|
return resolver.query(
|
||||||
|
CalendarContract.Colors.CONTENT_URI,
|
||||||
|
arrayOf(CalendarContract.Colors.COLOR_KEY, CalendarContract.Colors.COLOR),
|
||||||
|
CalendarContract.Colors.ACCOUNT_NAME + " = ? AND " +
|
||||||
|
CalendarContract.Colors.ACCOUNT_TYPE + " = ? AND " +
|
||||||
|
CalendarContract.Colors.COLOR_TYPE + " = ?",
|
||||||
|
arrayOf(
|
||||||
|
account.name,
|
||||||
|
account.type,
|
||||||
|
CalendarContract.Colors.TYPE_EVENT.toString(),
|
||||||
|
),
|
||||||
|
null,
|
||||||
|
)?.use { c ->
|
||||||
|
c.mapAll { EventColorOption(key = it.getString(0).orEmpty(), argb = it.getInt(1)) }
|
||||||
|
}
|
||||||
|
?.filter { it.key.isNotEmpty() }
|
||||||
|
?.sortedBy { it.key }
|
||||||
|
?: emptyList()
|
||||||
|
}
|
||||||
|
|
||||||
|
/** The account a calendar belongs to, for scoping a `Colors` lookup. */
|
||||||
|
private fun calendarAccount(calendarId: Long): CalendarAccount? = resolver.query(
|
||||||
|
ContentUris.withAppendedId(CalendarContract.Calendars.CONTENT_URI, calendarId),
|
||||||
|
arrayOf(
|
||||||
|
CalendarContract.Calendars.ACCOUNT_NAME,
|
||||||
|
CalendarContract.Calendars.ACCOUNT_TYPE,
|
||||||
|
),
|
||||||
|
null, null, null,
|
||||||
|
)?.use { c ->
|
||||||
|
if (c.moveToFirst()) {
|
||||||
|
CalendarAccount(name = c.getString(0).orEmpty(), type = c.getString(1).orEmpty())
|
||||||
|
} else {
|
||||||
|
null
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
private data class CalendarAccount(val name: String, val type: String)
|
||||||
|
|
||||||
override fun insertEvent(form: EventForm): Long {
|
override fun insertEvent(form: EventForm): Long {
|
||||||
val times = form.toWriteTimes(ZoneId.systemDefault())
|
val times = form.toWriteTimes(ZoneId.systemDefault())
|
||||||
val values = ContentValues().apply {
|
val values = ContentValues().apply {
|
||||||
@@ -240,6 +290,13 @@ class AndroidCalendarDataSource @Inject constructor(
|
|||||||
?.let { put(CalendarContract.Events.EVENT_LOCATION, it) }
|
?.let { put(CalendarContract.Events.EVENT_LOCATION, it) }
|
||||||
form.description.trim().takeIf { it.isNotEmpty() }
|
form.description.trim().takeIf { it.isNotEmpty() }
|
||||||
?.let { put(CalendarContract.Events.DESCRIPTION, it) }
|
?.let { put(CalendarContract.Events.DESCRIPTION, it) }
|
||||||
|
// A null colour just leaves both columns unset (the event inherits
|
||||||
|
// its calendar's colour), so only the key/raw cases are written.
|
||||||
|
when {
|
||||||
|
form.colorKey != null ->
|
||||||
|
put(CalendarContract.Events.EVENT_COLOR_KEY, form.colorKey)
|
||||||
|
form.color != null -> put(CalendarContract.Events.EVENT_COLOR, form.color)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
val uri = resolver.insert(CalendarContract.Events.CONTENT_URI, values)
|
val uri = resolver.insert(CalendarContract.Events.CONTENT_URI, values)
|
||||||
?: throw WriteFailedException("insert event into calendar id=${form.calendarId}")
|
?: throw WriteFailedException("insert event into calendar id=${form.calendarId}")
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
package de.jeanlucmakiola.calendula.data.calendar
|
package de.jeanlucmakiola.calendula.data.calendar
|
||||||
|
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
|
import de.jeanlucmakiola.calendula.domain.EventColorOption
|
||||||
import de.jeanlucmakiola.calendula.domain.EventDetail
|
import de.jeanlucmakiola.calendula.domain.EventDetail
|
||||||
import de.jeanlucmakiola.calendula.domain.EventForm
|
import de.jeanlucmakiola.calendula.domain.EventForm
|
||||||
import de.jeanlucmakiola.calendula.domain.EventInstance
|
import de.jeanlucmakiola.calendula.domain.EventInstance
|
||||||
@@ -12,6 +13,12 @@ interface CalendarRepository {
|
|||||||
fun instances(range: ClosedRange<Instant>): Flow<List<EventInstance>>
|
fun instances(range: ClosedRange<Instant>): Flow<List<EventInstance>>
|
||||||
suspend fun eventDetail(eventId: Long): EventDetail
|
suspend fun eventDetail(eventId: Long): EventDetail
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The event-colour palette a calendar's account publishes; empty when it
|
||||||
|
* exposes none (see [CalendarDataSource.eventColorPalette]).
|
||||||
|
*/
|
||||||
|
suspend fun eventColorPalette(calendarId: Long): List<EventColorOption>
|
||||||
|
|
||||||
/** Create a device-only (LOCAL) calendar the app owns; returns its id. */
|
/** Create a device-only (LOCAL) calendar the app owns; returns its id. */
|
||||||
suspend fun createLocalCalendar(displayName: String, color: Int, description: String?): Long
|
suspend fun createLocalCalendar(displayName: String, color: Int, description: String?): Long
|
||||||
|
|
||||||
|
|||||||
@@ -3,6 +3,7 @@ package de.jeanlucmakiola.calendula.data.calendar
|
|||||||
import de.jeanlucmakiola.calendula.data.di.IoDispatcher
|
import de.jeanlucmakiola.calendula.data.di.IoDispatcher
|
||||||
import de.jeanlucmakiola.calendula.data.prefs.CalendarPrefs
|
import de.jeanlucmakiola.calendula.data.prefs.CalendarPrefs
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
|
import de.jeanlucmakiola.calendula.domain.EventColorOption
|
||||||
import de.jeanlucmakiola.calendula.domain.EventDetail
|
import de.jeanlucmakiola.calendula.domain.EventDetail
|
||||||
import de.jeanlucmakiola.calendula.domain.EventForm
|
import de.jeanlucmakiola.calendula.domain.EventForm
|
||||||
import de.jeanlucmakiola.calendula.domain.EventInstance
|
import de.jeanlucmakiola.calendula.domain.EventInstance
|
||||||
@@ -70,6 +71,9 @@ class CalendarRepositoryImpl @Inject constructor(
|
|||||||
dataSource.eventDetail(eventId) ?: throw NoSuchEventException(eventId)
|
dataSource.eventDetail(eventId) ?: throw NoSuchEventException(eventId)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
override suspend fun eventColorPalette(calendarId: Long): List<EventColorOption> =
|
||||||
|
withContext(io) { dataSource.eventColorPalette(calendarId) }
|
||||||
|
|
||||||
override suspend fun createLocalCalendar(
|
override suspend fun createLocalCalendar(
|
||||||
displayName: String,
|
displayName: String,
|
||||||
color: Int,
|
color: Int,
|
||||||
|
|||||||
@@ -46,11 +46,16 @@ internal fun ColumnReader.toEventDetailCore(
|
|||||||
// localized placeholder, and the edit form must prefill the true value.
|
// localized placeholder, and the edit form must prefill the true value.
|
||||||
val title = getString(EventDetailProjection.IDX_TITLE).orEmpty()
|
val title = getString(EventDetailProjection.IDX_TITLE).orEmpty()
|
||||||
|
|
||||||
val color = if (isNull(EventDetailProjection.IDX_EVENT_COLOR)) {
|
// The event's own colour (null = inherits the calendar's) is kept apart
|
||||||
getInt(EventDetailProjection.IDX_CALENDAR_COLOR)
|
// from the resolved display colour: the edit form needs to tell the two
|
||||||
|
// cases apart, while the instance carries the calendar fallback for display.
|
||||||
|
val eventColor = if (isNull(EventDetailProjection.IDX_EVENT_COLOR)) {
|
||||||
|
null
|
||||||
} else {
|
} else {
|
||||||
getInt(EventDetailProjection.IDX_EVENT_COLOR)
|
getInt(EventDetailProjection.IDX_EVENT_COLOR)
|
||||||
}
|
}
|
||||||
|
val eventColorKey = getString(EventDetailProjection.IDX_EVENT_COLOR_KEY)
|
||||||
|
val color = eventColor ?: getInt(EventDetailProjection.IDX_CALENDAR_COLOR)
|
||||||
|
|
||||||
val eventId = getLong(EventDetailProjection.IDX_EVENT_ID)
|
val eventId = getLong(EventDetailProjection.IDX_EVENT_ID)
|
||||||
val instance = EventInstance(
|
val instance = EventInstance(
|
||||||
@@ -87,6 +92,8 @@ internal fun ColumnReader.toEventDetailCore(
|
|||||||
accessLevel = mapAccessLevel(getInt(EventDetailProjection.IDX_ACCESS_LEVEL)),
|
accessLevel = mapAccessLevel(getInt(EventDetailProjection.IDX_ACCESS_LEVEL)),
|
||||||
eventTimezone = getString(EventDetailProjection.IDX_EVENT_TIMEZONE),
|
eventTimezone = getString(EventDetailProjection.IDX_EVENT_TIMEZONE),
|
||||||
selfStatus = mapAttendeeStatus(getInt(EventDetailProjection.IDX_SELF_ATTENDEE_STATUS)),
|
selfStatus = mapAttendeeStatus(getInt(EventDetailProjection.IDX_SELF_ATTENDEE_STATUS)),
|
||||||
|
eventColor = eventColor,
|
||||||
|
eventColorKey = eventColorKey,
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -85,6 +85,9 @@ internal fun buildEventUpdateValues(
|
|||||||
if (updated.accessLevel != original.accessLevel) {
|
if (updated.accessLevel != original.accessLevel) {
|
||||||
put(CalendarContract.Events.ACCESS_LEVEL, updated.accessLevel.toProviderValue())
|
put(CalendarContract.Events.ACCESS_LEVEL, updated.accessLevel.toProviderValue())
|
||||||
}
|
}
|
||||||
|
if (updated.colorKey != original.colorKey || updated.color != original.color) {
|
||||||
|
putAll(eventColorColumns(updated.colorKey, updated.color))
|
||||||
|
}
|
||||||
|
|
||||||
val timesChanged = updated.start != original.start ||
|
val timesChanged = updated.start != original.start ||
|
||||||
updated.end != original.end ||
|
updated.end != original.end ||
|
||||||
@@ -134,6 +137,28 @@ internal fun buildOccurrenceExceptionValues(
|
|||||||
put(CalendarContract.Events.ACCESS_LEVEL, form.accessLevel.toProviderValue())
|
put(CalendarContract.Events.ACCESS_LEVEL, form.accessLevel.toProviderValue())
|
||||||
put(CalendarContract.Events.EVENT_LOCATION, form.location.trim().ifEmpty { null })
|
put(CalendarContract.Events.EVENT_LOCATION, form.location.trim().ifEmpty { null })
|
||||||
put(CalendarContract.Events.DESCRIPTION, form.description.trim().ifEmpty { null })
|
put(CalendarContract.Events.DESCRIPTION, form.description.trim().ifEmpty { null })
|
||||||
|
putAll(eventColorColumns(form.colorKey, form.color))
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The `EVENT_COLOR` / `EVENT_COLOR_KEY` columns for a colour selection. A
|
||||||
|
* [colorKey] writes the key alone (the provider derives `EVENT_COLOR` from the
|
||||||
|
* account's palette, so the colour round-trips through sync); a raw [color]
|
||||||
|
* writes `EVENT_COLOR` and clears any key; "no colour" clears both so the event
|
||||||
|
* falls back to its calendar's colour. The two are never written together —
|
||||||
|
* the provider rejects a raw colour on a calendar that publishes a palette,
|
||||||
|
* which is exactly why palette calendars only ever go through the key.
|
||||||
|
*/
|
||||||
|
internal fun eventColorColumns(colorKey: String?, color: Int?): Map<String, Any?> = when {
|
||||||
|
colorKey != null -> mapOf(CalendarContract.Events.EVENT_COLOR_KEY to colorKey)
|
||||||
|
color != null -> mapOf(
|
||||||
|
CalendarContract.Events.EVENT_COLOR_KEY to null,
|
||||||
|
CalendarContract.Events.EVENT_COLOR to color,
|
||||||
|
)
|
||||||
|
else -> mapOf(
|
||||||
|
CalendarContract.Events.EVENT_COLOR_KEY to null,
|
||||||
|
CalendarContract.Events.EVENT_COLOR to null,
|
||||||
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
@@ -74,6 +74,7 @@ internal object EventDetailProjection {
|
|||||||
CalendarContract.Events.ACCESS_LEVEL,
|
CalendarContract.Events.ACCESS_LEVEL,
|
||||||
CalendarContract.Events.EVENT_TIMEZONE,
|
CalendarContract.Events.EVENT_TIMEZONE,
|
||||||
CalendarContract.Events.SELF_ATTENDEE_STATUS,
|
CalendarContract.Events.SELF_ATTENDEE_STATUS,
|
||||||
|
CalendarContract.Events.EVENT_COLOR_KEY,
|
||||||
)
|
)
|
||||||
|
|
||||||
const val IDX_EVENT_ID = 0
|
const val IDX_EVENT_ID = 0
|
||||||
@@ -93,6 +94,7 @@ internal object EventDetailProjection {
|
|||||||
const val IDX_ACCESS_LEVEL = 14
|
const val IDX_ACCESS_LEVEL = 14
|
||||||
const val IDX_EVENT_TIMEZONE = 15
|
const val IDX_EVENT_TIMEZONE = 15
|
||||||
const val IDX_SELF_ATTENDEE_STATUS = 16
|
const val IDX_SELF_ATTENDEE_STATUS = 16
|
||||||
|
const val IDX_EVENT_COLOR_KEY = 17
|
||||||
}
|
}
|
||||||
|
|
||||||
internal object AttendeeProjection {
|
internal object AttendeeProjection {
|
||||||
|
|||||||
@@ -99,6 +99,22 @@ class SettingsPrefs @Inject constructor(
|
|||||||
store.edit { it[REMINDERS_ENABLED_KEY] = enabled }
|
store.edit { it[REMINDERS_ENABLED_KEY] = enabled }
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Whether to offer a custom event colour even on calendars that publish no
|
||||||
|
* colour palette (most local calendars handle it fine; synced calendars
|
||||||
|
* without a palette — some CalDAV — may drop or overwrite a raw colour on
|
||||||
|
* their next sync). Defaults to OFF: such calendars hide the colour picker
|
||||||
|
* until the user opts in, accepting the limitation. Local calendars and
|
||||||
|
* palette-backed calendars (Google, …) are unaffected by this flag.
|
||||||
|
*/
|
||||||
|
val allowColorOnUnsupportedCalendars: Flow<Boolean> = store.data.map { prefs ->
|
||||||
|
prefs[ALLOW_COLOR_UNSUPPORTED_KEY] ?: false
|
||||||
|
}
|
||||||
|
|
||||||
|
suspend fun setAllowColorOnUnsupportedCalendars(enabled: Boolean) {
|
||||||
|
store.edit { it[ALLOW_COLOR_UNSUPPORTED_KEY] = enabled }
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Whether the one-time reminder onboarding step (after the calendar
|
* Whether the one-time reminder onboarding step (after the calendar
|
||||||
* grant) has been shown — also true for users who tapped "not now".
|
* grant) has been shown — also true for users who tapped "not now".
|
||||||
@@ -125,6 +141,8 @@ class SettingsPrefs @Inject constructor(
|
|||||||
internal val FORM_FIELDS_KEY = stringPreferencesKey("event_form_default_fields")
|
internal val FORM_FIELDS_KEY = stringPreferencesKey("event_form_default_fields")
|
||||||
internal val REMINDERS_ENABLED_KEY = booleanPreferencesKey("reminders_enabled")
|
internal val REMINDERS_ENABLED_KEY = booleanPreferencesKey("reminders_enabled")
|
||||||
internal val REMINDER_ONBOARDING_KEY = booleanPreferencesKey("reminder_onboarding_done")
|
internal val REMINDER_ONBOARDING_KEY = booleanPreferencesKey("reminder_onboarding_done")
|
||||||
|
internal val ALLOW_COLOR_UNSUPPORTED_KEY =
|
||||||
|
booleanPreferencesKey("allow_color_unsupported_calendars")
|
||||||
internal val DEFAULT_FORM_FIELDS =
|
internal val DEFAULT_FORM_FIELDS =
|
||||||
setOf(EventFormField.Location, EventFormField.Description)
|
setOf(EventFormField.Location, EventFormField.Description)
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -30,6 +30,17 @@ data class EventForm(
|
|||||||
* those are kept verbatim until the user picks something else.
|
* those are kept verbatim until the user picks something else.
|
||||||
*/
|
*/
|
||||||
val rrule: String? = null,
|
val rrule: String? = null,
|
||||||
|
/**
|
||||||
|
* The event's own colour, or null to inherit the calendar's colour.
|
||||||
|
* [colorKey] is a `Colors.COLOR_KEY` from the calendar account's published
|
||||||
|
* event palette (Google, some CalDAV) — written as `EVENT_COLOR_KEY` so it
|
||||||
|
* round-trips through sync. When it is null but [color] is set, [color] is
|
||||||
|
* a raw ARGB written as `EVENT_COLOR` (local calendars, or synced ones the
|
||||||
|
* user opted into despite no palette). [color] mirrors the key's swatch when
|
||||||
|
* [colorKey] is set, so the picker can highlight it.
|
||||||
|
*/
|
||||||
|
val colorKey: String? = null,
|
||||||
|
val color: Int? = null,
|
||||||
)
|
)
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -43,6 +54,7 @@ enum class EventFormField {
|
|||||||
Recurrence,
|
Recurrence,
|
||||||
Availability,
|
Availability,
|
||||||
Visibility,
|
Visibility,
|
||||||
|
Color,
|
||||||
}
|
}
|
||||||
|
|
||||||
enum class EventFormProblem {
|
enum class EventFormProblem {
|
||||||
@@ -91,6 +103,11 @@ fun EventDetail.toEditForm(beginMillis: Long, endMillis: Long, zone: TimeZone):
|
|||||||
availability = availability,
|
availability = availability,
|
||||||
accessLevel = accessLevel,
|
accessLevel = accessLevel,
|
||||||
rrule = rrule?.removePrefix("RRULE:")?.takeIf { it.isNotBlank() },
|
rrule = rrule?.removePrefix("RRULE:")?.takeIf { it.isNotBlank() },
|
||||||
|
// The provider fills EVENT_COLOR from the key, so [color] is the
|
||||||
|
// swatch either way; a null colour means the event inherits its
|
||||||
|
// calendar's colour.
|
||||||
|
colorKey = eventColorKey,
|
||||||
|
color = eventColor,
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -130,6 +147,7 @@ fun EventForm.populatedFields(): Set<EventFormField> = buildSet {
|
|||||||
if (rrule != null) add(EventFormField.Recurrence)
|
if (rrule != null) add(EventFormField.Recurrence)
|
||||||
if (availability != Availability.Busy) add(EventFormField.Availability)
|
if (availability != Availability.Busy) add(EventFormField.Availability)
|
||||||
if (accessLevel != AccessLevel.Default) add(EventFormField.Visibility)
|
if (accessLevel != AccessLevel.Default) add(EventFormField.Visibility)
|
||||||
|
if (colorKey != null || color != null) add(EventFormField.Color)
|
||||||
}
|
}
|
||||||
|
|
||||||
fun EventForm.problems(): Set<EventFormProblem> = buildSet {
|
fun EventForm.problems(): Set<EventFormProblem> = buildSet {
|
||||||
|
|||||||
@@ -58,8 +58,25 @@ data class EventDetail(
|
|||||||
val eventTimezone: String? = null,
|
val eventTimezone: String? = null,
|
||||||
/** This device user's own response (`Events.SELF_ATTENDEE_STATUS`). */
|
/** This device user's own response (`Events.SELF_ATTENDEE_STATUS`). */
|
||||||
val selfStatus: AttendeeStatus = AttendeeStatus.Unknown,
|
val selfStatus: AttendeeStatus = AttendeeStatus.Unknown,
|
||||||
|
/**
|
||||||
|
* The event's own raw colour (`Events.EVENT_COLOR`), null when the event
|
||||||
|
* inherits its calendar's colour. Unlike [EventInstance.color] (which
|
||||||
|
* already folds in the calendar fallback for display) this stays null so
|
||||||
|
* the edit form can tell "has own colour" from "inherits".
|
||||||
|
*/
|
||||||
|
val eventColor: Int? = null,
|
||||||
|
/** The event's `Events.EVENT_COLOR_KEY` (a calendar-palette key), or null. */
|
||||||
|
val eventColorKey: String? = null,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
/**
|
||||||
|
* One selectable event colour published by a calendar's account
|
||||||
|
* (`CalendarContract.Colors`, `TYPE_EVENT`): [key] is the account-scoped
|
||||||
|
* `COLOR_KEY` written as `EVENT_COLOR_KEY` (so the colour survives sync),
|
||||||
|
* [argb] is the swatch it renders as.
|
||||||
|
*/
|
||||||
|
data class EventColorOption(val key: String, val argb: Int)
|
||||||
|
|
||||||
data class Attendee(
|
data class Attendee(
|
||||||
val name: String,
|
val name: String,
|
||||||
val email: String?,
|
val email: String?,
|
||||||
|
|||||||
@@ -6,14 +6,10 @@ import android.content.Intent
|
|||||||
import android.provider.Settings
|
import android.provider.Settings
|
||||||
import androidx.activity.compose.BackHandler
|
import androidx.activity.compose.BackHandler
|
||||||
import androidx.compose.foundation.background
|
import androidx.compose.foundation.background
|
||||||
import androidx.compose.foundation.border
|
|
||||||
import androidx.compose.foundation.clickable
|
|
||||||
import androidx.compose.foundation.isSystemInDarkTheme
|
import androidx.compose.foundation.isSystemInDarkTheme
|
||||||
import androidx.compose.foundation.layout.Arrangement
|
import androidx.compose.foundation.layout.Arrangement
|
||||||
import androidx.compose.foundation.layout.Box
|
import androidx.compose.foundation.layout.Box
|
||||||
import androidx.compose.foundation.layout.Column
|
import androidx.compose.foundation.layout.Column
|
||||||
import androidx.compose.foundation.layout.ExperimentalLayoutApi
|
|
||||||
import androidx.compose.foundation.layout.FlowRow
|
|
||||||
import androidx.compose.foundation.layout.Row
|
import androidx.compose.foundation.layout.Row
|
||||||
import androidx.compose.foundation.layout.Spacer
|
import androidx.compose.foundation.layout.Spacer
|
||||||
import androidx.compose.foundation.layout.fillMaxSize
|
import androidx.compose.foundation.layout.fillMaxSize
|
||||||
@@ -31,7 +27,6 @@ import androidx.compose.material.icons.automirrored.filled.ArrowBack
|
|||||||
import androidx.compose.material.icons.automirrored.filled.Notes
|
import androidx.compose.material.icons.automirrored.filled.Notes
|
||||||
import androidx.compose.material.icons.filled.Add
|
import androidx.compose.material.icons.filled.Add
|
||||||
import androidx.compose.material.icons.filled.CalendarMonth
|
import androidx.compose.material.icons.filled.CalendarMonth
|
||||||
import androidx.compose.material.icons.filled.Check
|
|
||||||
import androidx.compose.material.icons.filled.Close
|
import androidx.compose.material.icons.filled.Close
|
||||||
import androidx.compose.material.icons.filled.Delete
|
import androidx.compose.material.icons.filled.Delete
|
||||||
import androidx.compose.material.icons.filled.Edit
|
import androidx.compose.material.icons.filled.Edit
|
||||||
@@ -40,8 +35,6 @@ import androidx.compose.material.icons.filled.Palette
|
|||||||
import androidx.compose.material3.AlertDialog
|
import androidx.compose.material3.AlertDialog
|
||||||
import androidx.compose.material3.Button
|
import androidx.compose.material3.Button
|
||||||
import androidx.compose.material3.ExperimentalMaterial3Api
|
import androidx.compose.material3.ExperimentalMaterial3Api
|
||||||
import androidx.compose.material3.FilledTonalButton
|
|
||||||
import androidx.compose.material3.HorizontalDivider
|
|
||||||
import androidx.compose.material3.Icon
|
import androidx.compose.material3.Icon
|
||||||
import androidx.compose.material3.IconButton
|
import androidx.compose.material3.IconButton
|
||||||
import androidx.compose.material3.MaterialTheme
|
import androidx.compose.material3.MaterialTheme
|
||||||
@@ -70,14 +63,20 @@ import androidx.compose.ui.platform.LocalContext
|
|||||||
import androidx.compose.ui.res.stringResource
|
import androidx.compose.ui.res.stringResource
|
||||||
import androidx.compose.ui.text.font.FontWeight
|
import androidx.compose.ui.text.font.FontWeight
|
||||||
import androidx.compose.ui.text.input.KeyboardCapitalization
|
import androidx.compose.ui.text.input.KeyboardCapitalization
|
||||||
import androidx.compose.ui.text.style.TextOverflow
|
|
||||||
import androidx.compose.ui.unit.dp
|
import androidx.compose.ui.unit.dp
|
||||||
import androidx.hilt.navigation.compose.hiltViewModel
|
import androidx.hilt.navigation.compose.hiltViewModel
|
||||||
import androidx.lifecycle.compose.collectAsStateWithLifecycle
|
import androidx.lifecycle.compose.collectAsStateWithLifecycle
|
||||||
import de.jeanlucmakiola.calendula.R
|
import de.jeanlucmakiola.calendula.R
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.CALENDAR_COLOR_PALETTE
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.CalendarColorChip
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.CollapsingScaffold
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.ColorSwatchRow
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.GroupedRow
|
||||||
import de.jeanlucmakiola.calendula.ui.common.InlineTextField
|
import de.jeanlucmakiola.calendula.ui.common.InlineTextField
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.Position
|
||||||
import de.jeanlucmakiola.calendula.ui.common.pastelize
|
import de.jeanlucmakiola.calendula.ui.common.pastelize
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.positionOf
|
||||||
|
|
||||||
/** Sentinel [editorId] meaning "the editor is composing a new calendar". */
|
/** Sentinel [editorId] meaning "the editor is composing a new calendar". */
|
||||||
private const val NEW_CALENDAR_ID = Long.MIN_VALUE
|
private const val NEW_CALENDAR_ID = Long.MIN_VALUE
|
||||||
@@ -139,7 +138,6 @@ fun CalendarsScreen(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@OptIn(ExperimentalMaterial3Api::class)
|
|
||||||
@Composable
|
@Composable
|
||||||
private fun CalendarsList(
|
private fun CalendarsList(
|
||||||
local: List<CalendarSource>,
|
local: List<CalendarSource>,
|
||||||
@@ -150,11 +148,10 @@ private fun CalendarsList(
|
|||||||
onAdd: () -> Unit,
|
onAdd: () -> Unit,
|
||||||
onEdit: (CalendarSource) -> Unit,
|
onEdit: (CalendarSource) -> Unit,
|
||||||
) {
|
) {
|
||||||
|
val context = LocalContext.current
|
||||||
val snackbarHostState = remember { SnackbarHostState() }
|
val snackbarHostState = remember { SnackbarHostState() }
|
||||||
val writeErrorText = stringResource(R.string.calendars_write_error)
|
val writeErrorText = stringResource(R.string.calendars_write_error)
|
||||||
val dark = isSystemInDarkTheme()
|
|
||||||
|
|
||||||
BackHandler(onBack = onBack)
|
|
||||||
LaunchedEffect(error) {
|
LaunchedEffect(error) {
|
||||||
if (error) {
|
if (error) {
|
||||||
snackbarHostState.showSnackbar(writeErrorText)
|
snackbarHostState.showSnackbar(writeErrorText)
|
||||||
@@ -162,42 +159,24 @@ private fun CalendarsList(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
Scaffold(
|
CollapsingScaffold(
|
||||||
modifier = Modifier
|
title = stringResource(R.string.calendars_title),
|
||||||
.fillMaxSize()
|
onBack = onBack,
|
||||||
.background(MaterialTheme.colorScheme.surface),
|
|
||||||
topBar = {
|
|
||||||
TopAppBar(
|
|
||||||
title = { Text(stringResource(R.string.calendars_title)) },
|
|
||||||
navigationIcon = {
|
|
||||||
IconButton(onClick = onBack) {
|
|
||||||
Icon(
|
|
||||||
Icons.AutoMirrored.Filled.ArrowBack,
|
|
||||||
contentDescription = stringResource(R.string.settings_back),
|
|
||||||
)
|
|
||||||
}
|
|
||||||
},
|
|
||||||
)
|
|
||||||
},
|
|
||||||
snackbarHost = { SnackbarHost(snackbarHostState) },
|
snackbarHost = { SnackbarHost(snackbarHostState) },
|
||||||
) { innerPadding ->
|
|
||||||
Column(
|
|
||||||
modifier = Modifier
|
|
||||||
.padding(innerPadding)
|
|
||||||
.fillMaxSize()
|
|
||||||
.verticalScroll(rememberScrollState()),
|
|
||||||
) {
|
) {
|
||||||
|
// Local (device-only) calendars — the calendars the app owns. The
|
||||||
|
// "Add calendar" entry closes the group as its final row.
|
||||||
SectionHeader(stringResource(R.string.calendars_local_header))
|
SectionHeader(stringResource(R.string.calendars_local_header))
|
||||||
if (local.isEmpty()) {
|
if (local.isEmpty()) {
|
||||||
HintText(stringResource(R.string.calendars_local_empty))
|
HintText(stringResource(R.string.calendars_local_empty))
|
||||||
} else {
|
}
|
||||||
local.forEach { calendar ->
|
val localCount = local.size + 1
|
||||||
CalendarRow(
|
local.forEachIndexed { index, calendar ->
|
||||||
name = calendar.displayName,
|
GroupedRow(
|
||||||
color = calendar.color,
|
title = calendar.displayName,
|
||||||
dark = dark,
|
summary = calendar.description,
|
||||||
subtitle = calendar.description,
|
position = positionOf(index, localCount),
|
||||||
onClick = { onEdit(calendar) },
|
leading = { CalendarColorChip(calendar.color) },
|
||||||
trailing = {
|
trailing = {
|
||||||
Icon(
|
Icon(
|
||||||
Icons.Default.Edit,
|
Icons.Default.Edit,
|
||||||
@@ -206,35 +185,43 @@ private fun CalendarsList(
|
|||||||
modifier = Modifier.size(20.dp),
|
modifier = Modifier.size(20.dp),
|
||||||
)
|
)
|
||||||
},
|
},
|
||||||
|
onClick = { onEdit(calendar) },
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
}
|
GroupedRow(
|
||||||
FilledTonalButton(
|
title = stringResource(R.string.calendars_add),
|
||||||
|
position = positionOf(local.size, localCount),
|
||||||
|
leading = { AddAvatar() },
|
||||||
onClick = onAdd,
|
onClick = onAdd,
|
||||||
modifier = Modifier.padding(horizontal = 24.dp, vertical = 8.dp),
|
)
|
||||||
) {
|
|
||||||
Icon(Icons.Default.Add, contentDescription = null, modifier = Modifier.size(18.dp))
|
|
||||||
Spacer(Modifier.width(8.dp))
|
|
||||||
Text(stringResource(R.string.calendars_add))
|
|
||||||
}
|
|
||||||
|
|
||||||
HorizontalDivider(Modifier.padding(vertical = 8.dp))
|
Spacer(Modifier.height(16.dp))
|
||||||
|
|
||||||
|
// Synced calendars — read-only, grouped by account, each with a
|
||||||
|
// per-account "manage in source app" link.
|
||||||
SectionHeader(stringResource(R.string.calendars_synced_header))
|
SectionHeader(stringResource(R.string.calendars_synced_header))
|
||||||
HintText(stringResource(R.string.calendars_synced_hint))
|
HintText(stringResource(R.string.calendars_synced_hint))
|
||||||
synced
|
synced
|
||||||
.groupBy { it.accountName.ifBlank { it.accountType } }
|
.groupBy { it.accountName.ifBlank { it.accountType } }
|
||||||
.forEach { (account, cals) ->
|
.forEach { (account, cals) ->
|
||||||
SyncedAccountGroup(
|
AccountHeader(account = account, accountType = cals.first().accountType)
|
||||||
account = account,
|
cals.forEachIndexed { index, calendar ->
|
||||||
accountType = cals.first().accountType,
|
GroupedRow(
|
||||||
calendars = cals,
|
title = calendar.displayName,
|
||||||
dark = dark,
|
position = positionOf(index, cals.size),
|
||||||
|
leading = { CalendarColorChip(calendar.color) },
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
AddAccountButton()
|
|
||||||
Spacer(Modifier.height(24.dp))
|
|
||||||
}
|
}
|
||||||
|
Spacer(Modifier.height(8.dp))
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.calendars_add_account),
|
||||||
|
position = Position.Alone,
|
||||||
|
leading = { AddAvatar() },
|
||||||
|
onClick = {
|
||||||
|
runCatching { context.startActivity(Intent(Settings.ACTION_ADD_ACCOUNT)) }
|
||||||
|
},
|
||||||
|
)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -335,7 +322,12 @@ private fun CalendarEditor(
|
|||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
)
|
)
|
||||||
Spacer(Modifier.height(12.dp))
|
Spacer(Modifier.height(12.dp))
|
||||||
ColorPalette(selected = color, onSelect = { color = it }, dark = dark)
|
ColorSwatchRow(
|
||||||
|
colors = CALENDAR_COLOR_PALETTE,
|
||||||
|
selected = color,
|
||||||
|
onSelect = { color = it },
|
||||||
|
dark = dark,
|
||||||
|
)
|
||||||
}
|
}
|
||||||
EditorCard(
|
EditorCard(
|
||||||
icon = Icons.AutoMirrored.Filled.Notes,
|
icon = Icons.AutoMirrored.Filled.Notes,
|
||||||
@@ -412,54 +404,13 @@ private fun EditorCard(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@OptIn(ExperimentalLayoutApi::class)
|
|
||||||
@Composable
|
@Composable
|
||||||
private fun ColorPalette(selected: Int, onSelect: (Int) -> Unit, dark: Boolean) {
|
private fun AccountHeader(account: String, accountType: String) {
|
||||||
FlowRow(horizontalArrangement = Arrangement.spacedBy(12.dp)) {
|
|
||||||
CALENDAR_COLOR_PALETTE.forEach { argb ->
|
|
||||||
val isSelected = argb == selected
|
|
||||||
// Show the pastel the calendar will actually render as, not the raw hue.
|
|
||||||
Box(
|
|
||||||
contentAlignment = Alignment.Center,
|
|
||||||
modifier = Modifier
|
|
||||||
.padding(vertical = 4.dp)
|
|
||||||
.size(40.dp)
|
|
||||||
.clip(CircleShape)
|
|
||||||
.background(pastelize(argb, dark))
|
|
||||||
.then(
|
|
||||||
if (isSelected) {
|
|
||||||
Modifier.border(2.dp, MaterialTheme.colorScheme.onSurface, CircleShape)
|
|
||||||
} else {
|
|
||||||
Modifier
|
|
||||||
},
|
|
||||||
)
|
|
||||||
.clickable { onSelect(argb) },
|
|
||||||
) {
|
|
||||||
if (isSelected) {
|
|
||||||
Icon(
|
|
||||||
Icons.Default.Check,
|
|
||||||
contentDescription = null,
|
|
||||||
tint = Color.Black.copy(alpha = 0.7f),
|
|
||||||
modifier = Modifier.size(20.dp),
|
|
||||||
)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
@Composable
|
|
||||||
private fun SyncedAccountGroup(
|
|
||||||
account: String,
|
|
||||||
accountType: String,
|
|
||||||
calendars: List<CalendarSource>,
|
|
||||||
dark: Boolean,
|
|
||||||
) {
|
|
||||||
val context = LocalContext.current
|
val context = LocalContext.current
|
||||||
Row(
|
Row(
|
||||||
modifier = Modifier
|
modifier = Modifier
|
||||||
.fillMaxWidth()
|
.fillMaxWidth()
|
||||||
.padding(start = 24.dp, end = 16.dp, top = 12.dp, bottom = 4.dp),
|
.padding(start = 28.dp, end = 16.dp, top = 16.dp, bottom = 4.dp),
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
) {
|
) {
|
||||||
Text(
|
Text(
|
||||||
@@ -476,65 +427,24 @@ private fun SyncedAccountGroup(
|
|||||||
Text(stringResource(R.string.calendars_manage_in_app))
|
Text(stringResource(R.string.calendars_manage_in_app))
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
calendars.forEach { calendar ->
|
|
||||||
CalendarRow(name = calendar.displayName, color = calendar.color, dark = dark)
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/** Neutral circular chip with a "+" — the leading icon for add-actions. */
|
||||||
@Composable
|
@Composable
|
||||||
private fun AddAccountButton() {
|
private fun AddAvatar() {
|
||||||
val context = LocalContext.current
|
|
||||||
FilledTonalButton(
|
|
||||||
onClick = {
|
|
||||||
runCatching { context.startActivity(Intent(Settings.ACTION_ADD_ACCOUNT)) }
|
|
||||||
},
|
|
||||||
modifier = Modifier.padding(horizontal = 24.dp, vertical = 8.dp),
|
|
||||||
) {
|
|
||||||
Icon(Icons.Default.Add, contentDescription = null, modifier = Modifier.size(18.dp))
|
|
||||||
Spacer(Modifier.width(8.dp))
|
|
||||||
Text(stringResource(R.string.calendars_add_account))
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
@Composable
|
|
||||||
private fun CalendarRow(
|
|
||||||
name: String,
|
|
||||||
color: Int,
|
|
||||||
dark: Boolean,
|
|
||||||
subtitle: String? = null,
|
|
||||||
onClick: (() -> Unit)? = null,
|
|
||||||
trailing: @Composable (() -> Unit)? = null,
|
|
||||||
) {
|
|
||||||
Row(
|
|
||||||
modifier = Modifier
|
|
||||||
.fillMaxWidth()
|
|
||||||
.then(if (onClick != null) Modifier.clickable(onClick = onClick) else Modifier)
|
|
||||||
.padding(horizontal = 24.dp, vertical = 12.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
) {
|
|
||||||
Box(
|
Box(
|
||||||
modifier = Modifier
|
modifier = Modifier
|
||||||
.size(16.dp)
|
.size(40.dp)
|
||||||
.clip(CircleShape)
|
.clip(CircleShape)
|
||||||
.background(pastelize(color, dark)),
|
.background(MaterialTheme.colorScheme.surfaceContainerHighest),
|
||||||
|
contentAlignment = Alignment.Center,
|
||||||
|
) {
|
||||||
|
Icon(
|
||||||
|
Icons.Default.Add,
|
||||||
|
contentDescription = null,
|
||||||
|
tint = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
modifier = Modifier.size(22.dp),
|
||||||
)
|
)
|
||||||
Spacer(Modifier.width(16.dp))
|
|
||||||
Column(modifier = Modifier.weight(1f)) {
|
|
||||||
Text(text = name, style = MaterialTheme.typography.bodyLarge)
|
|
||||||
if (!subtitle.isNullOrBlank()) {
|
|
||||||
Text(
|
|
||||||
text = subtitle,
|
|
||||||
style = MaterialTheme.typography.bodySmall,
|
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
|
||||||
maxLines = 2,
|
|
||||||
overflow = TextOverflow.Ellipsis,
|
|
||||||
)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if (trailing != null) {
|
|
||||||
Spacer(Modifier.width(8.dp))
|
|
||||||
trailing()
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -588,15 +498,3 @@ private fun curatedSourcePackage(accountType: String): String? = when {
|
|||||||
else -> null
|
else -> null
|
||||||
}
|
}
|
||||||
|
|
||||||
/** Google-Calendar-style palette; values are ARGB ints for `Calendars.CALENDAR_COLOR`. */
|
|
||||||
internal val CALENDAR_COLOR_PALETTE: List<Int> = listOf(
|
|
||||||
0xFFD50000, // red
|
|
||||||
0xFFE67C00, // orange
|
|
||||||
0xFFF6BF26, // amber
|
|
||||||
0xFF33B679, // green
|
|
||||||
0xFF0B8043, // dark green
|
|
||||||
0xFF039BE5, // blue
|
|
||||||
0xFF3F51B5, // indigo
|
|
||||||
0xFF8E24AA, // purple
|
|
||||||
0xFF616161, // graphite
|
|
||||||
).map { it.toInt() }
|
|
||||||
|
|||||||
@@ -1,6 +1,20 @@
|
|||||||
package de.jeanlucmakiola.calendula.ui.common
|
package de.jeanlucmakiola.calendula.ui.common
|
||||||
|
|
||||||
|
import androidx.compose.foundation.background
|
||||||
|
import androidx.compose.foundation.isSystemInDarkTheme
|
||||||
|
import androidx.compose.foundation.layout.Box
|
||||||
|
import androidx.compose.foundation.layout.size
|
||||||
|
import androidx.compose.foundation.shape.CircleShape
|
||||||
|
import androidx.compose.material.icons.Icons
|
||||||
|
import androidx.compose.material.icons.filled.CalendarMonth
|
||||||
|
import androidx.compose.material3.Icon
|
||||||
|
import androidx.compose.material3.MaterialTheme
|
||||||
|
import androidx.compose.runtime.Composable
|
||||||
|
import androidx.compose.ui.Alignment
|
||||||
|
import androidx.compose.ui.Modifier
|
||||||
|
import androidx.compose.ui.draw.clip
|
||||||
import androidx.compose.ui.graphics.Color
|
import androidx.compose.ui.graphics.Color
|
||||||
|
import androidx.compose.ui.unit.dp
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Soften a raw calendar color toward a pastel that fits the active theme.
|
* Soften a raw calendar color toward a pastel that fits the active theme.
|
||||||
@@ -15,3 +29,27 @@ fun pastelize(rawArgb: Int, dark: Boolean): Color {
|
|||||||
hsv[2] = if (dark) 0.82f else 0.72f
|
hsv[2] = if (dark) 0.82f else 0.72f
|
||||||
return Color(android.graphics.Color.HSVToColor(hsv))
|
return Color(android.graphics.Color.HSVToColor(hsv))
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Leading avatar for a calendar: a neutral chip holding a calendar glyph tinted
|
||||||
|
* in the calendar's (pastelised) colour. Shared by the calendar manager and the
|
||||||
|
* visibility filter so they read identically.
|
||||||
|
*/
|
||||||
|
@Composable
|
||||||
|
fun CalendarColorChip(color: Int, modifier: Modifier = Modifier) {
|
||||||
|
val dark = isSystemInDarkTheme()
|
||||||
|
Box(
|
||||||
|
modifier = modifier
|
||||||
|
.size(40.dp)
|
||||||
|
.clip(CircleShape)
|
||||||
|
.background(MaterialTheme.colorScheme.surfaceContainerHighest),
|
||||||
|
contentAlignment = Alignment.Center,
|
||||||
|
) {
|
||||||
|
Icon(
|
||||||
|
Icons.Filled.CalendarMonth,
|
||||||
|
contentDescription = null,
|
||||||
|
tint = pastelize(color, dark),
|
||||||
|
modifier = Modifier.size(22.dp),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|||||||
@@ -1,20 +1,33 @@
|
|||||||
package de.jeanlucmakiola.calendula.ui.common
|
package de.jeanlucmakiola.calendula.ui.common
|
||||||
|
|
||||||
|
import androidx.compose.foundation.Image
|
||||||
|
import androidx.compose.foundation.background
|
||||||
|
import androidx.compose.foundation.layout.Box
|
||||||
import androidx.compose.foundation.layout.Column
|
import androidx.compose.foundation.layout.Column
|
||||||
|
import androidx.compose.foundation.layout.Row
|
||||||
import androidx.compose.foundation.layout.Spacer
|
import androidx.compose.foundation.layout.Spacer
|
||||||
import androidx.compose.foundation.layout.fillMaxHeight
|
import androidx.compose.foundation.layout.fillMaxHeight
|
||||||
|
import androidx.compose.foundation.layout.fillMaxWidth
|
||||||
import androidx.compose.foundation.layout.height
|
import androidx.compose.foundation.layout.height
|
||||||
import androidx.compose.foundation.layout.padding
|
import androidx.compose.foundation.layout.padding
|
||||||
|
import androidx.compose.foundation.layout.requiredSize
|
||||||
|
import androidx.compose.foundation.layout.size
|
||||||
|
import androidx.compose.foundation.layout.width
|
||||||
|
import androidx.compose.foundation.rememberScrollState
|
||||||
|
import androidx.compose.foundation.shape.RoundedCornerShape
|
||||||
|
import androidx.compose.foundation.verticalScroll
|
||||||
import androidx.compose.material.icons.Icons
|
import androidx.compose.material.icons.Icons
|
||||||
import androidx.compose.material.icons.filled.Settings
|
import androidx.compose.material.icons.filled.Settings
|
||||||
import androidx.compose.material3.HorizontalDivider
|
|
||||||
import androidx.compose.material3.Icon
|
import androidx.compose.material3.Icon
|
||||||
import androidx.compose.material3.MaterialTheme
|
import androidx.compose.material3.MaterialTheme
|
||||||
import androidx.compose.material3.ModalDrawerSheet
|
import androidx.compose.material3.ModalDrawerSheet
|
||||||
import androidx.compose.material3.NavigationDrawerItem
|
|
||||||
import androidx.compose.material3.Text
|
import androidx.compose.material3.Text
|
||||||
import androidx.compose.runtime.Composable
|
import androidx.compose.runtime.Composable
|
||||||
|
import androidx.compose.ui.Alignment
|
||||||
import androidx.compose.ui.Modifier
|
import androidx.compose.ui.Modifier
|
||||||
|
import androidx.compose.ui.draw.clip
|
||||||
|
import androidx.compose.ui.res.colorResource
|
||||||
|
import androidx.compose.ui.res.painterResource
|
||||||
import androidx.compose.ui.res.stringResource
|
import androidx.compose.ui.res.stringResource
|
||||||
import androidx.compose.ui.unit.dp
|
import androidx.compose.ui.unit.dp
|
||||||
import de.jeanlucmakiola.calendula.R
|
import de.jeanlucmakiola.calendula.R
|
||||||
@@ -23,17 +36,12 @@ import de.jeanlucmakiola.calendula.ui.filter.CalendarFilterList
|
|||||||
/**
|
/**
|
||||||
* Navigation drawer shared by every top-level calendar screen.
|
* Navigation drawer shared by every top-level calendar screen.
|
||||||
*
|
*
|
||||||
* Visual language (kept deliberately small so sizes don't drift):
|
* Uses the app's grouped-card design system (see [GroupedRow]): a branded
|
||||||
* - Drawer title — `titleLarge`
|
* header, the View switcher as a grouped card (the active view highlighted),
|
||||||
* - Section headers (e.g. "Calendars") — `titleSmall`, primary, text only
|
* the per-calendar visibility filter (M3) inline, and a pinned Settings row.
|
||||||
* - Nav items (the views, Today / Settings) — Material `NavigationDrawerItem`
|
* The "View" section mirrors the top-bar switcher pill — tapping a view here
|
||||||
* (`labelLarge` label + a single 24dp leading icon)
|
* selects it (and closes the drawer) rather than cycling. The host screen owns
|
||||||
*
|
* the drawer state.
|
||||||
* The "View" section mirrors the top-bar switcher pill: tapping a view here
|
|
||||||
* selects it (and closes the drawer) rather than cycling. Also hosts the
|
|
||||||
* per-calendar visibility filter (M3) inline — the calendar list with its
|
|
||||||
* checkboxes lives here rather than in a separate sheet — plus a Settings
|
|
||||||
* entry (M4). The host screen owns the drawer state.
|
|
||||||
*/
|
*/
|
||||||
@Composable
|
@Composable
|
||||||
fun CalendarDrawer(
|
fun CalendarDrawer(
|
||||||
@@ -42,43 +50,72 @@ fun CalendarDrawer(
|
|||||||
onSettings: () -> Unit,
|
onSettings: () -> Unit,
|
||||||
) {
|
) {
|
||||||
ModalDrawerSheet {
|
ModalDrawerSheet {
|
||||||
Column(Modifier.fillMaxHeight()) {
|
// The whole sidebar scrolls as one — header, views, the calendar filter
|
||||||
|
// and Settings all flow in a single scroll container.
|
||||||
|
Column(
|
||||||
|
Modifier
|
||||||
|
.fillMaxHeight()
|
||||||
|
.verticalScroll(rememberScrollState()),
|
||||||
|
) {
|
||||||
|
DrawerHeader()
|
||||||
|
|
||||||
|
DrawerSectionHeader(stringResource(R.string.view_section))
|
||||||
|
IMPLEMENTED_VIEWS.forEachIndexed { index, view ->
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(view.labelRes),
|
||||||
|
position = positionOf(index, IMPLEMENTED_VIEWS.size),
|
||||||
|
selected = view == currentView,
|
||||||
|
minHeight = 56.dp,
|
||||||
|
leading = { Icon(view.icon, contentDescription = null) },
|
||||||
|
onClick = { onSelectView(view) },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
Spacer(Modifier.height(16.dp))
|
||||||
|
|
||||||
|
DrawerSectionHeader(stringResource(R.string.filter_title))
|
||||||
|
CalendarFilterList()
|
||||||
|
|
||||||
|
Spacer(Modifier.height(16.dp))
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.month_action_settings),
|
||||||
|
position = Position.Alone,
|
||||||
|
minHeight = 56.dp,
|
||||||
|
leading = { Icon(Icons.Filled.Settings, contentDescription = null) },
|
||||||
|
onClick = onSettings,
|
||||||
|
)
|
||||||
|
Spacer(Modifier.height(8.dp))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Branded header: the app-icon chip beside the app name. */
|
||||||
|
@Composable
|
||||||
|
private fun DrawerHeader() {
|
||||||
|
Row(
|
||||||
|
modifier = Modifier
|
||||||
|
.fillMaxWidth()
|
||||||
|
.padding(start = 28.dp, end = 28.dp, top = 24.dp, bottom = 16.dp),
|
||||||
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
|
) {
|
||||||
|
Box(
|
||||||
|
modifier = Modifier
|
||||||
|
.size(44.dp)
|
||||||
|
.clip(RoundedCornerShape(14.dp))
|
||||||
|
.background(colorResource(R.color.ic_launcher_background)),
|
||||||
|
contentAlignment = Alignment.Center,
|
||||||
|
) {
|
||||||
|
Image(
|
||||||
|
painter = painterResource(R.drawable.ic_launcher_foreground),
|
||||||
|
contentDescription = null,
|
||||||
|
modifier = Modifier.requiredSize(66.dp),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
Spacer(Modifier.width(16.dp))
|
||||||
Text(
|
Text(
|
||||||
text = stringResource(R.string.app_name),
|
text = stringResource(R.string.app_name),
|
||||||
style = MaterialTheme.typography.titleLarge,
|
style = MaterialTheme.typography.titleLarge,
|
||||||
modifier = Modifier.padding(horizontal = 28.dp, vertical = 24.dp),
|
|
||||||
)
|
)
|
||||||
HorizontalDivider()
|
|
||||||
|
|
||||||
DrawerSectionHeader(stringResource(R.string.view_section))
|
|
||||||
IMPLEMENTED_VIEWS.forEach { view ->
|
|
||||||
NavigationDrawerItem(
|
|
||||||
icon = { Icon(view.icon, contentDescription = null) },
|
|
||||||
label = { Text(stringResource(view.labelRes)) },
|
|
||||||
selected = view == currentView,
|
|
||||||
onClick = { onSelectView(view) },
|
|
||||||
modifier = Modifier.padding(horizontal = 12.dp),
|
|
||||||
)
|
|
||||||
}
|
|
||||||
Spacer(Modifier.height(8.dp))
|
|
||||||
HorizontalDivider()
|
|
||||||
|
|
||||||
// Calendars (M3) — visibility checkboxes, scrollable, takes the slack
|
|
||||||
// between the top actions and the pinned Settings entry.
|
|
||||||
DrawerSectionHeader(stringResource(R.string.filter_title))
|
|
||||||
CalendarFilterList(modifier = Modifier.weight(1f))
|
|
||||||
|
|
||||||
HorizontalDivider()
|
|
||||||
Spacer(Modifier.height(8.dp))
|
|
||||||
NavigationDrawerItem(
|
|
||||||
icon = { Icon(Icons.Filled.Settings, contentDescription = null) },
|
|
||||||
label = { Text(stringResource(R.string.month_action_settings)) },
|
|
||||||
selected = false,
|
|
||||||
onClick = onSettings,
|
|
||||||
modifier = Modifier.padding(horizontal = 12.dp),
|
|
||||||
)
|
|
||||||
Spacer(Modifier.height(8.dp))
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -0,0 +1,82 @@
|
|||||||
|
package de.jeanlucmakiola.calendula.ui.common
|
||||||
|
|
||||||
|
import androidx.compose.foundation.background
|
||||||
|
import androidx.compose.foundation.border
|
||||||
|
import androidx.compose.foundation.clickable
|
||||||
|
import androidx.compose.foundation.layout.Arrangement
|
||||||
|
import androidx.compose.foundation.layout.Box
|
||||||
|
import androidx.compose.foundation.layout.ExperimentalLayoutApi
|
||||||
|
import androidx.compose.foundation.layout.FlowRow
|
||||||
|
import androidx.compose.foundation.layout.padding
|
||||||
|
import androidx.compose.foundation.layout.size
|
||||||
|
import androidx.compose.foundation.shape.CircleShape
|
||||||
|
import androidx.compose.material.icons.Icons
|
||||||
|
import androidx.compose.material.icons.filled.Check
|
||||||
|
import androidx.compose.material3.Icon
|
||||||
|
import androidx.compose.material3.MaterialTheme
|
||||||
|
import androidx.compose.runtime.Composable
|
||||||
|
import androidx.compose.ui.Alignment
|
||||||
|
import androidx.compose.ui.Modifier
|
||||||
|
import androidx.compose.ui.draw.clip
|
||||||
|
import androidx.compose.ui.graphics.Color
|
||||||
|
import androidx.compose.ui.unit.dp
|
||||||
|
|
||||||
|
/**
|
||||||
|
* A wrapping row of round colour swatches; the one matching [selected] is
|
||||||
|
* ringed and checked. Shared by the calendar editor and the event-colour
|
||||||
|
* picker so both pick a colour the same way. Swatches render through
|
||||||
|
* [pastelize] — the softened colour the app actually paints, not the raw hue.
|
||||||
|
*/
|
||||||
|
@OptIn(ExperimentalLayoutApi::class)
|
||||||
|
@Composable
|
||||||
|
fun ColorSwatchRow(
|
||||||
|
colors: List<Int>,
|
||||||
|
selected: Int?,
|
||||||
|
onSelect: (Int) -> Unit,
|
||||||
|
dark: Boolean,
|
||||||
|
modifier: Modifier = Modifier,
|
||||||
|
) {
|
||||||
|
FlowRow(modifier = modifier, horizontalArrangement = Arrangement.spacedBy(12.dp)) {
|
||||||
|
colors.forEach { argb ->
|
||||||
|
val isSelected = argb == selected
|
||||||
|
Box(
|
||||||
|
contentAlignment = Alignment.Center,
|
||||||
|
modifier = Modifier
|
||||||
|
.padding(vertical = 4.dp)
|
||||||
|
.size(40.dp)
|
||||||
|
.clip(CircleShape)
|
||||||
|
.background(pastelize(argb, dark))
|
||||||
|
.then(
|
||||||
|
if (isSelected) {
|
||||||
|
Modifier.border(2.dp, MaterialTheme.colorScheme.onSurface, CircleShape)
|
||||||
|
} else {
|
||||||
|
Modifier
|
||||||
|
},
|
||||||
|
)
|
||||||
|
.clickable { onSelect(argb) },
|
||||||
|
) {
|
||||||
|
if (isSelected) {
|
||||||
|
Icon(
|
||||||
|
Icons.Default.Check,
|
||||||
|
contentDescription = null,
|
||||||
|
tint = Color.Black.copy(alpha = 0.7f),
|
||||||
|
modifier = Modifier.size(20.dp),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/** Google-Calendar-style palette; ARGB ints for a raw `CALENDAR_COLOR` / `EVENT_COLOR`. */
|
||||||
|
val CALENDAR_COLOR_PALETTE: List<Int> = listOf(
|
||||||
|
0xFFD50000, // red
|
||||||
|
0xFFE67C00, // orange
|
||||||
|
0xFFF6BF26, // amber
|
||||||
|
0xFF33B679, // green
|
||||||
|
0xFF0B8043, // dark green
|
||||||
|
0xFF039BE5, // blue
|
||||||
|
0xFF3F51B5, // indigo
|
||||||
|
0xFF8E24AA, // purple
|
||||||
|
0xFF616161, // graphite
|
||||||
|
).map { it.toInt() }
|
||||||
@@ -0,0 +1,191 @@
|
|||||||
|
package de.jeanlucmakiola.calendula.ui.common
|
||||||
|
|
||||||
|
import androidx.activity.compose.BackHandler
|
||||||
|
import androidx.compose.animation.core.animateDpAsState
|
||||||
|
import androidx.compose.foundation.background
|
||||||
|
import androidx.compose.foundation.interaction.MutableInteractionSource
|
||||||
|
import androidx.compose.foundation.interaction.collectIsPressedAsState
|
||||||
|
import androidx.compose.foundation.layout.Column
|
||||||
|
import androidx.compose.foundation.layout.ColumnScope
|
||||||
|
import androidx.compose.foundation.layout.RowScope
|
||||||
|
import androidx.compose.foundation.layout.fillMaxSize
|
||||||
|
import androidx.compose.foundation.layout.fillMaxWidth
|
||||||
|
import androidx.compose.foundation.layout.heightIn
|
||||||
|
import androidx.compose.foundation.layout.padding
|
||||||
|
import androidx.compose.foundation.rememberScrollState
|
||||||
|
import androidx.compose.foundation.shape.RoundedCornerShape
|
||||||
|
import androidx.compose.foundation.verticalScroll
|
||||||
|
import androidx.compose.material.icons.Icons
|
||||||
|
import androidx.compose.material.icons.automirrored.filled.ArrowBack
|
||||||
|
import androidx.compose.material3.ExperimentalMaterial3Api
|
||||||
|
import androidx.compose.material3.Icon
|
||||||
|
import androidx.compose.material3.IconButton
|
||||||
|
import androidx.compose.material3.LargeTopAppBar
|
||||||
|
import androidx.compose.material3.ListItem
|
||||||
|
import androidx.compose.material3.ListItemDefaults
|
||||||
|
import androidx.compose.material3.MaterialTheme
|
||||||
|
import androidx.compose.material3.Scaffold
|
||||||
|
import androidx.compose.material3.Surface
|
||||||
|
import androidx.compose.material3.Text
|
||||||
|
import androidx.compose.material3.TopAppBarDefaults
|
||||||
|
import androidx.compose.material3.rememberTopAppBarState
|
||||||
|
import androidx.compose.runtime.Composable
|
||||||
|
import androidx.compose.runtime.getValue
|
||||||
|
import androidx.compose.runtime.remember
|
||||||
|
import androidx.compose.ui.Modifier
|
||||||
|
import androidx.compose.ui.graphics.Color
|
||||||
|
import androidx.compose.ui.input.nestedscroll.nestedScroll
|
||||||
|
import androidx.compose.ui.res.stringResource
|
||||||
|
import androidx.compose.ui.unit.Dp
|
||||||
|
import androidx.compose.ui.unit.dp
|
||||||
|
import de.jeanlucmakiola.calendula.R
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Position of a row within a grouped list, after the Android-15 settings
|
||||||
|
* pattern: a run of rows shares one rounded container, with full corners at the
|
||||||
|
* group's outer edges and small corners between, separated by small gaps.
|
||||||
|
*/
|
||||||
|
enum class Position { Top, Middle, Bottom, Alone }
|
||||||
|
|
||||||
|
/** Maps an index within a group of [count] rows to its [Position]. */
|
||||||
|
fun positionOf(index: Int, count: Int): Position = when {
|
||||||
|
count <= 1 -> Position.Alone
|
||||||
|
index == 0 -> Position.Top
|
||||||
|
index == count - 1 -> Position.Bottom
|
||||||
|
else -> Position.Middle
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The app's standard full-screen list scaffold: a collapsing [LargeTopAppBar]
|
||||||
|
* whose title shrinks into the bar (next to the back button) as the content
|
||||||
|
* scrolls. Content is a scrollable column that feeds the toolbar via nested
|
||||||
|
* scroll. Used by Settings and the calendar manager so they share one shell.
|
||||||
|
*/
|
||||||
|
@OptIn(ExperimentalMaterial3Api::class)
|
||||||
|
@Composable
|
||||||
|
fun CollapsingScaffold(
|
||||||
|
title: String,
|
||||||
|
onBack: () -> Unit,
|
||||||
|
modifier: Modifier = Modifier,
|
||||||
|
snackbarHost: @Composable () -> Unit = {},
|
||||||
|
actions: @Composable RowScope.() -> Unit = {},
|
||||||
|
content: @Composable ColumnScope.() -> Unit,
|
||||||
|
) {
|
||||||
|
BackHandler(onBack = onBack)
|
||||||
|
val scrollBehavior =
|
||||||
|
TopAppBarDefaults.exitUntilCollapsedScrollBehavior(rememberTopAppBarState())
|
||||||
|
Scaffold(
|
||||||
|
modifier = modifier
|
||||||
|
.fillMaxSize()
|
||||||
|
.background(MaterialTheme.colorScheme.surface)
|
||||||
|
.nestedScroll(scrollBehavior.nestedScrollConnection),
|
||||||
|
topBar = {
|
||||||
|
LargeTopAppBar(
|
||||||
|
title = { Text(title) },
|
||||||
|
navigationIcon = {
|
||||||
|
IconButton(onClick = onBack) {
|
||||||
|
Icon(
|
||||||
|
Icons.AutoMirrored.Filled.ArrowBack,
|
||||||
|
contentDescription = stringResource(R.string.settings_back),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
},
|
||||||
|
actions = actions,
|
||||||
|
scrollBehavior = scrollBehavior,
|
||||||
|
colors = TopAppBarDefaults.topAppBarColors(
|
||||||
|
scrolledContainerColor = MaterialTheme.colorScheme.surface,
|
||||||
|
),
|
||||||
|
)
|
||||||
|
},
|
||||||
|
snackbarHost = snackbarHost,
|
||||||
|
) { innerPadding ->
|
||||||
|
Column(
|
||||||
|
modifier = Modifier
|
||||||
|
.padding(innerPadding)
|
||||||
|
.fillMaxSize()
|
||||||
|
.verticalScroll(rememberScrollState())
|
||||||
|
.padding(top = 8.dp, bottom = 24.dp),
|
||||||
|
content = content,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* One row in a grouped list: an M3 [ListItem] over a tonal [Surface] whose
|
||||||
|
* corner radii come from its [position] (so a run of rows reads as a single
|
||||||
|
* rounded card). Corners round further on press. A null [onClick] makes the
|
||||||
|
* row non-interactive (e.g. read-only entries).
|
||||||
|
*/
|
||||||
|
@OptIn(ExperimentalMaterial3Api::class)
|
||||||
|
@Composable
|
||||||
|
fun GroupedRow(
|
||||||
|
title: String,
|
||||||
|
position: Position,
|
||||||
|
modifier: Modifier = Modifier,
|
||||||
|
summary: String? = null,
|
||||||
|
selected: Boolean = false,
|
||||||
|
minHeight: Dp = 72.dp,
|
||||||
|
leading: @Composable (() -> Unit)? = null,
|
||||||
|
trailing: @Composable (() -> Unit)? = null,
|
||||||
|
onClick: (() -> Unit)? = null,
|
||||||
|
) {
|
||||||
|
val interaction = remember { MutableInteractionSource() }
|
||||||
|
val pressed by interaction.collectIsPressedAsState()
|
||||||
|
val full by animateDpAsState(if (pressed) 36.dp else 22.dp, label = "fullCorner")
|
||||||
|
val small by animateDpAsState(if (pressed) 36.dp else 6.dp, label = "smallCorner")
|
||||||
|
val shape = when (position) {
|
||||||
|
Position.Alone -> RoundedCornerShape(full)
|
||||||
|
Position.Top -> RoundedCornerShape(
|
||||||
|
topStart = full, topEnd = full, bottomStart = small, bottomEnd = small,
|
||||||
|
)
|
||||||
|
Position.Middle -> RoundedCornerShape(small)
|
||||||
|
Position.Bottom -> RoundedCornerShape(
|
||||||
|
topStart = small, topEnd = small, bottomStart = full, bottomEnd = full,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
val gap = when (position) {
|
||||||
|
Position.Top, Position.Middle -> Modifier.padding(bottom = 2.dp)
|
||||||
|
Position.Bottom, Position.Alone -> Modifier
|
||||||
|
}
|
||||||
|
val itemColors = if (selected) {
|
||||||
|
ListItemDefaults.colors(
|
||||||
|
containerColor = Color.Transparent,
|
||||||
|
headlineColor = MaterialTheme.colorScheme.onSecondaryContainer,
|
||||||
|
leadingIconColor = MaterialTheme.colorScheme.onSecondaryContainer,
|
||||||
|
supportingColor = MaterialTheme.colorScheme.onSecondaryContainer,
|
||||||
|
trailingIconColor = MaterialTheme.colorScheme.onSecondaryContainer,
|
||||||
|
)
|
||||||
|
} else {
|
||||||
|
ListItemDefaults.colors(containerColor = Color.Transparent)
|
||||||
|
}
|
||||||
|
val item: @Composable () -> Unit = {
|
||||||
|
ListItem(
|
||||||
|
headlineContent = { Text(title) },
|
||||||
|
supportingContent = summary?.let { text -> { Text(text) } },
|
||||||
|
leadingContent = leading,
|
||||||
|
trailingContent = trailing,
|
||||||
|
colors = itemColors,
|
||||||
|
modifier = Modifier.heightIn(min = minHeight),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
val base = modifier
|
||||||
|
.fillMaxWidth()
|
||||||
|
.padding(horizontal = 16.dp)
|
||||||
|
.then(gap)
|
||||||
|
val containerColor = if (selected) {
|
||||||
|
MaterialTheme.colorScheme.secondaryContainer
|
||||||
|
} else {
|
||||||
|
MaterialTheme.colorScheme.surfaceContainerHigh
|
||||||
|
}
|
||||||
|
if (onClick != null) {
|
||||||
|
Surface(
|
||||||
|
onClick = onClick,
|
||||||
|
color = containerColor,
|
||||||
|
shape = shape,
|
||||||
|
interactionSource = interaction,
|
||||||
|
modifier = base,
|
||||||
|
) { item() }
|
||||||
|
} else {
|
||||||
|
Surface(color = containerColor, shape = shape, modifier = base) { item() }
|
||||||
|
}
|
||||||
|
}
|
||||||
@@ -41,6 +41,7 @@ import androidx.compose.material.icons.filled.Close
|
|||||||
import androidx.compose.material.icons.filled.EventAvailable
|
import androidx.compose.material.icons.filled.EventAvailable
|
||||||
import androidx.compose.material.icons.filled.Lock
|
import androidx.compose.material.icons.filled.Lock
|
||||||
import androidx.compose.material.icons.filled.Notifications
|
import androidx.compose.material.icons.filled.Notifications
|
||||||
|
import androidx.compose.material.icons.filled.Palette
|
||||||
import androidx.compose.material.icons.filled.Place
|
import androidx.compose.material.icons.filled.Place
|
||||||
import androidx.compose.material.icons.filled.Public
|
import androidx.compose.material.icons.filled.Public
|
||||||
import androidx.compose.material.icons.filled.Repeat
|
import androidx.compose.material.icons.filled.Repeat
|
||||||
@@ -102,6 +103,7 @@ import de.jeanlucmakiola.calendula.R
|
|||||||
import de.jeanlucmakiola.calendula.domain.AccessLevel
|
import de.jeanlucmakiola.calendula.domain.AccessLevel
|
||||||
import de.jeanlucmakiola.calendula.domain.Availability
|
import de.jeanlucmakiola.calendula.domain.Availability
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
|
import de.jeanlucmakiola.calendula.domain.EventColorOption
|
||||||
import de.jeanlucmakiola.calendula.domain.EventFormField
|
import de.jeanlucmakiola.calendula.domain.EventFormField
|
||||||
import de.jeanlucmakiola.calendula.domain.EventFormProblem
|
import de.jeanlucmakiola.calendula.domain.EventFormProblem
|
||||||
import de.jeanlucmakiola.calendula.domain.RecurrenceEnd
|
import de.jeanlucmakiola.calendula.domain.RecurrenceEnd
|
||||||
@@ -110,6 +112,8 @@ import de.jeanlucmakiola.calendula.domain.RecurringWriteScope
|
|||||||
import de.jeanlucmakiola.calendula.domain.SimpleRecurrence
|
import de.jeanlucmakiola.calendula.domain.SimpleRecurrence
|
||||||
import de.jeanlucmakiola.calendula.domain.parseSimpleRecurrence
|
import de.jeanlucmakiola.calendula.domain.parseSimpleRecurrence
|
||||||
import de.jeanlucmakiola.calendula.domain.toRRule
|
import de.jeanlucmakiola.calendula.domain.toRRule
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.CALENDAR_COLOR_PALETTE
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.ColorSwatchRow
|
||||||
import de.jeanlucmakiola.calendula.ui.common.InlineTextField
|
import de.jeanlucmakiola.calendula.ui.common.InlineTextField
|
||||||
import de.jeanlucmakiola.calendula.ui.common.OptionCard
|
import de.jeanlucmakiola.calendula.ui.common.OptionCard
|
||||||
import de.jeanlucmakiola.calendula.ui.common.currentLocale
|
import de.jeanlucmakiola.calendula.ui.common.currentLocale
|
||||||
@@ -414,6 +418,7 @@ private fun EventEditContent(
|
|||||||
var showReminderPicker by rememberSaveable { mutableStateOf(false) }
|
var showReminderPicker by rememberSaveable { mutableStateOf(false) }
|
||||||
var showRecurrencePicker by rememberSaveable { mutableStateOf(false) }
|
var showRecurrencePicker by rememberSaveable { mutableStateOf(false) }
|
||||||
var showVisibilityPicker by rememberSaveable { mutableStateOf(false) }
|
var showVisibilityPicker by rememberSaveable { mutableStateOf(false) }
|
||||||
|
var showColorPicker by rememberSaveable { mutableStateOf(false) }
|
||||||
var showFieldPicker by rememberSaveable { mutableStateOf(false) }
|
var showFieldPicker by rememberSaveable { mutableStateOf(false) }
|
||||||
|
|
||||||
val selectedCalendar = state.calendars.firstOrNull { it.id == form.calendarId }
|
val selectedCalendar = state.calendars.firstOrNull { it.id == form.calendarId }
|
||||||
@@ -423,6 +428,16 @@ private fun EventEditContent(
|
|||||||
?: MaterialTheme.colorScheme.primary
|
?: MaterialTheme.colorScheme.primary
|
||||||
val gap = 12.dp
|
val gap = 12.dp
|
||||||
|
|
||||||
|
// Per-event colour applicability for the resolved calendar:
|
||||||
|
// - palette calendars (Google, …) and local calendars always support it;
|
||||||
|
// - synced calendars with no palette only when the user opted in, and even
|
||||||
|
// then the colour may not survive the calendar's next sync (the warning).
|
||||||
|
val isLocalCalendar = selectedCalendar?.isLocal == true
|
||||||
|
val colorSupported = state.colorPalette.isNotEmpty() || isLocalCalendar ||
|
||||||
|
state.allowColorOnUnsupportedCalendars
|
||||||
|
val colorSyncRisk = state.colorPalette.isEmpty() && !isLocalCalendar &&
|
||||||
|
state.allowColorOnUnsupportedCalendars
|
||||||
|
|
||||||
Column(
|
Column(
|
||||||
modifier = modifier
|
modifier = modifier
|
||||||
// Shrink the scroll viewport by the keyboard instead of letting
|
// Shrink the scroll viewport by the keyboard instead of letting
|
||||||
@@ -692,6 +707,67 @@ private fun EventEditContent(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
OptionalFormSection(visible = EventFormField.Color in state.visibleFields) {
|
||||||
|
Spacer(Modifier.height(gap))
|
||||||
|
// The swatch the event will paint with: its own colour, else the
|
||||||
|
// calendar's. The Palette icon takes that colour as a preview.
|
||||||
|
val swatch = form.color ?: selectedCalendar?.color
|
||||||
|
EditCard(
|
||||||
|
icon = Icons.Default.Palette,
|
||||||
|
iconContentDescription = stringResource(R.string.event_edit_color),
|
||||||
|
iconTint = if (colorSupported && swatch != null) {
|
||||||
|
pastelize(swatch, dark)
|
||||||
|
} else {
|
||||||
|
MaterialTheme.colorScheme.onSurfaceVariant
|
||||||
|
},
|
||||||
|
onClick = { showColorPicker = true }.takeIf { colorSupported },
|
||||||
|
) {
|
||||||
|
Row(
|
||||||
|
verticalAlignment = Alignment.CenterVertically,
|
||||||
|
modifier = Modifier.fillMaxWidth(),
|
||||||
|
) {
|
||||||
|
Column(modifier = Modifier.weight(1f)) {
|
||||||
|
Text(
|
||||||
|
text = stringResource(
|
||||||
|
when {
|
||||||
|
!colorSupported -> R.string.event_edit_color_unsupported
|
||||||
|
form.color != null -> R.string.event_edit_color_custom
|
||||||
|
else -> R.string.event_edit_color_default
|
||||||
|
},
|
||||||
|
),
|
||||||
|
style = MaterialTheme.typography.titleMedium,
|
||||||
|
)
|
||||||
|
Text(
|
||||||
|
text = stringResource(
|
||||||
|
if (colorSupported) {
|
||||||
|
R.string.event_edit_color
|
||||||
|
} else {
|
||||||
|
R.string.event_edit_color_unsupported_hint
|
||||||
|
},
|
||||||
|
),
|
||||||
|
style = MaterialTheme.typography.bodySmall,
|
||||||
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
)
|
||||||
|
if (colorSyncRisk) {
|
||||||
|
Spacer(Modifier.height(2.dp))
|
||||||
|
Text(
|
||||||
|
text = stringResource(R.string.event_edit_color_sync_warning),
|
||||||
|
style = MaterialTheme.typography.bodySmall,
|
||||||
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if (colorSupported) {
|
||||||
|
Icon(
|
||||||
|
imageVector = Icons.Default.ArrowDropDown,
|
||||||
|
contentDescription = null,
|
||||||
|
tint = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
OptionalFormSection(visible = state.hiddenFields.isNotEmpty()) {
|
OptionalFormSection(visible = state.hiddenFields.isNotEmpty()) {
|
||||||
Spacer(Modifier.height(20.dp))
|
Spacer(Modifier.height(20.dp))
|
||||||
TextButton(
|
TextButton(
|
||||||
@@ -779,6 +855,28 @@ private fun EventEditContent(
|
|||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (showColorPicker) {
|
||||||
|
ColorPickerDialog(
|
||||||
|
palette = state.colorPalette,
|
||||||
|
selected = form.color,
|
||||||
|
hasExplicitColor = form.color != null,
|
||||||
|
syncWarning = colorSyncRisk,
|
||||||
|
onPickKey = { key, argb ->
|
||||||
|
viewModel.setColorKey(key, argb)
|
||||||
|
showColorPicker = false
|
||||||
|
},
|
||||||
|
onPickRaw = { argb ->
|
||||||
|
viewModel.setColorRaw(argb)
|
||||||
|
showColorPicker = false
|
||||||
|
},
|
||||||
|
onClear = {
|
||||||
|
viewModel.clearColor()
|
||||||
|
showColorPicker = false
|
||||||
|
},
|
||||||
|
onDismiss = { showColorPicker = false },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
if (showFieldPicker) {
|
if (showFieldPicker) {
|
||||||
FieldPickerDialog(
|
FieldPickerDialog(
|
||||||
hiddenFields = state.hiddenFields,
|
hiddenFields = state.hiddenFields,
|
||||||
@@ -1294,6 +1392,7 @@ private fun fieldLabel(field: EventFormField): Int = when (field) {
|
|||||||
EventFormField.Recurrence -> R.string.event_detail_recurrence
|
EventFormField.Recurrence -> R.string.event_detail_recurrence
|
||||||
EventFormField.Availability -> R.string.event_edit_availability
|
EventFormField.Availability -> R.string.event_edit_availability
|
||||||
EventFormField.Visibility -> R.string.event_edit_visibility
|
EventFormField.Visibility -> R.string.event_edit_visibility
|
||||||
|
EventFormField.Color -> R.string.event_edit_color
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun fieldIcon(field: EventFormField): ImageVector = when (field) {
|
private fun fieldIcon(field: EventFormField): ImageVector = when (field) {
|
||||||
@@ -1303,6 +1402,7 @@ private fun fieldIcon(field: EventFormField): ImageVector = when (field) {
|
|||||||
EventFormField.Recurrence -> Icons.Default.Repeat
|
EventFormField.Recurrence -> Icons.Default.Repeat
|
||||||
EventFormField.Availability -> Icons.Default.EventAvailable
|
EventFormField.Availability -> Icons.Default.EventAvailable
|
||||||
EventFormField.Visibility -> Icons.Default.Lock
|
EventFormField.Visibility -> Icons.Default.Lock
|
||||||
|
EventFormField.Color -> Icons.Default.Palette
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -1336,6 +1436,72 @@ private fun VisibilityPickerDialog(
|
|||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Event-colour picker: just the swatches. A non-empty [palette] (the calendar
|
||||||
|
* account's published colours) picks by key so the colour round-trips through
|
||||||
|
* sync; otherwise the app's own palette writes a raw colour, with a
|
||||||
|
* [syncWarning] when that calendar may not keep it. The "Reset" button (shown
|
||||||
|
* only once a colour is set) drops back to the calendar's own colour.
|
||||||
|
*/
|
||||||
|
@Composable
|
||||||
|
private fun ColorPickerDialog(
|
||||||
|
palette: List<EventColorOption>,
|
||||||
|
selected: Int?,
|
||||||
|
hasExplicitColor: Boolean,
|
||||||
|
syncWarning: Boolean,
|
||||||
|
onPickKey: (String, Int) -> Unit,
|
||||||
|
onPickRaw: (Int) -> Unit,
|
||||||
|
onClear: () -> Unit,
|
||||||
|
onDismiss: () -> Unit,
|
||||||
|
) {
|
||||||
|
val dark = isSystemInDarkTheme()
|
||||||
|
AlertDialog(
|
||||||
|
onDismissRequest = onDismiss,
|
||||||
|
title = { Text(stringResource(R.string.event_edit_color)) },
|
||||||
|
text = {
|
||||||
|
Column(verticalArrangement = Arrangement.spacedBy(12.dp)) {
|
||||||
|
if (palette.isNotEmpty()) {
|
||||||
|
ColorSwatchRow(
|
||||||
|
colors = palette.map { it.argb },
|
||||||
|
selected = selected,
|
||||||
|
onSelect = { argb ->
|
||||||
|
palette.firstOrNull { it.argb == argb }
|
||||||
|
?.let { onPickKey(it.key, it.argb) }
|
||||||
|
},
|
||||||
|
dark = dark,
|
||||||
|
)
|
||||||
|
} else {
|
||||||
|
ColorSwatchRow(
|
||||||
|
colors = CALENDAR_COLOR_PALETTE,
|
||||||
|
selected = selected,
|
||||||
|
onSelect = onPickRaw,
|
||||||
|
dark = dark,
|
||||||
|
)
|
||||||
|
if (syncWarning) {
|
||||||
|
Text(
|
||||||
|
text = stringResource(R.string.event_edit_color_sync_warning),
|
||||||
|
style = MaterialTheme.typography.bodySmall,
|
||||||
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
confirmButton = {
|
||||||
|
TextButton(onClick = onDismiss) { Text(stringResource(R.string.dialog_cancel)) }
|
||||||
|
},
|
||||||
|
dismissButton = if (hasExplicitColor) {
|
||||||
|
{
|
||||||
|
TextButton(onClick = onClear) {
|
||||||
|
Text(stringResource(R.string.event_edit_color_reset))
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
null
|
||||||
|
},
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
private fun accessLevelIcon(level: AccessLevel): ImageVector = when (level) {
|
private fun accessLevelIcon(level: AccessLevel): ImageVector = when (level) {
|
||||||
AccessLevel.Default -> Icons.Default.Tune
|
AccessLevel.Default -> Icons.Default.Tune
|
||||||
AccessLevel.Public -> Icons.Default.Public
|
AccessLevel.Public -> Icons.Default.Public
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
package de.jeanlucmakiola.calendula.ui.edit
|
package de.jeanlucmakiola.calendula.ui.edit
|
||||||
|
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
|
import de.jeanlucmakiola.calendula.domain.EventColorOption
|
||||||
import de.jeanlucmakiola.calendula.domain.EventForm
|
import de.jeanlucmakiola.calendula.domain.EventForm
|
||||||
import de.jeanlucmakiola.calendula.domain.EventFormField
|
import de.jeanlucmakiola.calendula.domain.EventFormField
|
||||||
import de.jeanlucmakiola.calendula.domain.EventFormProblem
|
import de.jeanlucmakiola.calendula.domain.EventFormProblem
|
||||||
@@ -33,6 +34,18 @@ data class EventEditUiState(
|
|||||||
* then drops "only this event" (an exception row can't carry a rule).
|
* then drops "only this event" (an exception row can't carry a rule).
|
||||||
*/
|
*/
|
||||||
val recurrenceChanged: Boolean = false,
|
val recurrenceChanged: Boolean = false,
|
||||||
|
/**
|
||||||
|
* The event-colour palette the resolved target calendar publishes; empty
|
||||||
|
* when it exposes none. Non-empty → the colour picker offers these swatches
|
||||||
|
* (written as a key, sync-safe); empty → see [colorMode].
|
||||||
|
*/
|
||||||
|
val colorPalette: List<EventColorOption> = emptyList(),
|
||||||
|
/**
|
||||||
|
* Whether the user has opted into custom colours on calendars that publish
|
||||||
|
* no palette (a synced one may then drop the colour on sync). Mirrors the
|
||||||
|
* settings flag; ignored for local and palette-backed calendars.
|
||||||
|
*/
|
||||||
|
val allowColorOnUnsupportedCalendars: Boolean = false,
|
||||||
)
|
)
|
||||||
|
|
||||||
/** One-shot state of a save request, mirroring DeleteUiState on the detail screen. */
|
/** One-shot state of a save request, mirroring DeleteUiState on the detail screen. */
|
||||||
|
|||||||
@@ -12,6 +12,7 @@ import de.jeanlucmakiola.calendula.domain.AccessLevel
|
|||||||
import de.jeanlucmakiola.calendula.domain.Availability
|
import de.jeanlucmakiola.calendula.domain.Availability
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
import de.jeanlucmakiola.calendula.domain.EditSnapshot
|
import de.jeanlucmakiola.calendula.domain.EditSnapshot
|
||||||
|
import de.jeanlucmakiola.calendula.domain.EventColorOption
|
||||||
import de.jeanlucmakiola.calendula.domain.EventForm
|
import de.jeanlucmakiola.calendula.domain.EventForm
|
||||||
import de.jeanlucmakiola.calendula.domain.EventFormField
|
import de.jeanlucmakiola.calendula.domain.EventFormField
|
||||||
import de.jeanlucmakiola.calendula.domain.RecurringWriteScope
|
import de.jeanlucmakiola.calendula.domain.RecurringWriteScope
|
||||||
@@ -19,12 +20,17 @@ import de.jeanlucmakiola.calendula.domain.populatedFields
|
|||||||
import de.jeanlucmakiola.calendula.domain.problems
|
import de.jeanlucmakiola.calendula.domain.problems
|
||||||
import de.jeanlucmakiola.calendula.domain.toEditSnapshot
|
import de.jeanlucmakiola.calendula.domain.toEditSnapshot
|
||||||
import kotlinx.coroutines.CoroutineDispatcher
|
import kotlinx.coroutines.CoroutineDispatcher
|
||||||
|
import kotlinx.coroutines.ExperimentalCoroutinesApi
|
||||||
|
import kotlinx.coroutines.flow.Flow
|
||||||
import kotlinx.coroutines.flow.MutableStateFlow
|
import kotlinx.coroutines.flow.MutableStateFlow
|
||||||
import kotlinx.coroutines.flow.asStateFlow
|
import kotlinx.coroutines.flow.asStateFlow
|
||||||
import kotlinx.coroutines.flow.SharingStarted
|
import kotlinx.coroutines.flow.SharingStarted
|
||||||
import kotlinx.coroutines.flow.StateFlow
|
import kotlinx.coroutines.flow.StateFlow
|
||||||
import kotlinx.coroutines.flow.catch
|
import kotlinx.coroutines.flow.catch
|
||||||
import kotlinx.coroutines.flow.combine
|
import kotlinx.coroutines.flow.combine
|
||||||
|
import kotlinx.coroutines.flow.distinctUntilChanged
|
||||||
|
import kotlinx.coroutines.flow.flatMapLatest
|
||||||
|
import kotlinx.coroutines.flow.flow
|
||||||
import kotlinx.coroutines.flow.flowOn
|
import kotlinx.coroutines.flow.flowOn
|
||||||
import kotlinx.coroutines.flow.map
|
import kotlinx.coroutines.flow.map
|
||||||
import kotlinx.coroutines.flow.stateIn
|
import kotlinx.coroutines.flow.stateIn
|
||||||
@@ -98,19 +104,44 @@ class EventEditViewModel @Inject constructor(
|
|||||||
val writable: List<CalendarSource>,
|
val writable: List<CalendarSource>,
|
||||||
val lastUsed: Long?,
|
val lastUsed: Long?,
|
||||||
val defaultFields: Set<EventFormField>,
|
val defaultFields: Set<EventFormField>,
|
||||||
|
val allowColorOnUnsupported: Boolean,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
/** Writable calendars — the only valid event targets. */
|
||||||
|
private val writableCalendars: Flow<List<CalendarSource>> = repository.calendars()
|
||||||
|
.map { calendars -> calendars.filter { it.canModifyContents } }
|
||||||
|
.catch { emit(emptyList()) }
|
||||||
|
|
||||||
|
/** The target calendar id, resolved exactly as the form shows it. */
|
||||||
|
private val resolvedCalendarId: Flow<Long?> = combine(
|
||||||
|
_form.map { it?.calendarId },
|
||||||
|
writableCalendars,
|
||||||
|
prefs.lastUsedCalendarId,
|
||||||
|
) { picked, writable, lastUsed ->
|
||||||
|
picked
|
||||||
|
?: lastUsed?.takeIf { id -> writable.any { it.id == id } }
|
||||||
|
?: writable.firstOrNull()?.id
|
||||||
|
}.distinctUntilChanged()
|
||||||
|
|
||||||
|
/** The resolved calendar's published event palette, refetched when it changes. */
|
||||||
|
@OptIn(ExperimentalCoroutinesApi::class)
|
||||||
|
private val colorPalette: Flow<List<EventColorOption>> = resolvedCalendarId
|
||||||
|
.flatMapLatest { id ->
|
||||||
|
flow { emit(id?.let { repository.eventColorPalette(it) }.orEmpty()) }
|
||||||
|
}
|
||||||
|
.flowOn(io)
|
||||||
|
|
||||||
val state: StateFlow<EventEditUiState?> = combine(
|
val state: StateFlow<EventEditUiState?> = combine(
|
||||||
combine(_form, _saveState, _showProblems, _revealed, _editTarget, ::LocalInputs),
|
combine(_form, _saveState, _showProblems, _revealed, _editTarget, ::LocalInputs),
|
||||||
combine(
|
combine(
|
||||||
repository.calendars()
|
writableCalendars,
|
||||||
.map { calendars -> calendars.filter { it.canModifyContents } }
|
|
||||||
.catch { emit(emptyList()) },
|
|
||||||
prefs.lastUsedCalendarId,
|
prefs.lastUsedCalendarId,
|
||||||
settingsPrefs.defaultFormFields,
|
settingsPrefs.defaultFormFields,
|
||||||
|
settingsPrefs.allowColorOnUnsupportedCalendars,
|
||||||
::ExternalInputs,
|
::ExternalInputs,
|
||||||
).flowOn(io),
|
).flowOn(io),
|
||||||
) { local, external ->
|
colorPalette,
|
||||||
|
) { local, external, palette ->
|
||||||
val form = local.form ?: return@combine null
|
val form = local.form ?: return@combine null
|
||||||
val resolvedId = form.calendarId
|
val resolvedId = form.calendarId
|
||||||
?: external.lastUsed?.takeIf { id -> external.writable.any { it.id == id } }
|
?: external.lastUsed?.takeIf { id -> external.writable.any { it.id == id } }
|
||||||
@@ -129,6 +160,8 @@ class EventEditViewModel @Inject constructor(
|
|||||||
// the scope dialog drops "only this event" after a rule change.
|
// the scope dialog drops "only this event" after a rule change.
|
||||||
recurrenceChanged = local.editTarget != null &&
|
recurrenceChanged = local.editTarget != null &&
|
||||||
resolved.rrule != local.editTarget.original.rrule,
|
resolved.rrule != local.editTarget.original.rrule,
|
||||||
|
colorPalette = palette,
|
||||||
|
allowColorOnUnsupportedCalendars = external.allowColorOnUnsupported,
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
.stateIn(
|
.stateIn(
|
||||||
@@ -207,10 +240,25 @@ class EventEditViewModel @Inject constructor(
|
|||||||
fun setLocation(value: String) = update { it.copy(location = value) }
|
fun setLocation(value: String) = update { it.copy(location = value) }
|
||||||
fun setDescription(value: String) = update { it.copy(description = value) }
|
fun setDescription(value: String) = update { it.copy(description = value) }
|
||||||
fun setAllDay(value: Boolean) = update { it.copy(isAllDay = value) }
|
fun setAllDay(value: Boolean) = update { it.copy(isAllDay = value) }
|
||||||
fun setCalendar(id: Long) = update { it.copy(calendarId = id) }
|
|
||||||
|
/**
|
||||||
|
* Switching calendars drops any chosen colour: a palette key is
|
||||||
|
* account-scoped, and a raw colour may be invalid on the new calendar.
|
||||||
|
* The event falls back to the new calendar's colour until re-picked.
|
||||||
|
*/
|
||||||
|
fun setCalendar(id: Long) = update { it.copy(calendarId = id, colorKey = null, color = null) }
|
||||||
fun setAvailability(value: Availability) = update { it.copy(availability = value) }
|
fun setAvailability(value: Availability) = update { it.copy(availability = value) }
|
||||||
fun setAccessLevel(value: AccessLevel) = update { it.copy(accessLevel = value) }
|
fun setAccessLevel(value: AccessLevel) = update { it.copy(accessLevel = value) }
|
||||||
|
|
||||||
|
/** Pick a palette colour (round-trips via its key); [argb] is its swatch. */
|
||||||
|
fun setColorKey(key: String, argb: Int) = update { it.copy(colorKey = key, color = argb) }
|
||||||
|
|
||||||
|
/** Pick a raw colour (local / opted-in calendars), clearing any palette key. */
|
||||||
|
fun setColorRaw(argb: Int) = update { it.copy(colorKey = null, color = argb) }
|
||||||
|
|
||||||
|
/** Clear the colour so the event inherits its calendar's. */
|
||||||
|
fun clearColor() = update { it.copy(colorKey = null, color = null) }
|
||||||
|
|
||||||
/** Bare RRULE value from the recurrence picker; null = does not repeat. */
|
/** Bare RRULE value from the recurrence picker; null = does not repeat. */
|
||||||
fun setRecurrence(rrule: String?) = update { it.copy(rrule = rrule) }
|
fun setRecurrence(rrule: String?) = update { it.copy(rrule = rrule) }
|
||||||
|
|
||||||
|
|||||||
@@ -1,25 +1,17 @@
|
|||||||
package de.jeanlucmakiola.calendula.ui.filter
|
package de.jeanlucmakiola.calendula.ui.filter
|
||||||
|
|
||||||
import androidx.compose.foundation.background
|
import androidx.compose.foundation.background
|
||||||
import androidx.compose.foundation.isSystemInDarkTheme
|
|
||||||
import androidx.compose.foundation.layout.Arrangement
|
import androidx.compose.foundation.layout.Arrangement
|
||||||
import androidx.compose.foundation.layout.Box
|
import androidx.compose.foundation.layout.Box
|
||||||
import androidx.compose.foundation.layout.Column
|
import androidx.compose.foundation.layout.Column
|
||||||
import androidx.compose.foundation.layout.PaddingValues
|
|
||||||
import androidx.compose.foundation.layout.Row
|
|
||||||
import androidx.compose.foundation.layout.fillMaxWidth
|
import androidx.compose.foundation.layout.fillMaxWidth
|
||||||
import androidx.compose.foundation.layout.height
|
import androidx.compose.foundation.layout.height
|
||||||
import androidx.compose.foundation.layout.padding
|
import androidx.compose.foundation.layout.padding
|
||||||
import androidx.compose.foundation.layout.size
|
|
||||||
import androidx.compose.foundation.lazy.LazyColumn
|
|
||||||
import androidx.compose.foundation.lazy.items
|
|
||||||
import androidx.compose.foundation.shape.CircleShape
|
|
||||||
import androidx.compose.material3.Checkbox
|
import androidx.compose.material3.Checkbox
|
||||||
import androidx.compose.material3.MaterialTheme
|
import androidx.compose.material3.MaterialTheme
|
||||||
import androidx.compose.material3.Text
|
import androidx.compose.material3.Text
|
||||||
import androidx.compose.runtime.Composable
|
import androidx.compose.runtime.Composable
|
||||||
import androidx.compose.runtime.getValue
|
import androidx.compose.runtime.getValue
|
||||||
import androidx.compose.ui.Alignment
|
|
||||||
import androidx.compose.ui.Modifier
|
import androidx.compose.ui.Modifier
|
||||||
import androidx.compose.ui.res.stringResource
|
import androidx.compose.ui.res.stringResource
|
||||||
import androidx.compose.ui.text.style.TextAlign
|
import androidx.compose.ui.text.style.TextAlign
|
||||||
@@ -28,7 +20,9 @@ import androidx.hilt.navigation.compose.hiltViewModel
|
|||||||
import androidx.lifecycle.compose.collectAsStateWithLifecycle
|
import androidx.lifecycle.compose.collectAsStateWithLifecycle
|
||||||
import de.jeanlucmakiola.calendula.R
|
import de.jeanlucmakiola.calendula.R
|
||||||
import de.jeanlucmakiola.calendula.domain.FailureReason
|
import de.jeanlucmakiola.calendula.domain.FailureReason
|
||||||
import de.jeanlucmakiola.calendula.ui.common.pastelize
|
import de.jeanlucmakiola.calendula.ui.common.CalendarColorChip
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.GroupedRow
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.positionOf
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Calendar-visibility filter (M3), rendered inline in the navigation drawer.
|
* Calendar-visibility filter (M3), rendered inline in the navigation drawer.
|
||||||
@@ -53,67 +47,44 @@ fun CalendarFilterList(
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Renders as a plain (non-scrolling) [Column] so it flows inside the drawer's
|
||||||
|
* single scroll container — the whole sidebar scrolls as one. Calendar counts
|
||||||
|
* are small, so a lazy list isn't needed.
|
||||||
|
*/
|
||||||
@Composable
|
@Composable
|
||||||
private fun FilterList(
|
private fun FilterList(
|
||||||
groups: List<AccountGroup>,
|
groups: List<AccountGroup>,
|
||||||
onSetVisible: (Long, Boolean) -> Unit,
|
onSetVisible: (Long, Boolean) -> Unit,
|
||||||
modifier: Modifier = Modifier,
|
modifier: Modifier = Modifier,
|
||||||
) {
|
) {
|
||||||
val dark = isSystemInDarkTheme()
|
Column(modifier = modifier.fillMaxWidth()) {
|
||||||
LazyColumn(
|
|
||||||
modifier = modifier.fillMaxWidth(),
|
|
||||||
contentPadding = PaddingValues(vertical = 4.dp),
|
|
||||||
) {
|
|
||||||
groups.forEach { group ->
|
groups.forEach { group ->
|
||||||
item(key = "header-${group.account}") {
|
|
||||||
Text(
|
Text(
|
||||||
text = group.account,
|
text = group.account,
|
||||||
style = MaterialTheme.typography.labelMedium,
|
style = MaterialTheme.typography.labelMedium,
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
modifier = Modifier.padding(start = 28.dp, end = 28.dp, top = 12.dp, bottom = 4.dp),
|
modifier = Modifier.padding(start = 28.dp, end = 28.dp, top = 12.dp, bottom = 4.dp),
|
||||||
)
|
)
|
||||||
}
|
group.calendars.forEachIndexed { index, cal ->
|
||||||
items(group.calendars, key = { it.id }) { cal ->
|
GroupedRow(
|
||||||
CalendarToggleRow(
|
title = cal.displayName,
|
||||||
row = cal,
|
position = positionOf(index, group.calendars.size),
|
||||||
dark = dark,
|
minHeight = 56.dp,
|
||||||
|
leading = { CalendarColorChip(cal.color) },
|
||||||
|
trailing = {
|
||||||
|
Checkbox(
|
||||||
|
checked = cal.visible,
|
||||||
onCheckedChange = { onSetVisible(cal.id, it) },
|
onCheckedChange = { onSetVisible(cal.id, it) },
|
||||||
)
|
)
|
||||||
|
},
|
||||||
|
onClick = { onSetVisible(cal.id, !cal.visible) },
|
||||||
|
)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@Composable
|
|
||||||
private fun CalendarToggleRow(
|
|
||||||
row: CalendarRow,
|
|
||||||
dark: Boolean,
|
|
||||||
onCheckedChange: (Boolean) -> Unit,
|
|
||||||
) {
|
|
||||||
Row(
|
|
||||||
modifier = Modifier
|
|
||||||
.fillMaxWidth()
|
|
||||||
.padding(horizontal = 28.dp, vertical = 6.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
horizontalArrangement = Arrangement.spacedBy(16.dp),
|
|
||||||
) {
|
|
||||||
Box(
|
|
||||||
modifier = Modifier
|
|
||||||
.size(14.dp)
|
|
||||||
.background(pastelize(row.color, dark), CircleShape),
|
|
||||||
)
|
|
||||||
Text(
|
|
||||||
text = row.displayName,
|
|
||||||
style = MaterialTheme.typography.bodyMedium,
|
|
||||||
modifier = Modifier.weight(1f),
|
|
||||||
)
|
|
||||||
Checkbox(
|
|
||||||
checked = row.visible,
|
|
||||||
onCheckedChange = onCheckedChange,
|
|
||||||
)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
@Composable
|
@Composable
|
||||||
private fun FilterLoading(modifier: Modifier = Modifier) {
|
private fun FilterLoading(modifier: Modifier = Modifier) {
|
||||||
Column(
|
Column(
|
||||||
|
|||||||
@@ -1,65 +1,97 @@
|
|||||||
package de.jeanlucmakiola.calendula.ui.settings
|
package de.jeanlucmakiola.calendula.ui.settings
|
||||||
|
|
||||||
import android.Manifest
|
import android.Manifest
|
||||||
|
import android.content.Context
|
||||||
import android.content.Intent
|
import android.content.Intent
|
||||||
import android.content.pm.PackageManager
|
import android.content.pm.PackageManager
|
||||||
import android.os.Build
|
import android.os.Build
|
||||||
import androidx.activity.compose.BackHandler
|
|
||||||
import androidx.activity.compose.rememberLauncherForActivityResult
|
import androidx.activity.compose.rememberLauncherForActivityResult
|
||||||
import androidx.activity.result.contract.ActivityResultContracts
|
import androidx.activity.result.contract.ActivityResultContracts
|
||||||
import androidx.core.net.toUri
|
import androidx.compose.animation.AnimatedVisibility
|
||||||
|
import androidx.compose.animation.fadeIn
|
||||||
|
import androidx.compose.animation.fadeOut
|
||||||
|
import androidx.compose.animation.slideInHorizontally
|
||||||
|
import androidx.compose.animation.slideOutHorizontally
|
||||||
|
import androidx.compose.foundation.Image
|
||||||
import androidx.compose.foundation.background
|
import androidx.compose.foundation.background
|
||||||
import androidx.compose.foundation.clickable
|
|
||||||
import androidx.compose.foundation.layout.Arrangement
|
import androidx.compose.foundation.layout.Arrangement
|
||||||
import androidx.compose.foundation.layout.Box
|
import androidx.compose.foundation.layout.Box
|
||||||
import androidx.compose.foundation.layout.Column
|
import androidx.compose.foundation.layout.Column
|
||||||
|
import androidx.compose.foundation.layout.PaddingValues
|
||||||
import androidx.compose.foundation.layout.Row
|
import androidx.compose.foundation.layout.Row
|
||||||
import androidx.compose.foundation.layout.Spacer
|
import androidx.compose.foundation.layout.Spacer
|
||||||
import androidx.compose.foundation.layout.fillMaxSize
|
import androidx.compose.foundation.layout.fillMaxSize
|
||||||
import androidx.compose.foundation.layout.fillMaxWidth
|
import androidx.compose.foundation.layout.fillMaxWidth
|
||||||
import androidx.compose.foundation.layout.height
|
import androidx.compose.foundation.layout.height
|
||||||
import androidx.compose.foundation.layout.padding
|
import androidx.compose.foundation.layout.padding
|
||||||
|
import androidx.compose.foundation.layout.requiredSize
|
||||||
|
import androidx.compose.foundation.layout.size
|
||||||
import androidx.compose.foundation.layout.width
|
import androidx.compose.foundation.layout.width
|
||||||
import androidx.compose.foundation.rememberScrollState
|
import androidx.compose.foundation.shape.CircleShape
|
||||||
import androidx.compose.foundation.verticalScroll
|
import androidx.compose.foundation.shape.RoundedCornerShape
|
||||||
import androidx.compose.material.icons.Icons
|
import androidx.compose.material.icons.Icons
|
||||||
import androidx.compose.material.icons.automirrored.filled.ArrowBack
|
import androidx.compose.material.icons.filled.CalendarMonth
|
||||||
import androidx.compose.material.icons.filled.ArrowDropDown
|
import androidx.compose.material.icons.filled.Gavel
|
||||||
import androidx.compose.material3.DropdownMenu
|
import androidx.compose.material.icons.filled.Language
|
||||||
import androidx.compose.material3.DropdownMenuItem
|
import androidx.compose.material.icons.filled.Notifications
|
||||||
import androidx.compose.material3.ExperimentalMaterial3Api
|
import androidx.compose.material.icons.filled.Palette
|
||||||
import androidx.compose.material3.HorizontalDivider
|
import androidx.compose.material.icons.filled.Tune
|
||||||
|
import androidx.compose.material3.AlertDialog
|
||||||
import androidx.compose.material3.Icon
|
import androidx.compose.material3.Icon
|
||||||
import androidx.compose.material3.IconButton
|
|
||||||
import androidx.compose.material3.MaterialTheme
|
import androidx.compose.material3.MaterialTheme
|
||||||
import androidx.compose.material3.Scaffold
|
import androidx.compose.material3.OutlinedButton
|
||||||
|
import androidx.compose.material3.Surface
|
||||||
import androidx.compose.material3.Switch
|
import androidx.compose.material3.Switch
|
||||||
import androidx.compose.material3.Text
|
import androidx.compose.material3.Text
|
||||||
import androidx.compose.material3.TextButton
|
import androidx.compose.material3.TextButton
|
||||||
import androidx.compose.material3.TopAppBar
|
|
||||||
import androidx.compose.runtime.Composable
|
import androidx.compose.runtime.Composable
|
||||||
import androidx.compose.runtime.getValue
|
import androidx.compose.runtime.getValue
|
||||||
import androidx.compose.runtime.mutableStateOf
|
import androidx.compose.runtime.mutableStateOf
|
||||||
import androidx.compose.runtime.remember
|
import androidx.compose.runtime.remember
|
||||||
|
import androidx.compose.runtime.saveable.rememberSaveable
|
||||||
import androidx.compose.runtime.setValue
|
import androidx.compose.runtime.setValue
|
||||||
import androidx.compose.ui.Alignment
|
import androidx.compose.ui.Alignment
|
||||||
import androidx.compose.ui.Modifier
|
import androidx.compose.ui.Modifier
|
||||||
|
import androidx.compose.ui.draw.clip
|
||||||
|
import androidx.compose.ui.graphics.vector.ImageVector
|
||||||
import androidx.compose.ui.platform.LocalContext
|
import androidx.compose.ui.platform.LocalContext
|
||||||
|
import androidx.compose.ui.res.colorResource
|
||||||
|
import androidx.compose.ui.res.painterResource
|
||||||
import androidx.compose.ui.res.stringResource
|
import androidx.compose.ui.res.stringResource
|
||||||
|
import androidx.compose.ui.text.style.TextAlign
|
||||||
import androidx.compose.ui.unit.dp
|
import androidx.compose.ui.unit.dp
|
||||||
import androidx.core.content.ContextCompat
|
import androidx.core.content.ContextCompat
|
||||||
|
import androidx.core.net.toUri
|
||||||
import androidx.hilt.navigation.compose.hiltViewModel
|
import androidx.hilt.navigation.compose.hiltViewModel
|
||||||
import androidx.lifecycle.compose.collectAsStateWithLifecycle
|
import androidx.lifecycle.compose.collectAsStateWithLifecycle
|
||||||
import de.jeanlucmakiola.calendula.R
|
import de.jeanlucmakiola.calendula.R
|
||||||
import de.jeanlucmakiola.calendula.data.prefs.ThemeMode
|
import de.jeanlucmakiola.calendula.data.prefs.ThemeMode
|
||||||
import de.jeanlucmakiola.calendula.data.prefs.WeekStartPref
|
import de.jeanlucmakiola.calendula.data.prefs.WeekStartPref
|
||||||
import de.jeanlucmakiola.calendula.domain.EventFormField
|
import de.jeanlucmakiola.calendula.domain.EventFormField
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.CollapsingScaffold
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.GroupedRow
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.OptionCard
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.Position
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.positionOf
|
||||||
|
import de.jeanlucmakiola.calendula.ui.common.rememberCalendarSlideSpec
|
||||||
|
|
||||||
|
/** The settings sub-screens reached from the hub's category rows. */
|
||||||
|
private enum class SettingsSection { Appearance, EventForm, Notifications }
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Settings (M4) — appearance (theme, dynamic colour, week start), language,
|
* Token-based accent for a leading icon chip (container / on-container pair).
|
||||||
* and an about section. A full-screen destination; [onBack] pops it.
|
* Neutral chips stay grey; accents are drawn from the M3 scheme so they adapt
|
||||||
|
* to theme, dark mode and dynamic colour.
|
||||||
|
*/
|
||||||
|
private enum class ChipAccent { Neutral, Primary, Tertiary }
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Settings (M4), restructured in v2.3 into a category hub with sub-screens.
|
||||||
|
* Both the hub and the sub-screens use a collapsing [LargeTopAppBar] and the
|
||||||
|
* grouped-row card system. Calendars opens the separate manager hoisted in
|
||||||
|
* [CalendarHost]; Language opens an inline OptionCard dialog; About is a card
|
||||||
|
* at the top. A full-screen destination; [onBack] pops it.
|
||||||
*/
|
*/
|
||||||
@OptIn(ExperimentalMaterial3Api::class)
|
|
||||||
@Composable
|
@Composable
|
||||||
fun SettingsScreen(
|
fun SettingsScreen(
|
||||||
onBack: () -> Unit,
|
onBack: () -> Unit,
|
||||||
@@ -68,205 +100,357 @@ fun SettingsScreen(
|
|||||||
viewModel: SettingsViewModel = hiltViewModel(),
|
viewModel: SettingsViewModel = hiltViewModel(),
|
||||||
) {
|
) {
|
||||||
val state by viewModel.state.collectAsStateWithLifecycle()
|
val state by viewModel.state.collectAsStateWithLifecycle()
|
||||||
|
var section by rememberSaveable { mutableStateOf<SettingsSection?>(null) }
|
||||||
|
val slideSpec = rememberCalendarSlideSpec()
|
||||||
|
|
||||||
// Intercept the system back button/gesture — without this it falls through
|
Box(
|
||||||
// to the activity and closes the app instead of returning to the calendar.
|
|
||||||
BackHandler { onBack() }
|
|
||||||
|
|
||||||
Scaffold(
|
|
||||||
modifier = modifier
|
modifier = modifier
|
||||||
.fillMaxSize()
|
.fillMaxSize()
|
||||||
.background(MaterialTheme.colorScheme.surface),
|
.background(MaterialTheme.colorScheme.surface),
|
||||||
topBar = {
|
|
||||||
TopAppBar(
|
|
||||||
title = { Text(stringResource(R.string.settings_title)) },
|
|
||||||
navigationIcon = {
|
|
||||||
IconButton(onClick = onBack) {
|
|
||||||
Icon(
|
|
||||||
Icons.AutoMirrored.Filled.ArrowBack,
|
|
||||||
contentDescription = stringResource(R.string.settings_back),
|
|
||||||
)
|
|
||||||
}
|
|
||||||
},
|
|
||||||
)
|
|
||||||
},
|
|
||||||
) { innerPadding ->
|
|
||||||
Column(
|
|
||||||
modifier = Modifier
|
|
||||||
.padding(innerPadding)
|
|
||||||
.fillMaxSize()
|
|
||||||
.verticalScroll(rememberScrollState()),
|
|
||||||
) {
|
) {
|
||||||
SectionHeader(stringResource(R.string.settings_section_appearance))
|
SettingsHub(
|
||||||
|
onBack = onBack,
|
||||||
SettingDropdownRow(
|
onOpenSection = { section = it },
|
||||||
title = stringResource(R.string.settings_theme),
|
onManageCalendars = onManageCalendars,
|
||||||
selected = state.themeMode,
|
|
||||||
options = ThemeMode.entries,
|
|
||||||
optionLabel = { themeLabel(it) },
|
|
||||||
onSelect = viewModel::setThemeMode,
|
|
||||||
)
|
|
||||||
DynamicColorRow(
|
|
||||||
checked = state.dynamicColor,
|
|
||||||
enabled = state.dynamicColorAvailable,
|
|
||||||
onCheckedChange = viewModel::setDynamicColor,
|
|
||||||
)
|
|
||||||
SettingDropdownRow(
|
|
||||||
title = stringResource(R.string.settings_week_start),
|
|
||||||
selected = state.weekStart,
|
|
||||||
options = WeekStartPref.entries,
|
|
||||||
optionLabel = { weekStartLabel(it) },
|
|
||||||
onSelect = viewModel::setWeekStart,
|
|
||||||
)
|
)
|
||||||
|
|
||||||
HorizontalDivider(Modifier.padding(vertical = 8.dp))
|
AnimatedVisibility(
|
||||||
SectionHeader(stringResource(R.string.settings_section_event_form))
|
visible = section == SettingsSection.Appearance,
|
||||||
Text(
|
enter = slideInHorizontally(slideSpec) { it } + fadeIn(),
|
||||||
text = stringResource(R.string.settings_form_fields_hint),
|
exit = slideOutHorizontally(slideSpec) { it } + fadeOut(),
|
||||||
style = MaterialTheme.typography.bodySmall,
|
) {
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
AppearanceScreen(state = state, viewModel = viewModel, onBack = { section = null })
|
||||||
modifier = Modifier.padding(horizontal = 24.dp),
|
|
||||||
)
|
|
||||||
EventFormField.entries.forEach { field ->
|
|
||||||
FormFieldRow(
|
|
||||||
title = stringResource(formFieldLabel(field)),
|
|
||||||
checked = field in state.defaultFormFields,
|
|
||||||
onCheckedChange = { viewModel.setFormFieldDefault(field, it) },
|
|
||||||
)
|
|
||||||
}
|
}
|
||||||
|
AnimatedVisibility(
|
||||||
HorizontalDivider(Modifier.padding(vertical = 8.dp))
|
visible = section == SettingsSection.EventForm,
|
||||||
SectionHeader(stringResource(R.string.settings_section_notifications))
|
enter = slideInHorizontally(slideSpec) { it } + fadeIn(),
|
||||||
RemindersRow(
|
exit = slideOutHorizontally(slideSpec) { it } + fadeOut(),
|
||||||
checked = state.remindersEnabled,
|
) {
|
||||||
onCheckedChange = viewModel::setRemindersEnabled,
|
EventFormScreen(state = state, viewModel = viewModel, onBack = { section = null })
|
||||||
)
|
}
|
||||||
|
AnimatedVisibility(
|
||||||
HorizontalDivider(Modifier.padding(vertical = 8.dp))
|
visible = section == SettingsSection.Notifications,
|
||||||
SectionHeader(stringResource(R.string.settings_section_calendars))
|
enter = slideInHorizontally(slideSpec) { it } + fadeIn(),
|
||||||
NavigationRow(
|
exit = slideOutHorizontally(slideSpec) { it } + fadeOut(),
|
||||||
title = stringResource(R.string.settings_manage_calendars),
|
) {
|
||||||
subtitle = stringResource(R.string.settings_manage_calendars_hint),
|
NotificationsScreen(state = state, viewModel = viewModel, onBack = { section = null })
|
||||||
onClick = onManageCalendars,
|
|
||||||
)
|
|
||||||
|
|
||||||
HorizontalDivider(Modifier.padding(vertical = 8.dp))
|
|
||||||
SectionHeader(stringResource(R.string.settings_section_language))
|
|
||||||
LanguageRow()
|
|
||||||
|
|
||||||
HorizontalDivider(Modifier.padding(vertical = 8.dp))
|
|
||||||
SectionHeader(stringResource(R.string.settings_section_about))
|
|
||||||
AboutSection()
|
|
||||||
Spacer(Modifier.height(24.dp))
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// ---------------------------------------------------------------------------
|
||||||
|
// Hub
|
||||||
|
// ---------------------------------------------------------------------------
|
||||||
|
|
||||||
@Composable
|
@Composable
|
||||||
private fun LanguageRow() {
|
private fun SettingsHub(
|
||||||
|
onBack: () -> Unit,
|
||||||
|
onOpenSection: (SettingsSection) -> Unit,
|
||||||
|
onManageCalendars: () -> Unit,
|
||||||
|
) {
|
||||||
|
CollapsingScaffold(title = stringResource(R.string.settings_title), onBack = onBack) {
|
||||||
|
Box(Modifier.padding(horizontal = 16.dp)) { AboutCard() }
|
||||||
|
Spacer(Modifier.height(16.dp))
|
||||||
|
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_section_appearance),
|
||||||
|
summary = stringResource(R.string.settings_appearance_subtitle),
|
||||||
|
position = Position.Top,
|
||||||
|
leading = { CategoryIcon(Icons.Default.Palette, ChipAccent.Neutral) },
|
||||||
|
onClick = { onOpenSection(SettingsSection.Appearance) },
|
||||||
|
)
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_section_event_form),
|
||||||
|
summary = stringResource(R.string.settings_event_form_subtitle),
|
||||||
|
position = Position.Middle,
|
||||||
|
leading = { CategoryIcon(Icons.Default.Tune, ChipAccent.Neutral) },
|
||||||
|
onClick = { onOpenSection(SettingsSection.EventForm) },
|
||||||
|
)
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_section_notifications),
|
||||||
|
summary = stringResource(R.string.settings_notifications_subtitle),
|
||||||
|
position = Position.Middle,
|
||||||
|
leading = { CategoryIcon(Icons.Default.Notifications, ChipAccent.Primary) },
|
||||||
|
onClick = { onOpenSection(SettingsSection.Notifications) },
|
||||||
|
)
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_section_calendars),
|
||||||
|
summary = stringResource(R.string.settings_manage_calendars_hint),
|
||||||
|
position = Position.Middle,
|
||||||
|
leading = { CategoryIcon(Icons.Default.CalendarMonth, ChipAccent.Tertiary) },
|
||||||
|
onClick = onManageCalendars,
|
||||||
|
)
|
||||||
|
LanguageRow(position = Position.Bottom)
|
||||||
|
|
||||||
|
AppVersionText()
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Composable
|
||||||
|
private fun LanguageRow(position: Position) {
|
||||||
// Setting a locale recreates the activity; mirror the choice locally so the
|
// Setting a locale recreates the activity; mirror the choice locally so the
|
||||||
// dropdown updates instantly even before the recreation lands.
|
// row updates instantly even before the recreation lands.
|
||||||
var current by remember { mutableStateOf(AppLanguage.current()) }
|
var current by remember { mutableStateOf(AppLanguage.current()) }
|
||||||
SettingDropdownRow(
|
var showDialog by remember { mutableStateOf(false) }
|
||||||
|
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_language),
|
||||||
|
summary = languageLabel(current),
|
||||||
|
position = position,
|
||||||
|
leading = { CategoryIcon(Icons.Default.Language, ChipAccent.Neutral) },
|
||||||
|
onClick = { showDialog = true },
|
||||||
|
)
|
||||||
|
|
||||||
|
if (showDialog) {
|
||||||
|
OptionPickerDialog(
|
||||||
title = stringResource(R.string.settings_language),
|
title = stringResource(R.string.settings_language),
|
||||||
selected = current,
|
|
||||||
options = LanguagePref.entries,
|
options = LanguagePref.entries,
|
||||||
optionLabel = { languageLabel(it) },
|
selected = current,
|
||||||
|
label = { languageLabel(it) },
|
||||||
onSelect = {
|
onSelect = {
|
||||||
current = it
|
current = it
|
||||||
AppLanguage.apply(it)
|
AppLanguage.apply(it)
|
||||||
},
|
},
|
||||||
|
onDismiss = { showDialog = false },
|
||||||
)
|
)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@Composable
|
@Composable
|
||||||
private fun SectionHeader(text: String) {
|
private fun AboutCard() {
|
||||||
Text(
|
val context = LocalContext.current
|
||||||
text = text,
|
val sourceUrl = stringResource(R.string.about_source_url)
|
||||||
style = MaterialTheme.typography.labelLarge,
|
val licenseUrl = stringResource(R.string.about_license_url)
|
||||||
color = MaterialTheme.colorScheme.primary,
|
|
||||||
modifier = Modifier.padding(start = 24.dp, end = 24.dp, top = 16.dp, bottom = 4.dp),
|
|
||||||
)
|
|
||||||
}
|
|
||||||
|
|
||||||
@Composable
|
Surface(
|
||||||
private fun <T> SettingDropdownRow(
|
color = MaterialTheme.colorScheme.surfaceContainerHigh,
|
||||||
title: String,
|
shape = RoundedCornerShape(24.dp),
|
||||||
selected: T,
|
modifier = Modifier.fillMaxWidth(),
|
||||||
options: List<T>,
|
) {
|
||||||
optionLabel: @Composable (T) -> String,
|
Column(
|
||||||
onSelect: (T) -> Unit,
|
|
||||||
) {
|
|
||||||
var expanded by remember { mutableStateOf(false) }
|
|
||||||
Box {
|
|
||||||
Row(
|
|
||||||
modifier = Modifier
|
modifier = Modifier
|
||||||
.fillMaxWidth()
|
.fillMaxWidth()
|
||||||
.clickable { expanded = true }
|
.padding(16.dp),
|
||||||
.padding(horizontal = 24.dp, vertical = 16.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
) {
|
) {
|
||||||
|
Row(verticalAlignment = Alignment.CenterVertically) {
|
||||||
|
AppLogo()
|
||||||
|
Spacer(Modifier.width(16.dp))
|
||||||
|
Column(Modifier.weight(1f)) {
|
||||||
Text(
|
Text(
|
||||||
text = title,
|
text = stringResource(R.string.app_name),
|
||||||
style = MaterialTheme.typography.bodyLarge,
|
style = MaterialTheme.typography.titleLarge,
|
||||||
modifier = Modifier.weight(1f),
|
|
||||||
)
|
)
|
||||||
Text(
|
Text(
|
||||||
text = optionLabel(selected),
|
text = stringResource(R.string.settings_about_author),
|
||||||
style = MaterialTheme.typography.bodyMedium,
|
style = MaterialTheme.typography.bodyMedium,
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
)
|
)
|
||||||
Icon(
|
|
||||||
Icons.Default.ArrowDropDown,
|
|
||||||
contentDescription = null,
|
|
||||||
tint = MaterialTheme.colorScheme.onSurfaceVariant,
|
|
||||||
)
|
|
||||||
}
|
}
|
||||||
DropdownMenu(expanded = expanded, onDismissRequest = { expanded = false }) {
|
}
|
||||||
options.forEach { option ->
|
Spacer(Modifier.height(12.dp))
|
||||||
DropdownMenuItem(
|
Row(
|
||||||
text = { Text(optionLabel(option)) },
|
modifier = Modifier.fillMaxWidth(),
|
||||||
onClick = {
|
horizontalArrangement = Arrangement.spacedBy(8.dp),
|
||||||
expanded = false
|
) {
|
||||||
onSelect(option)
|
OutlinedButton(
|
||||||
},
|
onClick = { openUrl(context, sourceUrl) },
|
||||||
|
contentPadding = PaddingValues(horizontal = 12.dp),
|
||||||
|
modifier = Modifier.weight(1f),
|
||||||
|
) {
|
||||||
|
Icon(
|
||||||
|
painter = painterResource(R.drawable.ic_gitea),
|
||||||
|
contentDescription = null,
|
||||||
|
modifier = Modifier.size(18.dp),
|
||||||
)
|
)
|
||||||
|
Spacer(Modifier.width(8.dp))
|
||||||
|
Text(stringResource(R.string.settings_about_source))
|
||||||
|
}
|
||||||
|
OutlinedButton(
|
||||||
|
onClick = { openUrl(context, licenseUrl) },
|
||||||
|
contentPadding = PaddingValues(horizontal = 12.dp),
|
||||||
|
modifier = Modifier.weight(1f),
|
||||||
|
) {
|
||||||
|
Icon(
|
||||||
|
Icons.Default.Gavel,
|
||||||
|
contentDescription = null,
|
||||||
|
modifier = Modifier.size(18.dp),
|
||||||
|
)
|
||||||
|
Spacer(Modifier.width(8.dp))
|
||||||
|
Text(stringResource(R.string.settings_license))
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/** Plain centred version mark at the foot of the settings list (no card). */
|
||||||
@Composable
|
@Composable
|
||||||
private fun DynamicColorRow(
|
private fun AppVersionText() {
|
||||||
checked: Boolean,
|
val context = LocalContext.current
|
||||||
enabled: Boolean,
|
val versionName = remember {
|
||||||
onCheckedChange: (Boolean) -> Unit,
|
runCatching {
|
||||||
) {
|
context.packageManager.getPackageInfo(context.packageName, 0).versionName
|
||||||
Row(
|
}.getOrNull() ?: "—"
|
||||||
modifier = Modifier
|
}
|
||||||
.fillMaxWidth()
|
|
||||||
.padding(horizontal = 24.dp, vertical = 12.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
) {
|
|
||||||
Column(Modifier.weight(1f)) {
|
|
||||||
Text(
|
Text(
|
||||||
text = stringResource(R.string.settings_dynamic_color),
|
text = stringResource(R.string.settings_about_version, versionName),
|
||||||
style = MaterialTheme.typography.bodyLarge,
|
|
||||||
color = if (enabled) MaterialTheme.colorScheme.onSurface
|
|
||||||
else MaterialTheme.colorScheme.onSurfaceVariant,
|
|
||||||
)
|
|
||||||
if (!enabled) {
|
|
||||||
Text(
|
|
||||||
text = stringResource(R.string.settings_dynamic_color_unavailable),
|
|
||||||
style = MaterialTheme.typography.bodySmall,
|
style = MaterialTheme.typography.bodySmall,
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
textAlign = TextAlign.Center,
|
||||||
|
modifier = Modifier
|
||||||
|
.fillMaxWidth()
|
||||||
|
.padding(vertical = 16.dp),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* The app icon as a rounded chip: the off-white launcher mark over its slate
|
||||||
|
* background colour, rendered oversized and clipped to fill the chip the way a
|
||||||
|
* launcher mask would.
|
||||||
|
*/
|
||||||
|
@Composable
|
||||||
|
private fun AppLogo() {
|
||||||
|
Box(
|
||||||
|
modifier = Modifier
|
||||||
|
.size(72.dp)
|
||||||
|
.clip(RoundedCornerShape(20.dp))
|
||||||
|
.background(colorResource(R.color.ic_launcher_background)),
|
||||||
|
contentAlignment = Alignment.Center,
|
||||||
|
) {
|
||||||
|
Image(
|
||||||
|
painter = painterResource(R.drawable.ic_launcher_foreground),
|
||||||
|
contentDescription = stringResource(R.string.settings_about_logo_desc),
|
||||||
|
modifier = Modifier.requiredSize(108.dp),
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// ---------------------------------------------------------------------------
|
||||||
|
// Sub-screens
|
||||||
|
// ---------------------------------------------------------------------------
|
||||||
|
|
||||||
|
@Composable
|
||||||
|
private fun AppearanceScreen(
|
||||||
|
state: SettingsUiState,
|
||||||
|
viewModel: SettingsViewModel,
|
||||||
|
onBack: () -> Unit,
|
||||||
|
) {
|
||||||
|
var showTheme by remember { mutableStateOf(false) }
|
||||||
|
var showWeekStart by remember { mutableStateOf(false) }
|
||||||
|
|
||||||
|
CollapsingScaffold(
|
||||||
|
title = stringResource(R.string.settings_section_appearance),
|
||||||
|
onBack = onBack,
|
||||||
|
) {
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_theme),
|
||||||
|
summary = themeLabel(state.themeMode),
|
||||||
|
position = Position.Top,
|
||||||
|
onClick = { showTheme = true },
|
||||||
|
)
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_dynamic_color),
|
||||||
|
summary = if (state.dynamicColorAvailable) {
|
||||||
|
null
|
||||||
|
} else {
|
||||||
|
stringResource(R.string.settings_dynamic_color_unavailable)
|
||||||
|
},
|
||||||
|
position = Position.Middle,
|
||||||
|
trailing = {
|
||||||
|
Switch(
|
||||||
|
checked = state.dynamicColor,
|
||||||
|
onCheckedChange = viewModel::setDynamicColor,
|
||||||
|
enabled = state.dynamicColorAvailable,
|
||||||
|
)
|
||||||
|
},
|
||||||
|
onClick = if (state.dynamicColorAvailable) {
|
||||||
|
{ viewModel.setDynamicColor(!state.dynamicColor) }
|
||||||
|
} else {
|
||||||
|
null
|
||||||
|
},
|
||||||
|
)
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_week_start),
|
||||||
|
summary = weekStartLabel(state.weekStart),
|
||||||
|
position = Position.Bottom,
|
||||||
|
onClick = { showWeekStart = true },
|
||||||
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (showTheme) {
|
||||||
|
OptionPickerDialog(
|
||||||
|
title = stringResource(R.string.settings_theme),
|
||||||
|
options = ThemeMode.entries,
|
||||||
|
selected = state.themeMode,
|
||||||
|
label = { themeLabel(it) },
|
||||||
|
onSelect = viewModel::setThemeMode,
|
||||||
|
onDismiss = { showTheme = false },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
if (showWeekStart) {
|
||||||
|
OptionPickerDialog(
|
||||||
|
title = stringResource(R.string.settings_week_start),
|
||||||
|
options = WeekStartPref.entries,
|
||||||
|
selected = state.weekStart,
|
||||||
|
label = { weekStartLabel(it) },
|
||||||
|
onSelect = viewModel::setWeekStart,
|
||||||
|
onDismiss = { showWeekStart = false },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
@Composable
|
||||||
|
private fun EventFormScreen(
|
||||||
|
state: SettingsUiState,
|
||||||
|
viewModel: SettingsViewModel,
|
||||||
|
onBack: () -> Unit,
|
||||||
|
) {
|
||||||
|
CollapsingScaffold(
|
||||||
|
title = stringResource(R.string.settings_section_event_form),
|
||||||
|
onBack = onBack,
|
||||||
|
) {
|
||||||
|
Text(
|
||||||
|
text = stringResource(R.string.settings_form_fields_hint),
|
||||||
|
style = MaterialTheme.typography.bodyMedium,
|
||||||
|
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
||||||
|
modifier = Modifier.padding(horizontal = 16.dp, vertical = 4.dp),
|
||||||
|
)
|
||||||
|
Spacer(Modifier.height(8.dp))
|
||||||
|
val fields = EventFormField.entries
|
||||||
|
fields.forEachIndexed { index, field ->
|
||||||
|
val checked = field in state.defaultFormFields
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(formFieldLabel(field)),
|
||||||
|
position = positionOf(index, fields.size),
|
||||||
|
trailing = {
|
||||||
Switch(
|
Switch(
|
||||||
checked = checked,
|
checked = checked,
|
||||||
onCheckedChange = onCheckedChange,
|
onCheckedChange = { viewModel.setFormFieldDefault(field, it) },
|
||||||
enabled = enabled,
|
)
|
||||||
|
},
|
||||||
|
onClick = { viewModel.setFormFieldDefault(field, !checked) },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
// Per-event colour on calendars that publish no colour set (some
|
||||||
|
// CalDAV) — off by default, with the honest caveat that the colour may
|
||||||
|
// not survive their next sync. Local and palette calendars ignore it.
|
||||||
|
Spacer(Modifier.height(24.dp))
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_color_unsupported),
|
||||||
|
summary = stringResource(R.string.settings_color_unsupported_hint),
|
||||||
|
position = Position.Alone,
|
||||||
|
trailing = {
|
||||||
|
Switch(
|
||||||
|
checked = state.allowColorOnUnsupportedCalendars,
|
||||||
|
onCheckedChange = { viewModel.setAllowColorOnUnsupportedCalendars(it) },
|
||||||
|
)
|
||||||
|
},
|
||||||
|
onClick = {
|
||||||
|
viewModel.setAllowColorOnUnsupportedCalendars(
|
||||||
|
!state.allowColorOnUnsupportedCalendars,
|
||||||
|
)
|
||||||
|
},
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@@ -277,36 +461,17 @@ private fun DynamicColorRow(
|
|||||||
* the pref is set either way; the OS permission is the real gate.
|
* the pref is set either way; the OS permission is the real gate.
|
||||||
*/
|
*/
|
||||||
@Composable
|
@Composable
|
||||||
private fun RemindersRow(
|
private fun NotificationsScreen(
|
||||||
checked: Boolean,
|
state: SettingsUiState,
|
||||||
onCheckedChange: (Boolean) -> Unit,
|
viewModel: SettingsViewModel,
|
||||||
|
onBack: () -> Unit,
|
||||||
) {
|
) {
|
||||||
val context = LocalContext.current
|
val context = LocalContext.current
|
||||||
val launcher = rememberLauncherForActivityResult(
|
val launcher = rememberLauncherForActivityResult(
|
||||||
contract = ActivityResultContracts.RequestPermission(),
|
contract = ActivityResultContracts.RequestPermission(),
|
||||||
) { /* The pref is already on; a denial just leaves the OS gate shut. */ }
|
) { /* The pref is already on; a denial just leaves the OS gate shut. */ }
|
||||||
Row(
|
val toggleReminders: (Boolean) -> Unit = { enabled ->
|
||||||
modifier = Modifier
|
viewModel.setRemindersEnabled(enabled)
|
||||||
.fillMaxWidth()
|
|
||||||
.padding(horizontal = 24.dp, vertical = 12.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
) {
|
|
||||||
Column(Modifier.weight(1f)) {
|
|
||||||
Text(
|
|
||||||
text = stringResource(R.string.settings_reminders),
|
|
||||||
style = MaterialTheme.typography.bodyLarge,
|
|
||||||
)
|
|
||||||
Text(
|
|
||||||
text = stringResource(R.string.settings_reminders_hint),
|
|
||||||
style = MaterialTheme.typography.bodySmall,
|
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
|
||||||
)
|
|
||||||
}
|
|
||||||
Spacer(Modifier.width(16.dp))
|
|
||||||
Switch(
|
|
||||||
checked = checked,
|
|
||||||
onCheckedChange = { enabled ->
|
|
||||||
onCheckedChange(enabled)
|
|
||||||
val needsPermission = enabled &&
|
val needsPermission = enabled &&
|
||||||
Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU &&
|
Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU &&
|
||||||
ContextCompat.checkSelfPermission(
|
ContextCompat.checkSelfPermission(
|
||||||
@@ -315,116 +480,93 @@ private fun RemindersRow(
|
|||||||
if (needsPermission) {
|
if (needsPermission) {
|
||||||
launcher.launch(Manifest.permission.POST_NOTIFICATIONS)
|
launcher.launch(Manifest.permission.POST_NOTIFICATIONS)
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
CollapsingScaffold(
|
||||||
|
title = stringResource(R.string.settings_section_notifications),
|
||||||
|
onBack = onBack,
|
||||||
|
) {
|
||||||
|
GroupedRow(
|
||||||
|
title = stringResource(R.string.settings_reminders),
|
||||||
|
summary = stringResource(R.string.settings_reminders_hint),
|
||||||
|
position = Position.Alone,
|
||||||
|
trailing = {
|
||||||
|
Switch(checked = state.remindersEnabled, onCheckedChange = toggleReminders)
|
||||||
|
},
|
||||||
|
onClick = { toggleReminders(!state.remindersEnabled) },
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// ---------------------------------------------------------------------------
|
||||||
|
// Shared building blocks
|
||||||
|
// ---------------------------------------------------------------------------
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Leading circular icon chip. Colours come from the M3 scheme via a container /
|
||||||
|
* on-container token pair, so each accent stays correctly paired across theme,
|
||||||
|
* dark mode and dynamic colour.
|
||||||
|
*/
|
||||||
|
@Composable
|
||||||
|
private fun CategoryIcon(icon: ImageVector, accent: ChipAccent) {
|
||||||
|
val scheme = MaterialTheme.colorScheme
|
||||||
|
val (background, iconColor) = when (accent) {
|
||||||
|
ChipAccent.Neutral -> scheme.surfaceContainerHighest to scheme.onSurfaceVariant
|
||||||
|
ChipAccent.Primary -> scheme.primaryContainer to scheme.onPrimaryContainer
|
||||||
|
ChipAccent.Tertiary -> scheme.tertiaryContainer to scheme.onTertiaryContainer
|
||||||
|
}
|
||||||
|
Box(
|
||||||
|
modifier = Modifier
|
||||||
|
.size(40.dp)
|
||||||
|
.clip(CircleShape)
|
||||||
|
.background(background),
|
||||||
|
contentAlignment = Alignment.Center,
|
||||||
|
) {
|
||||||
|
Icon(
|
||||||
|
imageVector = icon,
|
||||||
|
contentDescription = null,
|
||||||
|
tint = iconColor,
|
||||||
|
modifier = Modifier.size(22.dp),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/** OptionCard selection dialog — the app's only sanctioned picker style. */
|
||||||
|
@Composable
|
||||||
|
private fun <T> OptionPickerDialog(
|
||||||
|
title: String,
|
||||||
|
options: List<T>,
|
||||||
|
selected: T,
|
||||||
|
label: @Composable (T) -> String,
|
||||||
|
onSelect: (T) -> Unit,
|
||||||
|
onDismiss: () -> Unit,
|
||||||
|
) {
|
||||||
|
AlertDialog(
|
||||||
|
onDismissRequest = onDismiss,
|
||||||
|
title = { Text(title) },
|
||||||
|
text = {
|
||||||
|
Column(verticalArrangement = Arrangement.spacedBy(8.dp)) {
|
||||||
|
options.forEach { option ->
|
||||||
|
OptionCard(
|
||||||
|
label = label(option),
|
||||||
|
onClick = {
|
||||||
|
onSelect(option)
|
||||||
|
onDismiss()
|
||||||
|
},
|
||||||
|
selected = option == selected,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
confirmButton = {
|
||||||
|
TextButton(onClick = onDismiss) { Text(stringResource(R.string.dialog_cancel)) }
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
@Composable
|
private fun openUrl(context: Context, url: String) {
|
||||||
private fun AboutSection() {
|
val intent = Intent(Intent.ACTION_VIEW, url.toUri())
|
||||||
val context = LocalContext.current
|
|
||||||
val versionName = remember {
|
|
||||||
runCatching {
|
|
||||||
context.packageManager.getPackageInfo(context.packageName, 0).versionName
|
|
||||||
}.getOrNull() ?: "—"
|
|
||||||
}
|
|
||||||
val sourceUrl = stringResource(R.string.about_source_url)
|
|
||||||
|
|
||||||
AboutRow(
|
|
||||||
title = stringResource(R.string.settings_version),
|
|
||||||
value = versionName,
|
|
||||||
)
|
|
||||||
AboutRow(
|
|
||||||
title = stringResource(R.string.settings_license),
|
|
||||||
value = stringResource(R.string.settings_license_value),
|
|
||||||
)
|
|
||||||
Row(
|
|
||||||
modifier = Modifier
|
|
||||||
.fillMaxWidth()
|
|
||||||
.padding(horizontal = 16.dp, vertical = 4.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
) {
|
|
||||||
Column(Modifier.weight(1f).padding(start = 8.dp)) {
|
|
||||||
Text(
|
|
||||||
text = stringResource(R.string.settings_source),
|
|
||||||
style = MaterialTheme.typography.bodyLarge,
|
|
||||||
)
|
|
||||||
Text(
|
|
||||||
text = sourceUrl.removePrefix("https://"),
|
|
||||||
style = MaterialTheme.typography.bodySmall,
|
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
|
||||||
)
|
|
||||||
}
|
|
||||||
TextButton(onClick = {
|
|
||||||
val intent = Intent(Intent.ACTION_VIEW, sourceUrl.toUri())
|
|
||||||
runCatching { context.startActivity(intent) }
|
runCatching { context.startActivity(intent) }
|
||||||
}) {
|
|
||||||
Text(stringResource(R.string.settings_source_open))
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
@Composable
|
|
||||||
private fun AboutRow(title: String, value: String) {
|
|
||||||
Row(
|
|
||||||
modifier = Modifier
|
|
||||||
.fillMaxWidth()
|
|
||||||
.padding(horizontal = 24.dp, vertical = 12.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
) {
|
|
||||||
Text(
|
|
||||||
text = title,
|
|
||||||
style = MaterialTheme.typography.bodyLarge,
|
|
||||||
modifier = Modifier.weight(1f),
|
|
||||||
)
|
|
||||||
Text(
|
|
||||||
text = value,
|
|
||||||
style = MaterialTheme.typography.bodyMedium,
|
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
|
||||||
)
|
|
||||||
Spacer(Modifier.width(8.dp))
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
@Composable
|
|
||||||
private fun NavigationRow(title: String, subtitle: String, onClick: () -> Unit) {
|
|
||||||
Row(
|
|
||||||
modifier = Modifier
|
|
||||||
.fillMaxWidth()
|
|
||||||
.clickable(onClick = onClick)
|
|
||||||
.padding(horizontal = 24.dp, vertical = 12.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
) {
|
|
||||||
Column(Modifier.weight(1f)) {
|
|
||||||
Text(text = title, style = MaterialTheme.typography.bodyLarge)
|
|
||||||
Text(
|
|
||||||
text = subtitle,
|
|
||||||
style = MaterialTheme.typography.bodySmall,
|
|
||||||
color = MaterialTheme.colorScheme.onSurfaceVariant,
|
|
||||||
)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
@Composable
|
|
||||||
private fun FormFieldRow(
|
|
||||||
title: String,
|
|
||||||
checked: Boolean,
|
|
||||||
onCheckedChange: (Boolean) -> Unit,
|
|
||||||
) {
|
|
||||||
Row(
|
|
||||||
modifier = Modifier
|
|
||||||
.fillMaxWidth()
|
|
||||||
.padding(horizontal = 24.dp, vertical = 8.dp),
|
|
||||||
verticalAlignment = Alignment.CenterVertically,
|
|
||||||
) {
|
|
||||||
Text(
|
|
||||||
text = title,
|
|
||||||
style = MaterialTheme.typography.bodyLarge,
|
|
||||||
modifier = Modifier.weight(1f),
|
|
||||||
)
|
|
||||||
Switch(checked = checked, onCheckedChange = onCheckedChange)
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun formFieldLabel(field: EventFormField): Int = when (field) {
|
private fun formFieldLabel(field: EventFormField): Int = when (field) {
|
||||||
@@ -434,6 +576,7 @@ private fun formFieldLabel(field: EventFormField): Int = when (field) {
|
|||||||
EventFormField.Recurrence -> R.string.event_detail_recurrence
|
EventFormField.Recurrence -> R.string.event_detail_recurrence
|
||||||
EventFormField.Availability -> R.string.event_edit_availability
|
EventFormField.Availability -> R.string.event_edit_availability
|
||||||
EventFormField.Visibility -> R.string.event_edit_visibility
|
EventFormField.Visibility -> R.string.event_edit_visibility
|
||||||
|
EventFormField.Color -> R.string.event_edit_color
|
||||||
}
|
}
|
||||||
|
|
||||||
@Composable
|
@Composable
|
||||||
|
|||||||
@@ -20,4 +20,9 @@ data class SettingsUiState(
|
|||||||
val defaultFormFields: Set<EventFormField> = SettingsPrefs.DEFAULT_FORM_FIELDS,
|
val defaultFormFields: Set<EventFormField> = SettingsPrefs.DEFAULT_FORM_FIELDS,
|
||||||
/** Whether Calendula posts reminder notifications (v1.4). */
|
/** Whether Calendula posts reminder notifications (v1.4). */
|
||||||
val remindersEnabled: Boolean = true,
|
val remindersEnabled: Boolean = true,
|
||||||
|
/**
|
||||||
|
* Whether the event-colour picker is offered on calendars that publish no
|
||||||
|
* colour palette (the colour may then not survive their next sync).
|
||||||
|
*/
|
||||||
|
val allowColorOnUnsupportedCalendars: Boolean = false,
|
||||||
)
|
)
|
||||||
|
|||||||
@@ -23,6 +23,9 @@ class SettingsViewModel @Inject constructor(
|
|||||||
private val dynamicColorAvailable = Build.VERSION.SDK_INT >= Build.VERSION_CODES.S
|
private val dynamicColorAvailable = Build.VERSION.SDK_INT >= Build.VERSION_CODES.S
|
||||||
|
|
||||||
val state: StateFlow<SettingsUiState> =
|
val state: StateFlow<SettingsUiState> =
|
||||||
|
combine(
|
||||||
|
// combine() only types up to five flows, so the sixth pref folds
|
||||||
|
// into the assembled state in an outer combine.
|
||||||
combine(
|
combine(
|
||||||
prefs.themeMode,
|
prefs.themeMode,
|
||||||
prefs.dynamicColor,
|
prefs.dynamicColor,
|
||||||
@@ -38,6 +41,10 @@ class SettingsViewModel @Inject constructor(
|
|||||||
defaultFormFields = formFields,
|
defaultFormFields = formFields,
|
||||||
remindersEnabled = reminders,
|
remindersEnabled = reminders,
|
||||||
)
|
)
|
||||||
|
},
|
||||||
|
prefs.allowColorOnUnsupportedCalendars,
|
||||||
|
) { base, allowColor ->
|
||||||
|
base.copy(allowColorOnUnsupportedCalendars = allowColor)
|
||||||
}.stateIn(
|
}.stateIn(
|
||||||
scope = viewModelScope,
|
scope = viewModelScope,
|
||||||
started = SharingStarted.WhileSubscribed(5_000L),
|
started = SharingStarted.WhileSubscribed(5_000L),
|
||||||
@@ -63,4 +70,8 @@ class SettingsViewModel @Inject constructor(
|
|||||||
fun setRemindersEnabled(enabled: Boolean) {
|
fun setRemindersEnabled(enabled: Boolean) {
|
||||||
viewModelScope.launch { prefs.setRemindersEnabled(enabled) }
|
viewModelScope.launch { prefs.setRemindersEnabled(enabled) }
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fun setAllowColorOnUnsupportedCalendars(enabled: Boolean) {
|
||||||
|
viewModelScope.launch { prefs.setAllowColorOnUnsupportedCalendars(enabled) }
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
16
app/src/main/res/drawable/ic_gitea.xml
Normal file
16
app/src/main/res/drawable/ic_gitea.xml
Normal file
@@ -0,0 +1,16 @@
|
|||||||
|
<?xml version="1.0" encoding="utf-8"?>
|
||||||
|
<!--
|
||||||
|
Gitea brand mark, used on the "Source" button in Settings → About.
|
||||||
|
Single-path logo from Simple Icons (https://simpleicons.org, CC0),
|
||||||
|
pathData kept verbatim so Android's PathParser reads the arc flags.
|
||||||
|
fillColor is a placeholder; the Compose Icon recolours it via tint.
|
||||||
|
-->
|
||||||
|
<vector xmlns:android="http://schemas.android.com/apk/res/android"
|
||||||
|
android:width="24dp"
|
||||||
|
android:height="24dp"
|
||||||
|
android:viewportWidth="24"
|
||||||
|
android:viewportHeight="24">
|
||||||
|
<path
|
||||||
|
android:fillColor="#FFFFFFFF"
|
||||||
|
android:pathData="M4.209 4.603c-.247 0-.525.02-.84.088-.333.07-1.28.283-2.054 1.027C-.403 7.25.035 9.685.089 10.052c.065.446.263 1.687 1.21 2.768 1.749 2.141 5.513 2.092 5.513 2.092s.462 1.103 1.168 2.119c.955 1.263 1.936 2.248 2.89 2.367 2.406 0 7.212-.004 7.212-.004s.458.004 1.08-.394c.535-.324 1.013-.893 1.013-.893s.492-.527 1.18-1.73c.21-.37.385-.729.538-1.068 0 0 2.107-4.471 2.107-8.823-.042-1.318-.367-1.55-.443-1.627-.156-.156-.366-.153-.366-.153s-4.475.252-6.792.306c-.508.011-1.012.023-1.512.027v4.474l-.634-.301c0-1.39-.004-4.17-.004-4.17-1.107.016-3.405-.084-3.405-.084s-5.399-.27-5.987-.324c-.187-.011-.401-.032-.648-.032zm.354 1.832h.111s.271 2.269.6 3.597C5.549 11.147 6.22 13 6.22 13s-.996-.119-1.641-.348c-.99-.324-1.409-.714-1.409-.714s-.73-.511-1.096-1.52C1.444 8.73 2.021 7.7 2.021 7.7s.32-.859 1.47-1.145c.395-.106.863-.12 1.072-.12zm8.33 2.554c.26.003.509.127.509.127l.868.422-.529 1.075a.686.686 0 0 0-.614.359.685.685 0 0 0 .072.756l-.939 1.924a.69.69 0 0 0-.66.527.687.687 0 0 0 .347.763.686.686 0 0 0 .867-.206.688.688 0 0 0-.069-.882l.916-1.874a.667.667 0 0 0 .237-.02.657.657 0 0 0 .271-.137 8.826 8.826 0 0 1 1.016.512.761.761 0 0 1 .286.282c.073.21-.073.569-.073.569-.087.29-.702 1.55-.702 1.55a.692.692 0 0 0-.676.477.681.681 0 1 0 1.157-.252c.073-.141.141-.282.214-.431.19-.397.515-1.16.515-1.16.035-.066.218-.394.103-.814-.095-.435-.48-.638-.48-.638-.467-.301-1.116-.58-1.116-.58s0-.156-.042-.27a.688.688 0 0 0-.148-.241l.516-1.062 2.89 1.401s.48.218.583.619c.073.282-.019.534-.069.657-.24.587-2.1 4.317-2.1 4.317s-.232.554-.748.588a1.065 1.065 0 0 1-.393-.045l-.202-.08-4.31-2.1s-.417-.218-.49-.596c-.083-.31.104-.691.104-.691l2.073-4.272s.183-.37.466-.497a.855.855 0 0 1 .35-.077z" />
|
||||||
|
</vector>
|
||||||
@@ -82,6 +82,15 @@
|
|||||||
<string name="event_edit_availability">Verfügbarkeit</string>
|
<string name="event_edit_availability">Verfügbarkeit</string>
|
||||||
<string name="event_edit_visibility">Sichtbarkeit</string>
|
<string name="event_edit_visibility">Sichtbarkeit</string>
|
||||||
|
|
||||||
|
<!-- Termin-Formular — eigene Terminfarbe -->
|
||||||
|
<string name="event_edit_color">Farbe</string>
|
||||||
|
<string name="event_edit_color_default">Kalenderfarbe</string>
|
||||||
|
<string name="event_edit_color_custom">Eigene Farbe</string>
|
||||||
|
<string name="event_edit_color_reset">Zurücksetzen</string>
|
||||||
|
<string name="event_edit_color_unsupported">Für diesen Kalender nicht verfügbar</string>
|
||||||
|
<string name="event_edit_color_unsupported_hint">Dieser Kalender stellt keine Farbpalette bereit. Du kannst eigene Farben für solche Kalender in den Einstellungen aktivieren.</string>
|
||||||
|
<string name="event_edit_color_sync_warning">Dieser Kalender verwirft oder überschreibt die Farbe unter Umständen bei der nächsten Synchronisierung.</string>
|
||||||
|
|
||||||
<!-- Termin-Formular — Speicher-Konflikt (v2.0) -->
|
<!-- Termin-Formular — Speicher-Konflikt (v2.0) -->
|
||||||
<string name="event_edit_conflict_title">Termin wurde extern geändert</string>
|
<string name="event_edit_conflict_title">Termin wurde extern geändert</string>
|
||||||
<string name="event_edit_conflict_body">Während du bearbeitet hast, wurde dieser Termin anderswo geändert — durch Synchronisierung oder eine andere App. Was soll mit deinen Änderungen passieren?</string>
|
<string name="event_edit_conflict_body">Während du bearbeitet hast, wurde dieser Termin anderswo geändert — durch Synchronisierung oder eine andere App. Was soll mit deinen Änderungen passieren?</string>
|
||||||
@@ -208,6 +217,8 @@
|
|||||||
<string name="settings_week_start_sunday">Sonntag</string>
|
<string name="settings_week_start_sunday">Sonntag</string>
|
||||||
<string name="settings_section_event_form">Termin-Formular</string>
|
<string name="settings_section_event_form">Termin-Formular</string>
|
||||||
<string name="settings_form_fields_hint">Standardmäßig angezeigte Felder — alles Weitere liegt hinter \"Weitere Felder\"</string>
|
<string name="settings_form_fields_hint">Standardmäßig angezeigte Felder — alles Weitere liegt hinter \"Weitere Felder\"</string>
|
||||||
|
<string name="settings_color_unsupported">Farben auf nicht unterstützten Kalendern erlauben</string>
|
||||||
|
<string name="settings_color_unsupported_hint">Manche Kalender (z. B. bestimmte CalDAV) stellen keine Farbpalette bereit; eine eigene Terminfarbe wird dort bei der nächsten Synchronisierung unter Umständen verworfen oder überschrieben. Das ist eine Einschränkung dieser Kalender und kann von Calendula nicht behoben werden.</string>
|
||||||
<string name="settings_section_notifications">Benachrichtigungen</string>
|
<string name="settings_section_notifications">Benachrichtigungen</string>
|
||||||
<string name="settings_reminders">Termin-Erinnerungen</string>
|
<string name="settings_reminders">Termin-Erinnerungen</string>
|
||||||
<string name="settings_reminders_hint">Erinnerungen doppelt? Eine andere Kalender-App zeigt sie ebenfalls — schalte sie in einer der beiden ab.</string>
|
<string name="settings_reminders_hint">Erinnerungen doppelt? Eine andere Kalender-App zeigt sie ebenfalls — schalte sie in einer der beiden ab.</string>
|
||||||
@@ -219,12 +230,17 @@
|
|||||||
<string name="settings_language_auto">Systemstandard</string>
|
<string name="settings_language_auto">Systemstandard</string>
|
||||||
<string name="settings_language_german">Deutsch</string>
|
<string name="settings_language_german">Deutsch</string>
|
||||||
<string name="settings_language_english">English</string>
|
<string name="settings_language_english">English</string>
|
||||||
|
<!-- Hub category subtitles -->
|
||||||
|
<string name="settings_appearance_subtitle">Design, dynamische Farben, Wochenstart</string>
|
||||||
|
<string name="settings_event_form_subtitle">Standardfelder für neue Termine</string>
|
||||||
|
<string name="settings_notifications_subtitle">Termin-Erinnerungen</string>
|
||||||
<string name="settings_section_about">Über</string>
|
<string name="settings_section_about">Über</string>
|
||||||
<string name="settings_version">Version</string>
|
|
||||||
<string name="settings_license">Lizenz</string>
|
<string name="settings_license">Lizenz</string>
|
||||||
<string name="settings_license_value">MIT</string>
|
<string name="settings_license_value">MIT</string>
|
||||||
<string name="settings_source">Quellcode</string>
|
<string name="settings_about_author">von Jean-Luc Makiola</string>
|
||||||
<string name="settings_source_open">Öffnen</string>
|
<string name="settings_about_source">Quellcode</string>
|
||||||
|
<string name="settings_about_version">Version %1$s</string>
|
||||||
|
<string name="settings_about_logo_desc">Calendula-App-Symbol</string>
|
||||||
|
|
||||||
<!-- Calendar manager -->
|
<!-- Calendar manager -->
|
||||||
<string name="calendars_title">Kalender</string>
|
<string name="calendars_title">Kalender</string>
|
||||||
|
|||||||
@@ -83,6 +83,15 @@
|
|||||||
<string name="event_edit_availability">Availability</string>
|
<string name="event_edit_availability">Availability</string>
|
||||||
<string name="event_edit_visibility">Visibility</string>
|
<string name="event_edit_visibility">Visibility</string>
|
||||||
|
|
||||||
|
<!-- Event form — per-event color -->
|
||||||
|
<string name="event_edit_color">Color</string>
|
||||||
|
<string name="event_edit_color_default">Calendar color</string>
|
||||||
|
<string name="event_edit_color_custom">Custom color</string>
|
||||||
|
<string name="event_edit_color_reset">Reset</string>
|
||||||
|
<string name="event_edit_color_unsupported">Not available for this calendar</string>
|
||||||
|
<string name="event_edit_color_unsupported_hint">This calendar publishes no color set. You can allow custom colors for such calendars in Settings.</string>
|
||||||
|
<string name="event_edit_color_sync_warning">This calendar may drop or overwrite the color on its next sync.</string>
|
||||||
|
|
||||||
<!-- Event form — save conflict (v2.0) -->
|
<!-- Event form — save conflict (v2.0) -->
|
||||||
<string name="event_edit_conflict_title">Event changed elsewhere</string>
|
<string name="event_edit_conflict_title">Event changed elsewhere</string>
|
||||||
<string name="event_edit_conflict_body">While you were editing, this event was changed — by sync or another app. What should happen to your changes?</string>
|
<string name="event_edit_conflict_body">While you were editing, this event was changed — by sync or another app. What should happen to your changes?</string>
|
||||||
@@ -209,6 +218,8 @@
|
|||||||
<string name="settings_week_start_sunday">Sunday</string>
|
<string name="settings_week_start_sunday">Sunday</string>
|
||||||
<string name="settings_section_event_form">New event form</string>
|
<string name="settings_section_event_form">New event form</string>
|
||||||
<string name="settings_form_fields_hint">Fields shown by default — everything else sits behind \"More fields\"</string>
|
<string name="settings_form_fields_hint">Fields shown by default — everything else sits behind \"More fields\"</string>
|
||||||
|
<string name="settings_color_unsupported">Allow colors on unsupported calendars</string>
|
||||||
|
<string name="settings_color_unsupported_hint">Some calendars (e.g. certain CalDAV) publish no color set; a custom event color may be dropped or overwritten on their next sync. That\'s a limitation of those calendars, not something Calendula can fix.</string>
|
||||||
<string name="settings_section_notifications">Notifications</string>
|
<string name="settings_section_notifications">Notifications</string>
|
||||||
<string name="settings_reminders">Event reminders</string>
|
<string name="settings_reminders">Event reminders</string>
|
||||||
<string name="settings_reminders_hint">Seeing reminders twice? Another calendar app is posting them too — turn them off in one of the two.</string>
|
<string name="settings_reminders_hint">Seeing reminders twice? Another calendar app is posting them too — turn them off in one of the two.</string>
|
||||||
@@ -220,12 +231,17 @@
|
|||||||
<string name="settings_language_auto">System default</string>
|
<string name="settings_language_auto">System default</string>
|
||||||
<string name="settings_language_german">Deutsch</string>
|
<string name="settings_language_german">Deutsch</string>
|
||||||
<string name="settings_language_english">English</string>
|
<string name="settings_language_english">English</string>
|
||||||
|
<!-- Hub category subtitles -->
|
||||||
|
<string name="settings_appearance_subtitle">Theme, dynamic colour, week start</string>
|
||||||
|
<string name="settings_event_form_subtitle">Default fields for new events</string>
|
||||||
|
<string name="settings_notifications_subtitle">Event reminders</string>
|
||||||
<string name="settings_section_about">About</string>
|
<string name="settings_section_about">About</string>
|
||||||
<string name="settings_version">Version</string>
|
|
||||||
<string name="settings_license">License</string>
|
<string name="settings_license">License</string>
|
||||||
<string name="settings_license_value">MIT</string>
|
<string name="settings_license_value">MIT</string>
|
||||||
<string name="settings_source">Source code</string>
|
<string name="settings_about_author">by Jean-Luc Makiola</string>
|
||||||
<string name="settings_source_open">Open</string>
|
<string name="settings_about_source">Source</string>
|
||||||
|
<string name="settings_about_version">Version %1$s</string>
|
||||||
|
<string name="settings_about_logo_desc">Calendula app icon</string>
|
||||||
|
|
||||||
<!-- Calendar manager -->
|
<!-- Calendar manager -->
|
||||||
<string name="calendars_title">Calendars</string>
|
<string name="calendars_title">Calendars</string>
|
||||||
@@ -245,4 +261,5 @@
|
|||||||
<string name="calendars_delete_confirm_message">\"%1$s\" and all of its events will be permanently removed from this device.</string>
|
<string name="calendars_delete_confirm_message">\"%1$s\" and all of its events will be permanently removed from this device.</string>
|
||||||
<string name="calendars_write_error">Couldn\'t save the change.</string>
|
<string name="calendars_write_error">Couldn\'t save the change.</string>
|
||||||
<string name="about_source_url" translatable="false">https://gitea.jeanlucmakiola.de/makiolaj/calendula</string>
|
<string name="about_source_url" translatable="false">https://gitea.jeanlucmakiola.de/makiolaj/calendula</string>
|
||||||
|
<string name="about_license_url" translatable="false">https://gitea.jeanlucmakiola.de/makiolaj/calendula/src/branch/main/LICENSE</string>
|
||||||
</resources>
|
</resources>
|
||||||
|
|||||||
@@ -7,6 +7,7 @@ import app.cash.turbine.test
|
|||||||
import com.google.common.truth.Truth.assertThat
|
import com.google.common.truth.Truth.assertThat
|
||||||
import de.jeanlucmakiola.calendula.data.prefs.CalendarPrefs
|
import de.jeanlucmakiola.calendula.data.prefs.CalendarPrefs
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
|
import de.jeanlucmakiola.calendula.domain.EventColorOption
|
||||||
import de.jeanlucmakiola.calendula.domain.EventForm
|
import de.jeanlucmakiola.calendula.domain.EventForm
|
||||||
import de.jeanlucmakiola.calendula.domain.EventInstance
|
import de.jeanlucmakiola.calendula.domain.EventInstance
|
||||||
import kotlinx.coroutines.Dispatchers
|
import kotlinx.coroutines.Dispatchers
|
||||||
@@ -400,4 +401,20 @@ class CalendarRepositoryImplTest {
|
|||||||
assertThat(expected.message).contains("999")
|
assertThat(expected.message).contains("999")
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `eventColorPalette delegates to the data source for the given calendar`(
|
||||||
|
@TempDir tempDir: Path,
|
||||||
|
) = runTest {
|
||||||
|
val fake = FakeCalendarDataSource().apply {
|
||||||
|
eventColorPaletteResult = { id ->
|
||||||
|
if (id == 7L) listOf(EventColorOption("5", 0xFF33B679.toInt())) else emptyList()
|
||||||
|
}
|
||||||
|
}
|
||||||
|
val repo = CalendarRepositoryImpl(fake, newPrefs(tempDir), Dispatchers.Unconfined)
|
||||||
|
|
||||||
|
assertThat(repo.eventColorPalette(7L))
|
||||||
|
.containsExactly(EventColorOption("5", 0xFF33B679.toInt()))
|
||||||
|
assertThat(repo.eventColorPalette(8L)).isEmpty()
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -20,6 +20,7 @@ class EventDetailMapperTest {
|
|||||||
organizer: String? = "x@y",
|
organizer: String? = "x@y",
|
||||||
rrule: String? = null,
|
rrule: String? = null,
|
||||||
eventColor: Any? = null,
|
eventColor: Any? = null,
|
||||||
|
eventColorKey: String? = null,
|
||||||
calendarColor: Int = 0xFFAABBCC.toInt(),
|
calendarColor: Int = 0xFFAABBCC.toInt(),
|
||||||
dtstart: Long = 1_000_000_000L,
|
dtstart: Long = 1_000_000_000L,
|
||||||
dtend: Long = 1_000_003_600L,
|
dtend: Long = 1_000_003_600L,
|
||||||
@@ -49,6 +50,7 @@ class EventDetailMapperTest {
|
|||||||
EventDetailProjection.IDX_ACCESS_LEVEL to accessLevel,
|
EventDetailProjection.IDX_ACCESS_LEVEL to accessLevel,
|
||||||
EventDetailProjection.IDX_EVENT_TIMEZONE to timezone,
|
EventDetailProjection.IDX_EVENT_TIMEZONE to timezone,
|
||||||
EventDetailProjection.IDX_SELF_ATTENDEE_STATUS to selfStatus,
|
EventDetailProjection.IDX_SELF_ATTENDEE_STATUS to selfStatus,
|
||||||
|
EventDetailProjection.IDX_EVENT_COLOR_KEY to eventColorKey,
|
||||||
)
|
)
|
||||||
|
|
||||||
private fun attendeeReader(
|
private fun attendeeReader(
|
||||||
@@ -99,6 +101,22 @@ class EventDetailMapperTest {
|
|||||||
val detail = detailReader(eventColor = null, calendarColor = 0xFF112233.toInt())
|
val detail = detailReader(eventColor = null, calendarColor = 0xFF112233.toInt())
|
||||||
.toDetail()
|
.toDetail()
|
||||||
assertThat(detail!!.instance.color).isEqualTo(0xFF112233.toInt())
|
assertThat(detail!!.instance.color).isEqualTo(0xFF112233.toInt())
|
||||||
|
// No own colour: the edit form must see this as "inherits".
|
||||||
|
assertThat(detail.eventColor).isNull()
|
||||||
|
assertThat(detail.eventColorKey).isNull()
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `own event color and key are surfaced apart from the resolved color`() {
|
||||||
|
val detail = detailReader(
|
||||||
|
eventColor = 0xFF33B679.toInt(),
|
||||||
|
eventColorKey = "5",
|
||||||
|
calendarColor = 0xFF112233.toInt(),
|
||||||
|
).toDetail()
|
||||||
|
// Resolved display colour is the event's own, not the calendar fallback.
|
||||||
|
assertThat(detail!!.instance.color).isEqualTo(0xFF33B679.toInt())
|
||||||
|
assertThat(detail.eventColor).isEqualTo(0xFF33B679.toInt())
|
||||||
|
assertThat(detail.eventColorKey).isEqualTo("5")
|
||||||
}
|
}
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
|
|||||||
@@ -218,4 +218,83 @@ class EventWriteMapperTest {
|
|||||||
assertThat(values).containsEntry(CalendarContract.Events.EVENT_LOCATION, null)
|
assertThat(values).containsEntry(CalendarContract.Events.EVENT_LOCATION, null)
|
||||||
assertThat(values).containsEntry(CalendarContract.Events.DESCRIPTION, null)
|
assertThat(values).containsEntry(CalendarContract.Events.DESCRIPTION, null)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// --- per-event colour ---
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `palette colour writes only the key, never a raw colour`() {
|
||||||
|
assertThat(eventColorColumns(colorKey = "5", color = 0xFF33B679.toInt()))
|
||||||
|
.containsExactly(CalendarContract.Events.EVENT_COLOR_KEY, "5")
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `raw colour writes the colour and clears any key`() {
|
||||||
|
assertThat(eventColorColumns(colorKey = null, color = 0xFF8E24AA.toInt()))
|
||||||
|
.containsExactly(
|
||||||
|
CalendarContract.Events.EVENT_COLOR_KEY, null,
|
||||||
|
CalendarContract.Events.EVENT_COLOR, 0xFF8E24AA.toInt(),
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `no colour clears both columns so the event inherits its calendar`() {
|
||||||
|
assertThat(eventColorColumns(colorKey = null, color = null))
|
||||||
|
.containsExactly(
|
||||||
|
CalendarContract.Events.EVENT_COLOR_KEY, null,
|
||||||
|
CalendarContract.Events.EVENT_COLOR, null,
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `setting a palette colour on update writes just the key`() {
|
||||||
|
val original = form()
|
||||||
|
val values = update(original, original.copy(colorKey = "3", color = 0xFFF6BF26.toInt()))
|
||||||
|
assertThat(values).containsExactly(CalendarContract.Events.EVENT_COLOR_KEY, "3")
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `setting a raw colour on update writes the colour and a null key`() {
|
||||||
|
val original = form()
|
||||||
|
val values = update(original, original.copy(color = 0xFF039BE5.toInt()))
|
||||||
|
assertThat(values).containsEntry(CalendarContract.Events.EVENT_COLOR, 0xFF039BE5.toInt())
|
||||||
|
assertThat(values).containsEntry(CalendarContract.Events.EVENT_COLOR_KEY, null)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `clearing a colour on update writes explicit nulls`() {
|
||||||
|
val original = form().copy(color = 0xFFD50000.toInt())
|
||||||
|
val values = update(original, original.copy(color = null))
|
||||||
|
assertThat(values).containsEntry(CalendarContract.Events.EVENT_COLOR, null)
|
||||||
|
assertThat(values).containsEntry(CalendarContract.Events.EVENT_COLOR_KEY, null)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `unchanged colour writes no colour columns`() {
|
||||||
|
val original = form().copy(colorKey = "7", color = 0xFF3F51B5.toInt())
|
||||||
|
val values = update(original, original.copy(title = "Renamed"))
|
||||||
|
assertThat(values).doesNotContainKey(CalendarContract.Events.EVENT_COLOR_KEY)
|
||||||
|
assertThat(values).doesNotContainKey(CalendarContract.Events.EVENT_COLOR)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `occurrence exception carries the palette key`() {
|
||||||
|
val values = buildOccurrenceExceptionValues(
|
||||||
|
form = form().copy(colorKey = "2", color = 0xFFE67C00.toInt()),
|
||||||
|
originalInstanceMillis = 0L,
|
||||||
|
zone = berlin,
|
||||||
|
)
|
||||||
|
assertThat(values).containsEntry(CalendarContract.Events.EVENT_COLOR_KEY, "2")
|
||||||
|
assertThat(values).doesNotContainKey(CalendarContract.Events.EVENT_COLOR)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `occurrence exception with no colour clears both columns`() {
|
||||||
|
val values = buildOccurrenceExceptionValues(
|
||||||
|
form = form(),
|
||||||
|
originalInstanceMillis = 0L,
|
||||||
|
zone = berlin,
|
||||||
|
)
|
||||||
|
assertThat(values).containsEntry(CalendarContract.Events.EVENT_COLOR, null)
|
||||||
|
assertThat(values).containsEntry(CalendarContract.Events.EVENT_COLOR_KEY, null)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
package de.jeanlucmakiola.calendula.data.calendar
|
package de.jeanlucmakiola.calendula.data.calendar
|
||||||
|
|
||||||
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
import de.jeanlucmakiola.calendula.domain.CalendarSource
|
||||||
|
import de.jeanlucmakiola.calendula.domain.EventColorOption
|
||||||
import de.jeanlucmakiola.calendula.domain.EventDetail
|
import de.jeanlucmakiola.calendula.domain.EventDetail
|
||||||
import de.jeanlucmakiola.calendula.domain.EventForm
|
import de.jeanlucmakiola.calendula.domain.EventForm
|
||||||
import de.jeanlucmakiola.calendula.domain.EventInstance
|
import de.jeanlucmakiola.calendula.domain.EventInstance
|
||||||
@@ -14,6 +15,7 @@ internal class FakeCalendarDataSource : CalendarDataSource {
|
|||||||
var calendarsResult: List<CalendarSource> = emptyList()
|
var calendarsResult: List<CalendarSource> = emptyList()
|
||||||
var instancesResult: (Long, Long) -> List<EventInstance> = { _, _ -> emptyList() }
|
var instancesResult: (Long, Long) -> List<EventInstance> = { _, _ -> emptyList() }
|
||||||
var eventDetailResult: (Long) -> EventDetail? = { null }
|
var eventDetailResult: (Long) -> EventDetail? = { null }
|
||||||
|
var eventColorPaletteResult: (Long) -> List<EventColorOption> = { emptyList() }
|
||||||
/** Set to make the next write call throw. */
|
/** Set to make the next write call throw. */
|
||||||
var writeError: Exception? = null
|
var writeError: Exception? = null
|
||||||
/** Id returned by the next [insertEvent]. */
|
/** Id returned by the next [insertEvent]. */
|
||||||
@@ -45,6 +47,8 @@ internal class FakeCalendarDataSource : CalendarDataSource {
|
|||||||
override fun instances(beginMillis: Long, endMillis: Long): List<EventInstance> =
|
override fun instances(beginMillis: Long, endMillis: Long): List<EventInstance> =
|
||||||
instancesResult(beginMillis, endMillis)
|
instancesResult(beginMillis, endMillis)
|
||||||
override fun eventDetail(eventId: Long): EventDetail? = eventDetailResult(eventId)
|
override fun eventDetail(eventId: Long): EventDetail? = eventDetailResult(eventId)
|
||||||
|
override fun eventColorPalette(calendarId: Long): List<EventColorOption> =
|
||||||
|
eventColorPaletteResult(calendarId)
|
||||||
|
|
||||||
override fun createLocalCalendar(displayName: String, color: Int, description: String?): Long {
|
override fun createLocalCalendar(displayName: String, color: Int, description: String?): Long {
|
||||||
writeError?.let { throw it }
|
writeError?.let { throw it }
|
||||||
|
|||||||
@@ -115,6 +115,8 @@ class EventFormTest {
|
|||||||
rowStart: Long = 0L,
|
rowStart: Long = 0L,
|
||||||
rowEnd: Long = 0L,
|
rowEnd: Long = 0L,
|
||||||
attendees: List<Attendee> = emptyList(),
|
attendees: List<Attendee> = emptyList(),
|
||||||
|
eventColor: Int? = null,
|
||||||
|
eventColorKey: String? = null,
|
||||||
): EventDetail = EventDetail(
|
): EventDetail = EventDetail(
|
||||||
instance = EventInstance(
|
instance = EventInstance(
|
||||||
instanceId = 1L,
|
instanceId = 1L,
|
||||||
@@ -134,6 +136,8 @@ class EventFormTest {
|
|||||||
reminders = reminders,
|
reminders = reminders,
|
||||||
availability = availability,
|
availability = availability,
|
||||||
accessLevel = accessLevel,
|
accessLevel = accessLevel,
|
||||||
|
eventColor = eventColor,
|
||||||
|
eventColorKey = eventColorKey,
|
||||||
)
|
)
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
@@ -227,6 +231,7 @@ class EventFormTest {
|
|||||||
rrule = "FREQ=DAILY",
|
rrule = "FREQ=DAILY",
|
||||||
availability = Availability.Free,
|
availability = Availability.Free,
|
||||||
accessLevel = AccessLevel.Private,
|
accessLevel = AccessLevel.Private,
|
||||||
|
color = 0xFFD50000.toInt(),
|
||||||
)
|
)
|
||||||
assertThat(full.populatedFields()).containsExactly(
|
assertThat(full.populatedFields()).containsExactly(
|
||||||
EventFormField.Location,
|
EventFormField.Location,
|
||||||
@@ -235,6 +240,33 @@ class EventFormTest {
|
|||||||
EventFormField.Recurrence,
|
EventFormField.Recurrence,
|
||||||
EventFormField.Availability,
|
EventFormField.Availability,
|
||||||
EventFormField.Visibility,
|
EventFormField.Visibility,
|
||||||
|
EventFormField.Color,
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `toEditForm carries a palette colour as key plus swatch`() {
|
||||||
|
val prefilled = detail(eventColor = 0xFF33B679.toInt(), eventColorKey = "5")
|
||||||
|
.toEditForm(beginMillis = 0L, endMillis = 3_600_000L, zone = berlin)
|
||||||
|
assertThat(prefilled.colorKey).isEqualTo("5")
|
||||||
|
assertThat(prefilled.color).isEqualTo(0xFF33B679.toInt())
|
||||||
|
assertThat(prefilled.populatedFields()).contains(EventFormField.Color)
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `toEditForm carries a raw colour with no key`() {
|
||||||
|
val prefilled = detail(eventColor = 0xFF8E24AA.toInt())
|
||||||
|
.toEditForm(beginMillis = 0L, endMillis = 3_600_000L, zone = berlin)
|
||||||
|
assertThat(prefilled.colorKey).isNull()
|
||||||
|
assertThat(prefilled.color).isEqualTo(0xFF8E24AA.toInt())
|
||||||
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun `toEditForm leaves an inheriting event without a colour`() {
|
||||||
|
val prefilled = detail()
|
||||||
|
.toEditForm(beginMillis = 0L, endMillis = 3_600_000L, zone = berlin)
|
||||||
|
assertThat(prefilled.colorKey).isNull()
|
||||||
|
assertThat(prefilled.color).isNull()
|
||||||
|
assertThat(prefilled.populatedFields()).doesNotContain(EventFormField.Color)
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user