Refactor the onboarding click popover event handler to be more declarative in nature
the handleClickPopoverButton
function is starting to do too much for me to reason about it. I'm thinking of refactoring these pieces out into a more declarative model by perhaps emitting different events than merely clickPopoverButton
for most things.
splitting into three different functions would make sense:
-
clickStepContentButton
: Handles click event on the buttons displayed on the regular content popovers (Got it
) -
clickExitTourButton
: Handles click event on theClose "Learn GitLab"
button -
clickFeedbackButton
: For the new functionality introduced in this MR