Skip to content

[8.19] [Reporting] set zoom to 1 for long reports (#248785)#251552

Merged
pmuellr merged 1 commit intoelastic:8.19from
pmuellr:backport/8.19/pr-248785
Feb 4, 2026
Merged

[8.19] [Reporting] set zoom to 1 for long reports (#248785)#251552
pmuellr merged 1 commit intoelastic:8.19from
pmuellr:backport/8.19/pr-248785

Conversation

@pmuellr
Copy link
Copy Markdown
Contributor

@pmuellr pmuellr commented Feb 3, 2026

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

resolves elastic#138812

## Summary

Chrome has a hard limit of 16000 rows in a page screenshot. We currently
set the zoom level to 2, which means any report > 8000 in height won't
be rendered correctly.

As a partial fix, we'll set zoom to 1 for any report that is > 8000
height.

for more info: puppeteer/puppeteer#359

(cherry picked from commit 19df262)

# Conflicts:
#	x-pack/platform/plugins/private/reporting/server/routes/common/request_handler/validator.ts
@pmuellr pmuellr added the backport This PR is a backport of another PR label Feb 3, 2026
@pmuellr pmuellr enabled auto-merge (squash) February 3, 2026 19:40
Copy link
Copy Markdown
Contributor

@ersin-erdal ersin-erdal left a comment

Choose a reason for hiding this comment

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

LGTM

@pmuellr pmuellr merged commit 693c171 into elastic:8.19 Feb 4, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants