fix(Popup): Adding toggle functionality if trigger element is not button

[kolaps33]

Currently if the trigger element for the popup is not button, then popup is not open by Enter/Space key.

Specification by @jurokapsiar regarding Mouse vs Keyboard action.
Mouse hover > then with keyboard "Enter" should open popup and "ESC" close
Focus > then with keyboard "Focus" should open popup and "ESC" close
Click >

• then with keyboard "Enter" should open popup and "ESC" close
• then with keyboard "Enter" should open popup and "Enter" close (toggle)

[levithomason]

Let's get a test added that shows this works. Ensure to make assertions only on what the end user has access to. That means avoid making assertions that the state changed, and prefer things like checking if the popup is actually open on the page.

[silviuaavram]

Split the test into 2 distinct ones. But the thing you actually test is ok.

[mnajdova]

This has regressions on the popups that have on="hover" or on="focus". In this cases, the enter/space should always open the popup not open and close it. We should add the toggling behavior only if we have click as a combination in the value of the on prop.

[jurokapsiar]

We should add the toggling behavior only if we have click as a combination in the value of the on prop.

agree for click. for other combinations we should do the same what click event does - open the popup if not already open.

[kuzhelov]

lets use interpolated class names instead of hardcoded strings:

return popup.find(`${Popup.Content.className}`)

[kuzhelov]

nit: lets rename it in a way togglePopupTest -> testPopupToggleOn, this will make test's code more readable

The requested changes were implemented.