Never mind, you did change it to notification-inactive in the other patch. So this is good to go IMO.
As for landing it, we have two options to make sure that rolling release distro users don't see their notification icon change to the one with the red line through it:
- Land this patch for 5.63 (the Frameworks release aligned with Plasma 5.17)
- Change the inactive applet to use the notification-inactive icon in another patch and land that on the stable branch, then land this