-
-
Notifications
You must be signed in to change notification settings - Fork 2k
[17.0] [ADD] web_font_size_report_layout #3371
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 17.0
Are you sure you want to change the base?
Conversation
jorgeglez1990
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, functional review.
jelenapoblet
left a comment
There was a problem hiding this 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.
09644bb to
8b4ed3b
Compare
Done |
adasatorres
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code review. LGTM
|
This PR has the |
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).