GitHub user dosubot[bot] added a comment to the discussion: [6.0.0] The width 
of PDFs generated via "Dashboard">"Download">"Export PDF" is set to 1600 pxl or 
a multiple if "WEBDRIVER_WINDOW = {"pixel_density": [>1],} and are not ISO 
formats (A4, A3...)

<!-- Answer -->
That's a great idea—superset/config.py currently has only minimal comments for 
WEBDRIVER_WINDOW and related export settings, and it doesn't clarify that only 
pixel_density affects dashboard PDF export width, while the dashboard/slice 
window sizes have no effect on PDFs. Adding comments to explain these 
limitations (for example: which settings impact PNG vs. PDF exports, the fixed 
1600px base width for PDFs, and the role of pixel_density) would help users 
avoid confusion and better understand how to tune export quality and resource 
usage. Community discussions show that many users run into these issues, so 
improving the comments would be valuable for clarity and troubleshooting 
[example](https://github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset/config.py#L1524-L1546)
 [discussion](https://github.com/apache/superset/discussions/35327). 

If you decide to add comments, it's helpful to be explicit about which settings 
affect which export types, and to note the current limitations (like lack of 
ISO page sizing or DPI control for dashboard PDFs). This matches common open 
source best practices for config files.


<!-- Dosu Comment Footer -->
*To reply, just mention [@dosu](https://go.dosu.dev/dosubot).*

---
<sup>How did I do? 
[Good](https://app.dosu.dev/response-feedback/7f7e7ef2-269e-4acf-87cf-18f5f21d7c3e?feedback_type=great_response)
 | 
[Irrelevant](https://app.dosu.dev/response-feedback/7f7e7ef2-269e-4acf-87cf-18f5f21d7c3e?feedback_type=irrelevant_answer)
 | 
[Incorrect](https://app.dosu.dev/response-feedback/7f7e7ef2-269e-4acf-87cf-18f5f21d7c3e?feedback_type=incorrect_sources)
 | 
[Verbose](https://app.dosu.dev/response-feedback/7f7e7ef2-269e-4acf-87cf-18f5f21d7c3e?feedback_type=too_verbose)
 | 
[Hallucination](https://app.dosu.dev/response-feedback/7f7e7ef2-269e-4acf-87cf-18f5f21d7c3e?feedback_type=hallucination)
 | [Report 
🐛](https://app.dosu.dev/response-feedback/7f7e7ef2-269e-4acf-87cf-18f5f21d7c3e?feedback_type=bug_report)
 | 
[Other](https://app.dosu.dev/response-feedback/7f7e7ef2-269e-4acf-87cf-18f5f21d7c3e?feedback_type=other)</sup>

[![Chat with 
Dosu](https://dosu.dev/dosu-chat-badge.svg)](https://app.dosu.dev/a28d3c7e-a9d3-459e-9fb6-3a6f9ff4f357/ask?utm_source=github)&nbsp;[![Open
 in 
Cursor](https://dosu.dev/dosu-in-cursor.svg)](https://cursor.com/link/prompt?text=That%27s%20a%20great%20idea%E2%80%94superset/config.py%20currently%20has%20only%20minimal%20comments%20for%20WEBDRIVER_WINDOW%20and%20related%20export%20settings%2C%20and%20it%20doesn%27t%20clarify%20that%20only%20pixel_density%20affects%20dashboard%20PDF%20export%20width%2C%20while%20the%20dashboard/slice%20window%20sizes%20have%20no%20effect%20on%20PDFs.%20Adding%20comments%20to%20explain%20these%20limitations%20%28for%20example%3A%20which%20settings%20impact%20PNG%20vs.%20PDF%20exports%2C%20the%20fixed%201600px%20base%20width%20for%20PDFs%2C%20and%20the%20role%20of%20pixel_density%29%20would%20help%20users%20avoid%20confusion%20and%20better%20understand%20how%20to%20tune%20export%20quality%20and%20resource%20usage.%20Community%20discussions%20show%
 
20that%20many%20users%20run%20into%20these%20issues%2C%20so%20improving%20the%20comments%20would%20be%20valuable%20for%20clarity%20and%20troubleshooting%20%5Bexample%5D%28https%3A//github.com/apache/superset/blob/829e4d92d91ceae4b43b1ed3b063ffe45377799c/superset/config.py%23L1524-L1546%29%20%5Bdiscussion%5D%28https%3A//github.com/apache/superset/discussions/35327%29.%20%0A%0AIf%20you%20decide%20to%20add%20comments%2C%20it%27s%20helpful%20to%20be%20explicit%20about%20which%20settings%20affect%20which%20export%20types%2C%20and%20to%20note%20the%20current%20limitations%20%28like%20lack%20of%20ISO%20page%20sizing%20or%20DPI%20control%20for%20dashboard%20PDFs%29.%20This%20matches%20common%20open%20source%20best%20practices%20for%20config%20files.)&nbsp;[![Join
 
Discord](https://img.shields.io/badge/join-5865F2?logo=discord&logoColor=white&label=)](https://go.dosu.dev/discord-bot)&nbsp;[![Share
 on 
X](https://img.shields.io/badge/X-share-black)](https://twitter.com/intent/tweet?text=%40dosu
 
_ai%20helped%20me%20solve%20this%20issue!&url=https%3A//github.com/apache/superset/discussions/36949)

GitHub link: 
https://github.com/apache/superset/discussions/36949#discussioncomment-15434805

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: 
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to