Commit fbbbb21
Premium Analytics: post detail parity — overflow fixes, inline date presets, no comparison (#50971)
* Premium Analytics: post detail parity — overflow fixes, timezone, no comparison
Applies to the post/email detail page the fixes proven on the video
detail page: the breadcrumb/summary overflow bugs (contain: inline-size
plus the breadcrumbs-slot shrink shim), aspect-ratio: 1 so the boot
shell's img reset cannot squash the featured image, post-views bucket
keys parsed as site-local calendar dates (parseSiteDateTime, UTC-12
regression test), and block-size nits.
The page's design has no period-over-period comparison: the post-views
and email-time-series widgets drop their comparison series, the route
normalizes comparison params away, and DateFiltersPanel gains a minimal
showComparison prop so the Compare control hides at the existing
fixed-bar call site.
Moving the panel onto the summary's title row (per the mock) is
deferred until the preset measurement rework lands (WOOA7S-1816) — it
would collide head-on, the same reason the video-detail PR dropped its
filters row. Rebased onto trunk accordingly: nothing here depends on
the video-detail branch anymore.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* Pass comparison params through the post-detail route
Mirror the video-detail review outcome (#50970): the page renders no
comparison and its widgets ignore the params, but the breadcrumb's
dashboard link reads the URL state back out, so stripping them lost the
user's comparison settings on a Dashboard → Post → Dashboard round trip.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* Premium Analytics: align comparison pass-through comments with the route contract
* Strip comparison params from injected widget reportParams and drop the parseISO fallback
Per review: the page-wide no-comparison invariant now holds by
construction — usePostDetailTabs injects comparison-stripped
reportParams (via the new omitComparisonReportParams helper) into every
layout entry, so comparison-capable widgets like UTM insights and
highlights can no longer render deltas from URL state. The URL keeps
the comparison params for the breadcrumb round trip.
Also remove the parseISO fallback in use-post-views: bucket dates come
from format(start, 'yyyy-MM-dd') so parseSiteDateTime cannot fail, and
the fallback would silently reintroduce the browser-local day shift;
an unparseable bucket now drops the point instead.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* Record the crypto-js deprecation in pnpm-lock.yaml
The npm registry marked crypto-js@4.2.0 deprecated after this lock was
written, so the lock check's fresh resolution now expects the
deprecation note. Registry-metadata drift only; no dependency change.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* Put the date filter presets on the summary title row per the mocks
One header row per the design mocks: title left, presets right,
vertically centered. The summary grows into free space and absorbs the
squeeze down to a 400px floor (title ellipsis); its inline-size
containment keeps a long title from wrapping the row.
Known rough edge, deferred to #51088: the panel self-measures its root,
which in this shrink-to-fit slot always sees its own content width — so
the presets keep their full layout and narrow rows degrade poorly. The
external-measurement wiring (containerElement / reservedInlineSize)
ships there to keep this PR free of shared-component changes beyond the
already-reviewed showComparison prop.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* Drop the route-level breadcrumb shim in favor of the component-owned fix
Superseded by #51085: the ':has(> nav)' rule targets StatsBreadcrumbs'
display:contents trail wrapper (no box for min-inline-size to act on)
and the nav rule is covered by the component stylesheet. Tracked
upstream as WordPress/gutenberg#81297.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
* ci: retrigger checks — workflow runs were never created for the previous push
---------
Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Committed via a GitHub action: https://github.com/Automattic/jetpack/actions/runs/31156351346
Upstream-Ref: Automattic/jetpack@72071721 parent 51adb3a commit fbbbb21
32 files changed
Lines changed: 44 additions & 45 deletions
File tree
- build
- modules
- ui
- widgets-toolkit
- routes
- post-detail
- reports
- video-detail
- widgets
- email-time-series
- latest-post
- popular-post
- post-views
- top-posts
- videopress
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| 26 | + | |
26 | 27 | | |
27 | 28 | | |
28 | 29 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
0 commit comments