fix(Dropdown): Do not focus Dropdown if menu is not opened

[karelhala]

What:
If user clicks anywhere on screen and dropdown is present it will automatically scroll to such dropdown even if it's not opened.

Additional issues:
RedHatInsights/insights-frontend-components#222

[karelhala]

Honestly I think that this bug should have been fixed differently, but this works as a hot fix.

Proper fix would require adding domlisteners to DropdownMenu and triggering toggle from there because this menu is destroyed everytime we close it. Plus with this fix we would clean event listener (quite expensive to fire onToggle for multiple dropdowns).

[patternfly-build]

PatternFly-React preview: https://1222-pr-patternfly-react-patternfly.surge.sh

[coveralls]

Pull Request Test Coverage Report for Build 4066

• 1 of 2 (50.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.008%) to 80.356%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
packages/patternfly-4/react-core/src/components/Dropdown/Toggle.js	1	2	50.0%

Totals
Change from base Build 4063:	-0.008%
Covered Lines:	4544
Relevant Lines:	5320

---

💛 - Coveralls

[jgiardino]

Thanks for submitting a bug fix for this! I wasn't able to reproduce this issue in the workspace, but also didn't see any issues with the dropdown after these changes.

[jgiardino]

The interaction and behavior looks fine to me.

[karelhala]

@jgiardino you can reproduce it by clicking on any dropdown, scrolling a bit so this dropdown is not visible and clicking anywhere on screen.

[jgiardino]

Ah, ok. Thanks for the clarification. I was able to see this issue.
😄👍

[tlabaj]

LGTM