Skip to content

Add button to delete all WireGuard configs#2596

Open
niranjankartha wants to merge 2 commits intocelzero:mainfrom
niranjankartha:delete-all-wg-profiles
Open

Add button to delete all WireGuard configs#2596
niranjankartha wants to merge 2 commits intocelzero:mainfrom
niranjankartha:delete-all-wg-profiles

Conversation

@niranjankartha
Copy link
Copy Markdown

@niranjankartha niranjankartha commented Mar 15, 2026

Add a button to delete all the added WireGuard config options at once. Show a dialog box before deleting which indicates that it's irreversible (unless they still have access to the wireguard files).

This button shows up along side the "Add via QR / File / Create" buttons, so its position can be changed if necessary.

This closes #2595

@niranjankartha niranjankartha force-pushed the delete-all-wg-profiles branch from 0b799d9 to 702705e Compare March 15, 2026 16:45
@niranjankartha niranjankartha changed the title Add button to delete all WireGuard configs (#2595) Add button to delete all WireGuard configs Mar 15, 2026
@ignoramous
Copy link
Copy Markdown
Collaborator

ignoramous commented Mar 15, 2026

If you can, will you pls post a screengrab / video of how this works?

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a “Delete All” action to the WireGuard configuration screen to help users quickly remove large imported config sets, with a confirmation dialog warning that the action is irreversible.

Changes:

  • Added new “Delete All” confirmation dialog strings (including Malayalam localization entries).
  • Added a new Extended FAB to the WireGuard main layout and expanded/collapsed FAB animation offsets.
  • Implemented delete-all flow in WgMainActivity to delete all WireGuard configs after confirmation.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
app/src/main/res/values/strings.xml Adds dialog title/description strings for “Delete All”.
app/src/main/res/values/dimens.xml Adds a new dimension used for FAB expansion spacing.
app/src/main/res/values-ml/strings.xml Adds Malayalam localized strings for the new dialog.
app/src/main/res/layout/activity_wireguard_main.xml Adds a new “delete all” Extended FAB alongside existing action FABs.
app/src/full/java/com/celzero/bravedns/ui/activity/WgMainActivity.kt Wires the new FAB to a confirmation dialog and performs delete-all logic + FAB expand/collapse updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/src/full/java/com/celzero/bravedns/ui/activity/WgMainActivity.kt Outdated
Comment thread app/src/full/java/com/celzero/bravedns/ui/activity/WgMainActivity.kt Outdated
Comment thread app/src/main/res/layout/activity_wireguard_main.xml Outdated
Comment thread app/src/main/res/values-ml/strings.xml Outdated
Add a new button in the toggle buttons to delete all the added WireGuard
config options at once. Show a dialog box before deleting which
indicates that it's irreversible (unless they still have access to the
wireguard files).
Add Malayalam translation for the dialog.
@niranjankartha niranjankartha force-pushed the delete-all-wg-profiles branch from 702705e to 704d47b Compare March 15, 2026 18:33
@niranjankartha
Copy link
Copy Markdown
Author

niranjankartha commented Mar 15, 2026

Here you go, @ignoramous:

rethink-2596-screencap-compressed.mp4

There's also a toast saying "Done" that shows up after all the deletes complete but my screen recording didn't capture it.

I put the "DELETE ALL" button over there because I couldn't think of any other place to put it. Accepting suggestions on where y'all think would be appropriate.

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.

Add function to delete all WireGuard configs at once

4 participants