Fix deeplink to operation under tag with space

[jeffy-grh]

Description

When a tag had space(s), the deep link to operations under the tag won't work since the id generated for operation has spaces. This issue is not present in deep link to the tag where the space(s) are replaced by underscores.

Motivation and Context

Our use of Swagger UI had operations under a tag with space(s) in between, which we had to link to from our documentation.
Fixes #4120 .

How Has This Been Tested?

The fix was tested manually.
I haven't run any existing test scripts or added any tests to cover this case. I need help for adding test scripts, if possible.

Checklist

My PR contains...

• No code changes (src/ is unmodified: changes to documentation, CI, metadata, etc.)
• Dependency changes (any modification to dependencies in package.json)
• Bug fixes (non-breaking change which fixes an issue)
• Improvements (misc. changes to existing features)
• Features (non-breaking change which adds functionality)

My changes...

• are breaking changes to a public API (config options, System API, major UI change, etc).
• are breaking changes to a private API (Redux, component props, utility functions, etc.).
• are breaking changes to a developer API (npm script behavior changes, new dev system dependencies, etc).
• are not breaking changes.

Documentation

• My changes do not require a change to the project documentation.
• My changes require a change to the project documentation.
• If yes to above: I have updated the documentation accordingly.

Automated tests

• My changes can not or do not need to be tested.
• My changes can and should be tested by unit and/or integration tests.
• If yes to above: I have added tests to cover my changes.
• If yes to above: I have taken care to cover edge cases in my tests.
• All new and existing tests passed.

[jeffy-grh]

@shockey , Do these changes look good for the issue?

[shockey]

Hey @jeffy-grh! Using the createDeepLinkPath helper seems good, but I'd prefer the escaping to happen where the id is being set (here for tags and here for operations) rather than changing the layout keys entirely.

[shockey]

I'd prefer the escaping to happen where the id is being set rather than changing the layout keys entirely.

[shockey]

Closing due to inactivity.

This is simply to keep our issue tracker clean - feel free to comment if there are any further thoughts or concerns, and we'll be happy to reopen this.