fix: When adding a film from the Collection Details modal, the render order was wrong: Radarr Library -> Collection Table -> Add Movie Prompt (missing the Collection details prompt too). Correct order is: Collection Table -> Collection Details Modal -> Add Movie Modal

This commit is contained in:
2025-03-10 15:08:02 -06:00
parent 3afd74dcbf
commit 0834802481
3 changed files with 33 additions and 19 deletions
@@ -33,7 +33,7 @@ impl DrawUi for CollectionDetailsUi {
if let Route::Radarr(active_radarr_block, context_option) = route { if let Route::Radarr(active_radarr_block, context_option) = route {
if let Some(context) = context_option { if let Some(context) = context_option {
return COLLECTION_DETAILS_BLOCKS.contains(&active_radarr_block) return COLLECTION_DETAILS_BLOCKS.contains(&active_radarr_block)
&& context == ActiveRadarrBlock::CollectionDetails; || context == ActiveRadarrBlock::CollectionDetails;
} }
return COLLECTION_DETAILS_BLOCKS.contains(&active_radarr_block); return COLLECTION_DETAILS_BLOCKS.contains(&active_radarr_block);
@@ -25,5 +25,12 @@ mod tests {
) )
.into() .into()
)); ));
assert!(CollectionDetailsUi::accepts(
(
ActiveRadarrBlock::AddMoviePrompt,
Some(ActiveRadarrBlock::CollectionDetails)
)
.into()
));
} }
} }
+25 -18
View File
@@ -44,25 +44,32 @@ impl DrawUi for LibraryUi {
fn draw(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) { fn draw(f: &mut Frame<'_>, app: &mut App<'_>, area: Rect) {
let route = app.get_current_route(); let route = app.get_current_route();
draw_library(f, app, area); if let Route::Radarr(_, context_option) = route {
if context_option.is_some() && AddMovieUi::accepts(route) {
match route { AddMovieUi::draw(f, app, area);
_ if MovieDetailsUi::accepts(route) => MovieDetailsUi::draw(f, app, area), return;
_ if AddMovieUi::accepts(route) => AddMovieUi::draw(f, app, area), }
_ if EditMovieUi::accepts(route) => EditMovieUi::draw(f, app, area),
_ if DeleteMovieUi::accepts(route) => DeleteMovieUi::draw(f, app, area), draw_library(f, app, area);
Route::Radarr(ActiveRadarrBlock::UpdateAllMoviesPrompt, _) => {
let confirmation_prompt = ConfirmationPrompt::new() match route {
.title("Update All Movies") _ if MovieDetailsUi::accepts(route) => MovieDetailsUi::draw(f, app, area),
.prompt("Do you want to update info and scan your disks for all of your movies?") _ if AddMovieUi::accepts(route) => AddMovieUi::draw(f, app, area),
.yes_no_value(app.data.radarr_data.prompt_confirm); _ if EditMovieUi::accepts(route) => EditMovieUi::draw(f, app, area),
_ if DeleteMovieUi::accepts(route) => DeleteMovieUi::draw(f, app, area),
f.render_widget( Route::Radarr(ActiveRadarrBlock::UpdateAllMoviesPrompt, _) => {
Popup::new(confirmation_prompt).size(Size::MediumPrompt), let confirmation_prompt = ConfirmationPrompt::new()
f.area(), .title("Update All Movies")
); .prompt("Do you want to update info and scan your disks for all of your movies?")
.yes_no_value(app.data.radarr_data.prompt_confirm);
f.render_widget(
Popup::new(confirmation_prompt).size(Size::MediumPrompt),
f.area(),
);
}
_ => (),
} }
_ => (),
} }
} }
} }