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][IMP] product_print_category_food_report Some improvements after big upgrade #156

Conversation

quentinDupont
Copy link
Member

@quentinDupont quentinDupont commented Jul 2, 2024

Ticket 1156

Correctif sur les étiquettes, suite aux grosses modifications en Juin.

  • Tous les formats : espace entre chaque étiquette pour mieux gérer la découpe
  • Formats Étagère carré :
    • plus petit code barre
    • plus gros nom de produit sur 2 lignes au lieu de 3
    • marge en haut et en bas
  • Formats Étagère rectangulaire : espace plus petit entre Fabrication et Fabricant·e, et labels plus gros
  • Format Comptoir : nom du produit plus gros
  • Formats Vrac :
    • nom de produit plus gros
    • champ ingrédients ne dépasse plus,
    • labels collé en bas du centre de l'étiquette
  • ☀️ 2 nouveaux format d'étiquettes
    • pour les vraciers Mobilwood n°33 : 68x68mm
    • pour les produits emballés ne nécessitant ni label ni code-barre n°40 : 61x24mm

@quentinDupont quentinDupont marked this pull request as ready for review July 3, 2024 09:45
@legalsylvain
Copy link
Member

Tous les formats : espace entre chaque étiquette pour mieux gérer la découpe

ça ce n'était pas le cas avant. (à part sur une je crois.) Du coup, il s'agit d'un changement.

Copy link
Member

@legalsylvain legalsylvain left a comment

Choose a reason for hiding this comment

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

Cool ces changements.
J'ai un petit bémol pour l'ajout de padding. j'ai une préférence pour que les blocs remplisse toute la page, je trouve ça plus simple à gérer, et y'a pas d'effet de bord. (on peut en causer de visu si tu veux.)
Nota : je sais pas si tu as vu, mais y'a un style qui permet de voir tous les blocs, et vérifier que tout rentre, quand tu développes.

https://github.com/grap/grap-odoo-business/blob/12.0/product_print_category_food_report/report/qweb_reports.xml#L31

@quentinDupont
Copy link
Member Author

Cool ces changements. J'ai un petit bémol pour l'ajout de padding. j'ai une préférence pour que les blocs remplisse toute la page, je trouve ça plus simple à gérer, et y'a pas d'effet de bord. (on peut en causer de visu si tu veux.) Nota : je sais pas si tu as vu, mais y'a un style qui permet de voir tous les blocs, et vérifier que tout rentre, quand tu développes.

https://github.com/grap/grap-odoo-business/blob/12.0/product_print_category_food_report/report/qweb_reports.xml#L31

Ouais c'est de la balle la feuille de débug, trop bonne idée et yes je m'en suis servi !
Justement, avec le padding ça déplace pas le bloc mais juste le texte à l'intérieur, exemple :
image

@legalsylvain
Copy link
Member

Justement, avec le padding ça déplace pas le bloc mais juste le texte à l'intérieur, exemple :

ah bah oui !
https://stackoverflow.com/a/5888753

Suis une buse en CSS définitively.

Comment on lines 6 to 22
height: 1cm;
height: 1.2cm;

.main_price_text {
font-size: 30px;
}

}

.product_name{
font-size:17px;
height: 1.9cm;
font-size: 20px;
line-height: 0.8cm;
height: 1.5cm;
}

.secondary_price {
height: 0.4cm;
font-size: 9px;
height: 0.5cm;
font-size: 11px;
Copy link
Member

@legalsylvain legalsylvain Jul 5, 2024

Choose a reason for hiding this comment

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

main_price : 1 -> 1.2 (+0.2)
product_name : 1.9 -> 1.5 (-0.4)
secondary_price : 0.4 -> 0.5 (+0.1)

du coup, ça fait une diminition de 0.1. et je me dis que les comptes sont pas bon Kévin ;-)
t'en penses quoi ?

Copy link
Member Author

Choose a reason for hiding this comment

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

🙈

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed !

@quentinDupont quentinDupont force-pushed the 12.0-IMP-product_print_category_food_report-after-big-upgrade branch from e4dc6a6 to 958c7da Compare July 5, 2024 12:42
@quentinDupont quentinDupont force-pushed the 12.0-IMP-product_print_category_food_report-after-big-upgrade branch from 14fde93 to 3366752 Compare July 23, 2024 14:03
@legalsylvain
Copy link
Member

MeP du 28/07/2024

/ocabot merge minor

@github-grap-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-156-by-legalsylvain-bump-minor, awaiting test results.

@github-grap-bot github-grap-bot merged commit 55679a4 into grap:12.0 Jul 29, 2024
2 checks passed
@github-grap-bot
Copy link
Contributor

Congratulations, your PR was merged at 26e27ed. Thanks a lot for contributing to grap. ❤️

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.

3 participants