Skip to content

Conversation

@syci
Copy link

@syci syci commented Nov 21, 2025

Odoo’s default report layout uses a fixed font size for all PDF documents.
In many real use cases, especially when invoice or sales order lines include long product descriptions or detailed content, this results in documents that are difficult to read or excessively long.

This module introduces a configurable font size in the Document Layout settings.
Administrators can choose between 9pt, 10pt, 12pt, and 14pt to optimize readability and space usage, adapting reports to company needs (compactness vs. visibility).

Copy link

@jorgeglez1990 jorgeglez1990 left a comment

Choose a reason for hiding this comment

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

LGTM, functional review.

Copy link

@jelenapoblet jelenapoblet left a comment

Choose a reason for hiding this comment

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

Looks great overall, a simple and very useful contribution.
Just one small suggestion: the code includes the full range of sizes (9, 10, 11, 12, 13, 14 with 11 as default), but the DESCRIPTION.md and README only list 9/10/12/14. It would be good to update the docs so they match the actual selection options.
Everything else looks solid.

@ajfebres ajfebres force-pushed the 17.0-web_font_size_report_layout branch from 09644bb to 8b4ed3b Compare November 24, 2025 21:04
@syci
Copy link
Author

syci commented Nov 26, 2025

Looks great overall, a simple and very useful contribution. Just one small suggestion: the code includes the full range of sizes (9, 10, 11, 12, 13, 14 with 11 as default), but the DESCRIPTION.md and README only list 9/10/12/14. It would be good to update the docs so they match the actual selection options. Everything else looks solid.

Done

Copy link

@adasatorres adasatorres left a comment

Choose a reason for hiding this comment

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

Code review. LGTM

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants