Domain management: Show the DNS subpage in site context

[bogiii]

Closes #97967

Proposed Changes

• Update method to generate path based on the context

Why are these changes being made?

• Domains management revamp

Testing Instructions

• Enable the feature flag by window.sessionStorage.setItem('flags', 'calypso/all-domain-management')
• Purchase a blog domain and attach a site with it.
• Now go to calypso.localhost:3000/sites and open this site in flyout
• Scroll to the bottom and click on the row to open the domain in site context
• Open the DNS Records section and click on the "Manage" button
• Check if the screen is rendered in the site-context

• Enable the feature flag by window.sessionStorage.setItem('flags', 'calypso/all-domain-management')
• Purchase a blog domain and attach a site with it.
• Now go to calypso.localhost:3000/sites and open this site in flyout
• Scroll to the bottom, open the context menu, and select "Manage DNS"
• Check if you stay in the same context

Pre-merge Checklist

• Has the general commit checklist been followed? (PCYsg-hS-p2)
• Have you written new tests for your changes?
• Have you tested the feature in Simple (P9HQHe-k8-p2), Atomic (P9HQHe-jW-p2), and self-hosted Jetpack sites (PCYsg-g6b-p2)?
• Have you checked for TypeScript, React or other console errors?
• Have you used memoizing on expensive computations? More info in Memoizing with create-selector and Using memoizing selectors and Our Approach to Data
• Have we added the "[Status] String Freeze" label as soon as any new strings were ready for translation (p4TIVU-5Jq-p2)?
  • For UI changes, have we tested the change in various languages (for example, ES, PT, FR, or DE)? The length of text and words vary significantly between languages.
• For changes affecting Jetpack: Have we added the "[Status] Needs Privacy Updates" label if this pull request changes what data or activity we track or use (p4TIVU-aUh-p2)?

[github-actions]

Calypso Live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-129792

Jetpack Cloud live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-129792&env=jetpack

Automattic for Agencies live (direct link)

https://calypso.live?image=registry.a8c.com/calypso/app:build-129792&env=a8c-for-agencies

[matticbot]

This PR modifies the release build for the following Calypso Apps:

For info about this notification, see here: PCYsg-OT6-p2

• notifications
• wpcom-block-editor

To test WordPress.com changes, run install-plugin.sh $pluginSlug update/domains-management-email-13 on your sandbox.

[matticbot]

Here is how your PR affects size of JS and CSS bundles shipped to the user's browser:

App Entrypoints (~41 bytes added 📈 [gzipped])

name                   parsed_size           gzip_size
entry-main                   +85 B  (+0.0%)      +22 B  (+0.0%)
entry-login                  +85 B  (+0.0%)      +22 B  (+0.0%)
entry-domains-landing        +85 B  (+0.0%)      +17 B  (+0.0%)

Common code that is always downloaded and parsed every time the app is loaded, no matter which route is used.

Sections (~159 bytes added 📈 [gzipped])

name                  parsed_size           gzip_size
hosting                    +425 B  (+0.0%)     +136 B  (+0.0%)
domains                    +256 B  (+0.0%)     +110 B  (+0.0%)
site-overview               +91 B  (+0.0%)      +29 B  (+0.0%)
update-design-flow          +85 B  (+0.0%)      +22 B  (+0.0%)
link-in-bio-tld-flow        +85 B  (+0.0%)      +22 B  (+0.0%)
copy-site-flow              +85 B  (+0.0%)      +22 B  (+0.0%)

Sections contain code specific for a given set of routes. Is downloaded and parsed only when a particular route is navigated to.

Async-loaded Components (~69 bytes added 📈 [gzipped])

name                                                                              parsed_size           gzip_size
async-load-signup-steps-page-picker                                                     +85 B  (+0.0%)      +22 B  (+0.0%)
async-load-signup-steps-domains                                                         +85 B  (+0.0%)      +23 B  (+0.0%)
async-load-purchase-modal-wrapper                                                       +85 B  (+0.0%)      +22 B  (+0.0%)
async-load-my-sites-checkout-purchase-modal-is-eligible-for-one-click-checkou...        +85 B  (+0.0%)      +22 B  (+0.0%)
async-load-calypso-my-sites-checkout-modal                                              +85 B  (+0.0%)      +24 B  (+0.0%)

React components that are loaded lazily, when a certain part of UI is displayed for the first time.

Legend

What is parsed and gzip size?

Parsed Size: Uncompressed size of the JS and CSS files. This much code needs to be parsed and stored in memory.
Gzip Size: Compressed size of the JS and CSS files. This much data needs to be downloaded over network.

Generated by performance advisor bot at iscalypsofastyet.com.

[phcp]

Overall works great, just noticed that the breadcrumb is missing the "DNS records" item.

[phcp]

Nothing for this specific PR, but does this variable refer to the same thing as the inSiteContext? If that is the case, I think we should converge to use the same name. isHostingOverview seems more readable to me.

[bogiii]

Yeah, that refers to the hosting-overview context, which is indeed a site context.

[phcp]

Another thing I noticed is that the tabs remain displayed on the subpage, which is not expected in the design of Figma.

[Imran92]

Hi @bogiii 👋 There is a rebase conflict, can you kindly fix that to unblock the reviewing? Thanks!