Skip to content

Conversation

@kean
Copy link
Contributor

@kean kean commented Jan 16, 2026

Fixes CMM-785: Add a way to turn comparisons with previous periods off.

Screenshots

With this change, users can make the new interface appear more like the simpler production implementation. When you disable comparisons, it still shows the basic trend indicators, which matches the web and the production version on mobile. In the future, we should probably add a dedicated screen for managing these preferences and add more options – can't satisfy everyone with just the defaults.

Before / After

Screenshot 2026-01-16 at 2 07 13 PM Screenshot 2026-01-16 at 2 05 45 PM Screenshot 2026-01-16 at 2 07 17 PM Screenshot 2026-01-16 at 2 06 00 PM

I also slightly changed how the menu item looks:

Screenshot 2026-01-16 at 2 06 09 PM Screenshot 2026-01-16 at 2 07 09 PM

@dangermattic
Copy link
Collaborator

dangermattic commented Jan 16, 2026

1 Warning
⚠️ This PR is assigned to the milestone 26.6. This milestone is due in less than 4 days.
Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by 🚫 Danger

@kean kean added the Stats label Jan 16, 2026
@kean kean added this to the 26.6 milestone Jan 16, 2026
@kean kean force-pushed the task/comparison-period branch from 7cd32d0 to db3ea40 Compare January 16, 2026 19:20
VStack(spacing: Constants.step1) {
chartHeaderView
.padding(.trailing, -Constants.step0_5)
if dateRange.comparison != .off || metrics.count == 1 {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We still need to show it for a (pretty rate) scenario where you add a chart and remove all metrics form it except for one.

@kean kean force-pushed the task/comparison-period branch from e194d1e to 8079909 Compare January 16, 2026 19:31
@kean kean requested a review from crazytonyli January 16, 2026 19:31
@sonarqubecloud
Copy link

@wpmobilebot
Copy link
Contributor

App Icon📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack
ConfigurationRelease-Alpha
Build Number30472
VersionPR #25143
Bundle IDcom.jetpack.alpha
Commit8079909
Installation URL50cp207j4po6o
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

App Icon📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress
ConfigurationRelease-Alpha
Build Number30472
VersionPR #25143
Bundle IDorg.wordpress.alpha
Commit8079909
Installation URL3el50obmv2la8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Contributor

🤖 Build Failure Analysis

This build has failures. Claude has analyzed them - check the build annotations for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants