Skip to content

Feature: Add 'Clear All Downloads' button - Adds Clear All menu item …#2457

Open
PatilParas05 wants to merge 1 commit intoJunkFood02:mainfrom
PatilParas05:feature/clear-all-downloads
Open

Feature: Add 'Clear All Downloads' button - Adds Clear All menu item …#2457
PatilParas05 wants to merge 1 commit intoJunkFood02:mainfrom
PatilParas05:feature/clear-all-downloads

Conversation

@PatilParas05
Copy link
Copy Markdown

@PatilParas05 PatilParas05 commented Mar 3, 2026

Description

Added a "Clear All Downloads" feature that allows users to quickly delete all downloaded videos and audio files at once.

Problem

Users had to manually select and delete items one by one. There was no quick way to clear the entire download history.

Solution

Added a "Clear All Downloads" menu item in the dropdown menu of the video list. When clicked, it shows a confirmation dialog displaying the total number of items to be deleted, preventing accidental deletion.

Changes Made

  • VideoListPage.kt:

    • Added state variable showClearAllDialog to manage dialog visibility
    • Added "Clear All Downloads" menu item in the dropdown menu
    • Implemented confirmation dialog with item count
    • Dialog triggers deleteDownloadHistory with all items
  • VideoListViewModel.kt:

    • Added clearAllDownloads() function

Features

✅ Confirmation dialog prevents accidental deletion
✅ Shows count of items to be deleted
✅ Clean Material Design UI
✅ Uses existing deleteDownloadHistory function for consistency
✅ Only shows button when list is not empty

Type of Change

  • Feature
  • Bug fix
  • Documentation

…in dropdown - Shows confirmation dialog with item count - Deletes all downloads and files - Only shows button when list not empty - Uses existing deleteDownloadHistory function
@rajibvai2026cox-png
Copy link
Copy Markdown

PatilParas05:feature/clear-all-downloads

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.

2 participants