Skip to content
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

[12.0][FIX] l10n_es_vat_book: retocar informes del Libro de IVA para que el… #1231

Merged
merged 1 commit into from Dec 2, 2019

Conversation

jonerikceberio
Copy link

… wkhtmltopdf pueda imprimirlos correctamente
Son parte de los problemas #1144 y #1217 : los pdf salen en blanco.
El motivo: el wkhtmltopdf NO es capaz de evaluar los div con float:left como lo haría un navegador, es decir haciéndolos "saltar de línea", y lo que hace es ponerlos flotantes hacia la derecha hasta que se salen del área de impresión. Con ello ciertos datos "desaparecen" y el usuario cree que está en blanco.
Solución: poner un div con class="row" para forzarlo.

… wkhtmltopdf pueda imprimirlos correctamente
@pedrobaeza

This comment has been minimized.

@pedrobaeza pedrobaeza added this to the 12.0 milestone Dec 2, 2019
@pedrobaeza
Copy link
Member

@carlosdauden tú probaste esto, verdad?

@pedrobaeza
Copy link
Member

OK, revisando el histórico del PR parece que sí, que los PDFs no están actualizados, aunque no fue el foco de la actualización que era para el formato oficial BOE, así que esto siempre es bienvenido. Gracias por corregirlo.

@pedrobaeza pedrobaeza changed the title [FIX] l10n_es_vat_book: retocar informes del Libro de IVA para que el… [12.0][FIX] l10n_es_vat_book: retocar informes del Libro de IVA para que el… Dec 2, 2019
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Probado y funcionando.

Paso a fusionar sin más dilación.

Gracias!

@pedrobaeza
Copy link
Member

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-1231-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Dec 2, 2019
Signed-off-by pedrobaeza
@OCA-git-bot OCA-git-bot merged commit 38a7dbb into OCA:12.0 Dec 2, 2019
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 66a783d. Thanks a lot for contributing to OCA. ❤️

@OCA-git-bot OCA-git-bot changed the title [12.0][FIX] l10n_es_vat_book: retocar informes del Libro de IVA para que el… [12.0][FIX] l10n_es_vat_book: retocar informes del Libro de IVA para que el… Dec 2, 2019
@jonerikceberio jonerikceberio deleted the 12.0-fix_vat_book_reports branch December 3, 2019 08:09
pedrobaeza pushed a commit to Tecnativa/l10n-spain that referenced this pull request May 26, 2020
Signed-off-by pedrobaeza
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.

None yet

3 participants