Not sure if I can fully grasp the privacy implications of this.
Discussion
It's just putting together already public information inside the Lightning Wallet's transactions list.
Many nostr clients already show this information. Next step is for the Wallets to show it too, imo.
Setting the npub in the wallet will ease getting the public zap events easily.
Showing the (new) private zaps would require setting the nsec, though. That requires more trust on the Wallet, true.