Skip to content

Stats: Add days/weeks/months/years picker for charts#25093

Merged
kean merged 7 commits intotrunkfrom
task/stats-granularity
Jan 13, 2026
Merged

Stats: Add days/weeks/months/years picker for charts#25093
kean merged 7 commits intotrunkfrom
task/stats-granularity

Conversation

@kean
Copy link
Copy Markdown
Contributor

@kean kean commented Dec 23, 2025

Description

Note: I decided not to limit the available combinations. You can pick "Last 10 years" and select "Days" granularity. It likely will not load, but you can just not do that and it's unclear where to set the boundaries otherwise.

Testing instructions

Screen.Recording.2025-12-23.at.1.33.43.PM.mov

@kean kean added the Stats label Dec 23, 2025
@kean kean added this to the 26.6 milestone Dec 23, 2025
@kean kean force-pushed the task/stats-granularity branch from f1709c4 to 2e5a74d Compare December 23, 2025 18:40
@kean kean requested a review from crazytonyli December 23, 2025 18:40
@wpmobilebot
Copy link
Copy Markdown
Contributor

wpmobilebot commented Dec 23, 2025

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 Number30415
VersionPR #25093
Bundle IDorg.wordpress.alpha
Commit96d1b92
Installation URL471dqqncut9h8
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@wpmobilebot
Copy link
Copy Markdown
Contributor

wpmobilebot commented Dec 23, 2025

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 Number30415
VersionPR #25093
Bundle IDcom.jetpack.alpha
Commit96d1b92
Installation URL6ku2s8i23kdgg
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@crazytonyli
Copy link
Copy Markdown
Contributor

Should the available selections for granularity be limited to lower than the "interval" of the selected date range?

For example, if I'm viewing data within a week (2-8 Dec in the example below), there is no need to give the option to select "year" and "month" granularity, even "week", I think?

I'm not sure how to interpret the chart below, where I selected "months" granularity on a 7 days date period. What does the multiple "Dec" on x-axis mean?

Screenshot 2026-01-06 at 9 27 11 PM

@kean
Copy link
Copy Markdown
Contributor Author

kean commented Jan 8, 2026

Thanks for the feedback. It does need some improvements, especially in terms of how labels are shown. The X axis labels in general are not well aligned with the data. I think we might need a different strategy for implementing it.

Should the available selections for granularity be limited to lower than the "interval" of the selected date range?

It works like the web, and I think it's an OK approach. Here's CC rationale:

Screenshot 2026-01-08 at 2 20 19 PM

@kean kean force-pushed the task/stats-granularity branch from 2e5a74d to 96d1b92 Compare January 13, 2026 18:30
@kean
Copy link
Copy Markdown
Contributor Author

kean commented Jan 13, 2026

I fixed the width of the bars and also the duplicated x axis labels. I can't quite make it align with the bar though. I'll open a separate ticket for it.

Screenshot 2026-01-13 at 1 31 31 PM

@sonarqubecloud
Copy link
Copy Markdown

@kean kean enabled auto-merge January 13, 2026 18:34
@kean kean added this pull request to the merge queue Jan 13, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jan 13, 2026
@wpmobilebot
Copy link
Copy Markdown
Contributor

🤖 Build Failure Analysis

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

@kean kean added this pull request to the merge queue Jan 13, 2026
@kean kean removed this pull request from the merge queue due to a manual request Jan 13, 2026
@kean kean added this pull request to the merge queue Jan 13, 2026
Merged via the queue into trunk with commit d8ee570 Jan 13, 2026
29 of 35 checks passed
@kean kean deleted the task/stats-granularity branch January 13, 2026 19:52
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.

3 participants