Add button to delete all WireGuard configs#2596
Add button to delete all WireGuard configs#2596niranjankartha wants to merge 2 commits intocelzero:mainfrom
Conversation
0b799d9 to
702705e
Compare
|
If you can, will you pls post a screengrab / video of how this works? |
There was a problem hiding this comment.
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
WgMainActivityto 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.
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.
702705e to
704d47b
Compare
|
Here you go, @ignoramous: rethink-2596-screencap-compressed.mp4There'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. |
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