This removes all warnings. Main takeaway is that we should be using useCallback hooks more often, moving forward we have to be careful to not introduce new warnings. Warnings removed spanned the following files: preview.js, popover.js, inline.js, and tooltip.js, so in your testing please go through the various flows and there should be no visible changes.
https://phabricator.wikimedia.org/T365634
This removes all warnings. Main takeaway is that we should be using
useCallback
hooks more often, moving forward we have to be careful to not introduce new warnings. Warnings removed spanned the following files: preview.js, popover.js, inline.js, and tooltip.js, so in your testing please go through the various flows and there should be no visible changes.