Keep the Link Control settings drawer persistent

[abhi3315]

What?

This PR will keep the Link Control setting open if it is opened or closed if it was closed earlier.
By default, the setting drawer will be closed.

Why?

#47821
#47821 (comment)

How?

• Get the state of the setting drawer from the preferences store.
• Add a useEffect to set the preference whenever the setting drawer is opened/closed.

Testing Instructions

• Go to the Gutenberg Editor.
• Open/close the setting drawer of Link Control.
• Check if the setting drawer is keeping the state.

Testing Instructions for Keyboard

Screenshots or screencast

[github-actions]

👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @abhi3315! In case you missed it, we'd love to have you join us in our Slack community, where we hold regularly weekly meetings open to anyone to coordinate with each other.

If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information.

[getdave]

The tests are failing because of the toggle setting state because in each test setting drawer is supposed to be closed by default. But now the setting drawer state of LinkControl is persistent and will depend on the previous component state.

You could make a new hook (separate file) to get the preference and then mock the hook in the tests so you can control it.

For the existing tests we should set it to always return false (i.e. always closed by default) but we should try adding a test for the case when it returns true to assert the drawer behaves as we expect.

See edit/test/navigation-menu-selector.js for an example of this.

[getdave]

Minor: we tend to favour rebasing against trunk rather than merge commits.

[abhi3315]

@getdave As per the comment you made on my earlier PR #47930 (comment)

I should not query the store in the component itself. So I might have to refactor the code.

[getdave]

@draganescu I think you were envisaging some kind of heuristic here to determine when the drawer setting should be toggled?

Perhaps

if the user has toggled the drawer open/closed 3 or more times then set it to always be open/closed

If we do that I guess we'd need to store the temp "how many times toggled" state in Session Storage and then clear it once the preference is set?