ReVanced / revanced-manager

💊 Application to use ReVanced on Android
https://revanced.app
GNU General Public License v3.0
17.9k stars 736 forks source link

feat: Show added patches next to deleted patches #2212

Open oSumAtrIX opened 1 month ago

oSumAtrIX commented 1 month ago

Feature description

Show patches that were added in the dialog where it shows that patches have been deleted.

Motivation

The issue is that it does not show that patches were added at the same time. The issue is that Manager can not distinguish between deleted and renamed patches, so a rename is a delete & add at the same time, whereas the old name was deleted and the new name was added.

Acknowledgements

TheAabedKhan commented 1 month ago

Show all the new patches or only the new patches that the user has selected?

LisoUseInAIKyrios commented 2 weeks ago

The dialog should show the patches that were removed, but also show all the new patches added (regardless if the user included the new patches or not).

So the warning dialog would not be a warning but instead a "hey some patches were removed, but also some patches were also added".

It will solve the issue of patches being renamed and users losing their mind thinking the patch is no more.

LisoUseInAIKyrios commented 2 weeks ago

Just to clarify, the dialog shows for the situations of:

  1. Patches were removed, and no new patches are available since the last time the app was patched (no major code changes needed, and the existing 'warning' dialog should show).

  2. Patches were removed, and at least 1 new patch is available since the last time the app was patched (Show a non warning dialog, informing both patches were removed but new patches are now available).

The dialog should not show if new patches are available and no existing patches were removed.