OpenLauncherTeam / openlauncher

Customizable and Open Source Launcher for Android
Apache License 2.0
1.41k stars 413 forks source link

locked: Don't show "uninstall" and "info", esp. not for shortcuts #699

Closed chrysn closed 1 year ago

chrysn commented 1 year ago

Closes: https://github.com/OpenLauncherTeam/openlauncher/issues/698


This seems to have been a mixture of accidental implicit fall-through, and copy-pasted code in a little-used branch, and cases (unlocked and locked) getting out of sync. Removing not only "Uninstall" but also "Info" keeps consistency with the 0.7.4 behavior, and keeps the "locked" behavior of providing little avenue for clumsy users to get into areas they don't want to go.

Status: This is a draft PR while I have not tested an of this. (I don't really do Android development, have no SDK for reformatting around, and rely on CI to do all the building for me, as little as I like depending on it). Tested and works.