Skip to content

[18.0][ADD] web_font_size_report_layout: web_font_size_report_layout module added. - #3389

Open
adasatorres wants to merge 1 commit into
OCA:18.0from
BinhexTeam:18.0-add-web_font_size_report_layout
Open

[18.0][ADD] web_font_size_report_layout: web_font_size_report_layout module added.#3389
adasatorres wants to merge 1 commit into
OCA:18.0from
BinhexTeam:18.0-add-web_font_size_report_layout

Conversation

@adasatorres

Copy link
Copy Markdown

No description provided.

@rrebollo

rrebollo commented Dec 11, 2025

Copy link
Copy Markdown

This Odoo module adds the capability to customize font size in all PDF reports throughout the system. Through a simple configuration, administrators can select from a range of predefined font sizes (measured in points) to optimize readability and appearance of invoices, quotations, sales orders, and other printed documents.

The module offers the following font size options (in points):

Option Size Description
Extra Small 9pt For maximum compactness
Standard 10pt Odoo's default size
Small 11pt Just a little larger
Large 12pt Better readability
Extra Large 13pt Really Large
2Extra Large 14pt For high visibility needs

@rrebollo rrebollo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

CR. Thank you for your contribution. Please consider my suggestions. Specially the DRY ones.

Comment thread web_font_size_report_layout/models/base_document_layout.py Outdated
Comment thread web_font_size_report_layout/models/res_company.py
Comment thread web_font_size_report_layout/models/res_company.py
Comment thread web_font_size_report_layout/models/res_company.py Outdated
Comment thread web_font_size_report_layout/models/res_company.py Outdated
@adasatorres
adasatorres force-pushed the 18.0-add-web_font_size_report_layout branch from ff7825b to aa123b2 Compare December 12, 2025 08:55
@adasatorres
adasatorres requested a review from rrebollo December 12, 2025 09:02

@rrebollo rrebollo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review. LGTM!

@adasatorres
adasatorres force-pushed the 18.0-add-web_font_size_report_layout branch from aa123b2 to 5980359 Compare December 12, 2025 11:47
@adasatorres
adasatorres force-pushed the 18.0-add-web_font_size_report_layout branch from 5980359 to 9b4f60f Compare December 12, 2025 11:57
@@ -0,0 +1,3 @@
1. Copy the `web_font_size_report_layout` directory to your Odoo addons folder

@bealdav bealdav Dec 12, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks a lot for your module. Here is a suggestion

Suggested change
1. Copy the `web_font_size_report_layout` directory to your Odoo addons folder
1. Link the `web_font_size_report_layout` directory to your Odoo addons path

@edescalona edescalona left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@OCA-git-bot

Copy link
Copy Markdown
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). 🤖

@ivs-cetmix

Copy link
Copy Markdown
Member

Hey @adasatorres there other PRs for the same module already:
17.0 #3371
16.0 #3271

Looks like the 17.0 is the most up-to-date for now. So let's merge one and then migrate to other versions.

@flotho

flotho commented Apr 16, 2026

Copy link
Copy Markdown
Member

Hi @ivs-cetmix , the v17 module is merged, what's the process to have it in v18 ?

@ivs-cetmix

Copy link
Copy Markdown
Member

Hi @flotho we should open a migration PR so the commit history stays preserved. This one should be closed after that.

@DorianMAG

Copy link
Copy Markdown

@adasatorres
Hi, can you rebase plz.
Runboat is not load.
Thx

@DorianMAG

Copy link
Copy Markdown

@rrebollo
@flotho
I propose migration from 17.0 based on your work @adasatorres
#3661
Regards

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.

9 participants