fix(app): system bar leggibili + errore prenotazione non più dietro al calendario#8
fix(app): system bar leggibili + errore prenotazione non più dietro al calendario#8fabiodalez-dev wants to merge 1 commit into
Conversation
…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.
|
Important Review skippedNo new commits to review since the last review. ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughIl 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 ChangesErrore prenotazione
System bar nel tema
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
@coderabbitai review |
✅ Action performedReview finished.
|
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}Barsdal tema dell'app.2. Errore prenotazione nascosto sotto al calendario: su prenotazione fallita (es. "hai già una richiesta attiva") il
DatePickerDialogrestava 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
New Features