codeant-ai-for-open-source[bot] commented on code in PR #44243: URL: https://github.com/apache/superset/pull/44243#discussion_r4148236662
########## docs/admin_docs/configuration/dashboard-performance.mdx: ########## @@ -174,7 +174,7 @@ scheduled table in the warehouse so each chart query is a cheap lookup. - See [Feature Flags](./feature-flags.mdx) for the full list of supported flags and their lifecycle stages. - Server-side screenshot jobs (alerts, scheduled reports, thumbnails) - render the dashboard in a headless, webdriver-controlled browser, which + render the dashboard in a headless Chromium browser driven by Playwright, which intentionally bypasses row virtualization so the rendered artifact includes every chart, not just the ones above the fold. User-triggered Review Comment: ✅ **CodeAnt verified this suggestion was addressed in subsequent commits and marked this thread resolved** as of `ce01769`. The documentation now limits the screenshot claim to every chart on the active tab, rather than implying that charts on inactive tabs are included. <sub>If that's not right, unresolve this thread and CodeAnt will leave it open.</sub> <!-- codeant-auto-resolve-reply --> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
