[24.2] Fix usability of workflow best practice attribute checking.

[jmchilton]

We have these links that just bring up the attributes page without any added context about which parameter you clicked or why you want to edit it. I think this is jarring and a usability bug (I'd like to backport this to 24.2 after the activity bar stuff is merged).

With these changes it now highlights the attribute you clicked the link for and displays the same message about why it is important from the best practice panel as a popover:

I'd love to allow the editing of these things inline also (#10989) but I think that guiding users to the correct place to edit these things and explaining why in context has its own value.

How to test the changes?

(Select all options that apply)

• I've included appropriate automated tests.
• This is a refactoring of components with existing test coverage.
• Instructions for manual testing are as follows:
  1. Create a workflow without an annotation or creator or license and open the best practice panel and click the respective links. Notice the new popover and highlight.

License

• I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

[bgruening]

Thanks @jmchilton