Skip to content

fix(app): system bar leggibili + errore prenotazione non più dietro al calendario#8

Open
fabiodalez-dev wants to merge 1 commit into
mainfrom
fix/app-navbar-and-reserve-error
Open

fix(app): system bar leggibili + errore prenotazione non più dietro al calendario#8
fabiodalez-dev wants to merge 1 commit into
mainfrom
fix/app-navbar-and-reserve-error

Conversation

@fabiodalez-dev

@fabiodalez-dev fabiodalez-dev commented Jun 25, 2026

Copy link
Copy Markdown
Owner

Due fix UI.

1. Barra di navigazione/status illeggibile (frecce/soft-button chiari su sfondo chiaro): con edge-to-edge le icone delle system bar seguivano il dark-mode di sistema, non il tema dell'app. App in light + telefono in dark → icone bianche su sfondo bianco. ora imposta isAppearanceLight{Status,Navigation}Bars dal tema dell'app.

2. Errore prenotazione nascosto sotto al calendario: su prenotazione fallita (es. "hai già una richiesta attiva") il DatePickerDialog restava aperto e lo snackbar finiva dietro. Ora lo sheet si chiude anche sul fallimento, così il messaggio è visibile.

compileDebugKotlin verde.

Summary by CodeRabbit

  • Bug Fixes

    • Le schermate di prenotazione ora si chiudono automaticamente in caso di errore, così il messaggio di avviso risulta subito visibile.
    • I messaggi di errore mostrati durante la prenotazione sono ora più coerenti, incluse le situazioni di conflitto.
  • New Features

    • Le icone della barra di stato e della barra di navigazione si adattano al tema dell’app, migliorando la leggibilità in modalità chiara e scura.

…y the calendar

- System bars: with edge-to-edge the status/navigation bar icons followed the
  SYSTEM dark mode, so a light app under a dark-mode phone showed white soft-
  button/status icons on a light background (invisible). PinakesTheme now sets
  isAppearanceLight{Status,Navigation}Bars from the APP theme.
- Reserve error: on a failed reservation (e.g. "you already have an active
  request") the calendar DatePickerDialog stayed open, so the error snackbar
  rendered behind it. Close the loan sheet on failure too, so the message shows.
@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

No new commits to review since the last review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1341f05a-fe77-4fa7-8a72-83faff362b2e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

Il ramo di errore della prenotazione chiude lo sheet e aggiorna lo snackbar. Il tema applicativo ora sincronizza le icone di status bar e navigation bar con il valore dark durante il rendering edge-to-edge.

Changes

Errore prenotazione

Layer / File(s) Summary
Chiusura sheet su errore
app/src/main/java/com/pinakes/app/ui/screens/detail/BookDetailViewModel.kt
reserve(desiredDate: String) imposta showLoanSheet = false nei rami di errore e aggiorna reserveBusy e lo stato dello snackbar in base a ErrorCodes.CONFLICT e al messaggio ricevuto.

System bar nel tema

Layer / File(s) Summary
Icone system bar allineate al tema
app/src/main/java/com/pinakes/app/ui/theme/Theme.kt
PinakesTheme aggiunge SideEffect e WindowCompat.getInsetsController per impostare isAppearanceLightStatusBars e isAppearanceLightNavigationBars secondo dark quando l'app non è in edit mode.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hop hop, nel buio e nel chiaror,
lo sheet si chiude con dolce fervor.
Le barre brillano nel tema giusto,
e io salto felice: che bel costrutto! 🌙

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Il titolo descrive correttamente entrambe le correzioni principali: system bar leggibili e errore di prenotazione visibile.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/app-navbar-and-reserve-error

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@fabiodalez-dev

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jun 25, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant