select_titles Improved readability

[CodeName393]

Currently, select_titles is not readable and inconvenient because I can't know the name of the episode when I need to know it.

So, I suggest a code that improves readability as below.

        if select_titles and type(titles)==Series:
            console.print(Padding(Rule(f"[rule.text]\nSelect Titles Option\n"), (1, 2)))
            
            beaupy_titles = [
                f"{i+1}. {t.title} - S{t.season:02}E{t.number:02} - {t.name}"
                for i, t in enumerate(titles)
            ]
            
            selected_idx = select_multiple(
                beaupy_titles,
                preprocessor=lambda val: f"[rgb(205,214,244)]{val}[/rgb(205,214,244)]",
                minimal_count=1,
                page_size=8,
                pagination=True,
                return_indices=True,
                
            )

[vinefeeder]

Agreed. this makes more sense. Thanks for your input.

beaupy_titles = [
f"{i+1}. {t.title} - S{t.season:02}E{t.number:02}"
+ (f" - {t.name}" if t.name else "")
for i, t in enumerate(titles)
]
I've added a get-out to stop printing ' - None' if there are no episode names.

[vinefeeder]

Code updated.

[CodeName393]

Code updated.

Thank you. Would it be hard to reflect the modified color values for readability?

[vinefeeder]

Oh, I see, you want the pink to become grey? No real issue for me. Consider it done. And thanks again for your input.

[CodeName393]

Oh, I see, you want the pink to become grey? No real issue for me. Consider it done. And thanks again for your input.

Thank you. I thought it would be nice to know which items were chosen after the additional selection, so I improved the selected items and the design displayed a little more.

        # modification to enable beaupy module to list titles for download for manual selection 
        # use --select-titles after dl in unshackle command
        Config.transient = True
        if select_titles and type(titles)==Series:
            console.print(Padding(Rule(f"[rule.text]\nSelect Titles\n"), (1, 2)))
            
            beaupy_titles = [
                # thanks CodeName393 for the improvedformatting idea
                f"{i+1}. {t.title} - S{t.season:02}E{t.number:02}"
                + (f" - {t.name}" if t.name else "")
                for i, t in enumerate(titles)
            ]
            
            selected_idx = select_multiple(
                beaupy_titles,
                preprocessor=lambda val: f"[rgb(205,214,244)]{val}[/rgb(205,214,244)]", # console text
                minimal_count=1,
                page_size=8,
                pagination=True,
                return_indices=True,
                cursor_style="rgb(245,194,231)", # rule.text = pink
            )

            # Keep indices unique & ordered
            selected_idx = sorted(set(selected_idx))
            keep = set(selected_idx)

            # In-place filter: delete everything not selected (walk backwards!)
            # need to delete from original list to keep super classes meta data
            for i in range(len(titles) - 1, -1, -1):
                if i not in keep:
                    del titles[i]

            # Show selected titles
            if titles:
                season_map = {}
                for t in titles:
                    if t.season not in season_map:
                        season_map[t.season] = []
                    season_map[t.season].append(str(t.number))
                
                season_display_list = []
                for season in sorted(season_map.keys()):
                    ep_list_str = ",".join(season_map[season])
                    season_display_list.append(f"S{season}({ep_list_str})")
                
                final_output = ", ".join(season_display_list)
                console.print(Padding(f"[text]{final_output}[/]", (0, 5)))

[vinefeeder]

I think not. I consciously avoided that when designing the interface. I thought the main use-cases would be select-all or select just the last episode. The listing clearly shows the selection before accepting. Adding a summary of the selection looks and feels wrong to me.

Sometimes more is too-much. Thanks for your input.