Skip to content

Jetpack Cloud: Remove old site selection triggers#101850

Closed
tyxla wants to merge 1 commit intotrunkfrom
remove/old-site-selection-jetpack-cloud-sidebar
Closed

Jetpack Cloud: Remove old site selection triggers#101850
tyxla wants to merge 1 commit intotrunkfrom
remove/old-site-selection-jetpack-cloud-sidebar

Conversation

@tyxla
Copy link
Member

@tyxla tyxla commented Mar 25, 2025

Proposed Changes

We're removing the old site selection triggers in favor of the /sites screen that allows selection.

Why are these changes being made?

In all environments (Jetpack Cloud, Automattic for Agencies, WordPress.com) we already use /sites for selecting the current site. The old site selection picker is obsolete and only appears in a few awkward places. It is time to retire it for consistency.

This is part of a bigger initiative - unifying multisite dashboard experiences.

This step is one of the last pre-requisites to removing the old picker, which I'm addressing in #101844.

Testing Instructions

  • Open Jetpack Cloud with a partner account.
  • "All sites" at the top of the sidebar shouldn't be clickable anymore. The corresponding chevron down has also been removed.
  • Go to /sites
  • Select a site by clicking on it.
  • The site name at the top of the sidebar shouldn't be clickable anymore. The corresponding chevron down has also been removed.

Screenshots

Without a selected site:

Before After
Screenshot 2025-03-25 at 18 24 40 Screenshot 2025-03-25 at 18 24 34

With a selected site:

Before After
Screenshot 2025-03-25 at 18 23 47 Screenshot 2025-03-25 at 18 23 40

@tyxla tyxla added Sites [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Status] Needs Design Review Add this when you'd like to get a review / feedback from the Design team on your PR Jetpack Cloud Anything related to the Jetpack Cloud (cloud.jetpack.com) [Feature] Multi-site Management Features related to managing multiple sites from a single WordPress.com account. labels Mar 25, 2025
@tyxla tyxla requested review from a team and keoshi March 25, 2025 16:26
@tyxla tyxla self-assigned this Mar 25, 2025
@github-actions
Copy link

github-actions bot commented Mar 25, 2025

@github-actions github-actions bot added the [Status] Design Input Requested Label automatically added to PRs where design feedback is requested label Mar 25, 2025
@matticbot
Copy link
Contributor

matticbot commented Mar 25, 2025

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

Sections (~428 bytes removed 📉 [gzipped])

Details
name                             parsed_size           gzip_size
jetpack-cloud-plugin-management       -424 B  (-0.0%)     -138 B  (-0.0%)
jetpack-cloud-partner-portal          -424 B  (-0.0%)     -147 B  (-0.0%)
jetpack-cloud-overview                -424 B  (-0.1%)     -143 B  (-0.1%)
jetpack-cloud-agency-dashboard        -424 B  (-0.0%)     -147 B  (-0.0%)
jetpack-cloud                         -424 B  (-0.1%)     -143 B  (-0.1%)

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 (~143 bytes removed 📉 [gzipped])

Details
name                                                                              parsed_size           gzip_size
async-load-calypso-jetpack-cloud-sections-sidebar-navigation-manage-selected-...       -424 B  (-0.4%)     -143 B  (-0.4%)

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.

@tyxla tyxla force-pushed the remove/old-site-selection-jetpack-cloud-sidebar branch from 0078bad to ba465d6 Compare March 25, 2025 16:36
@matticbot
Copy link
Contributor

matticbot commented Mar 25, 2025

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

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

  • notifications

To test WordPress.com changes, run install-plugin.sh $pluginSlug remove/old-site-selection-jetpack-cloud-sidebar on your sandbox.

@tyxla
Copy link
Member Author

tyxla commented Mar 26, 2025

Yeah, looks like we'll need to add the Sites management screen for non-agency Jetpack Cloud users, it doesn't appear there (hat tip @jsnajdr for pointing that out):

Screenshot 2025-03-26 at 15 12 41

I'll switch this to draft for now as I look at how this can be done with as little friction as possible. We still have the site selection middleware and the site selector component, so it shouldn't be a big deal with a small design change.

@tyxla tyxla marked this pull request as draft March 26, 2025 13:14
@tyxla tyxla added [Status] In Progress and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Status] Needs Design Review Add this when you'd like to get a review / feedback from the Design team on your PR labels Mar 26, 2025
@matticbot matticbot added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Mar 26, 2025
@github-actions
Copy link

This PR has been marked as stale. This happened because:

  • It has been inactive for the past 3 months.
  • It hasn't been labeled `[Pri] BLOCKER`, `[Pri] High`, `[Status] Keep Open`, etc.

If this PR is still useful, please do a trunk merge or rebase and otherwise make sure it's up to date and has clear testing instructions. You may also want to ping possible reviewers in case they've forgotten about it. Please close this PR if you think it's not valid anymore — if you do, please add a brief explanation.

If the PR is not updated (or at least commented on) in another month, it will be automatically closed.

@tyxla
Copy link
Member Author

tyxla commented Aug 12, 2025

Not planning more work on this for now. May revisit as the hosting dashboard matures. Thanks everyone!

@tyxla tyxla closed this Aug 12, 2025
@tyxla tyxla deleted the remove/old-site-selection-jetpack-cloud-sidebar branch August 12, 2025 20:32
@github-actions github-actions bot removed [Status] In Progress [Status] Design Input Requested Label automatically added to PRs where design feedback is requested [Status] Stale labels Aug 12, 2025
@github-actions github-actions bot removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Multi-site Management Features related to managing multiple sites from a single WordPress.com account. Jetpack Cloud Anything related to the Jetpack Cloud (cloud.jetpack.com) Sites

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments