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

[16.0][FIX] - account_vat_period_end_statement - wrong trees proportions #3380

Conversation

michelerusti
Copy link
Contributor

Fixes #3231 e #3375 per la 16.0

@tafaRU
Copy link
Member

tafaRU commented Jun 14, 2023

@andreampiovesana @maxgrassi potete testarla e fare una review?

@tafaRU
Copy link
Member

tafaRU commented Jun 14, 2023

grazie @maxgrassi, per favore aggiungi la tua review, non solo un commento.

image

Copy link
Contributor

@andreampiovesana andreampiovesana left a comment

Choose a reason for hiding this comment

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

LGTM

@MaurizioPellegrinet
Copy link

Mi risulta ancora questa vista ...
photo1686778730

Copy link
Member

@tafaRU tafaRU left a comment

Choose a reason for hiding this comment

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

@michelerusti puoi verificare l'ultima segnalazione?

Anche a me provando su runboat risulta così:

image

@michelerusti
Copy link
Contributor Author

@michelerusti puoi verificare l'ultima segnalazione?

Anche a me provando su runboat risulta così:

image

@tafaRU il problema sta in come odoo si aspetta di fromattare le form
In questo caso, mette una regola CSS che va a spaccare in due colonne la grid forzando quella di sinistra a 150px

Se volessimo continuare ad avere le due <tree> di fianco, dobbiamo forzare il parametro a, ad esempio:
grid-template-columns: fit-content(150px) 1fr;
Ma dovrei sovrascrivere la regola nel foglio di stile di questo modulo

In alternativa potremmo metterle una sopra l'altra cambiando solo la vista.

Quale pensi sia l'opzione migliore?

@michelerusti michelerusti force-pushed the fix-16.0-account_vat_period_end_statement-view branch 4 times, most recently from e4fba14 to 988d91b Compare June 16, 2023 13:49
@michelerusti michelerusti force-pushed the fix-16.0-account_vat_period_end_statement-view branch from 988d91b to 65345eb Compare June 16, 2023 13:55
@andreampiovesana
Copy link
Contributor

andreampiovesana commented Jun 16, 2023

immagine
io toglierei col= e colspan=

@michelerusti
Copy link
Contributor Author

immagine io toglierei col= e colspan=

Nel runboat ora le tree sono larghe uguali, non come nella tua immagine

Se togliessimo col= e colspan= si rompono tutte e due le tree
image

Mi sto perdendo qualcosa?

Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

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

LGTM

@maxgrassi
Copy link

Per me è OK. Grazie @michelerusti

Copy link
Member

@tafaRU tafaRU left a comment

Choose a reason for hiding this comment

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

@michelerusti grazie della PR 👍

In merito al css aggiunto:
image

Come detto in call: apri una nuove issue per indagare il css pregresso.
Grazie.

@tafaRU
Copy link
Member

tafaRU commented Jun 19, 2023

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-3380-by-tafaRU-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit cb57f72 into OCA:16.0 Jun 19, 2023
@OCA-git-bot
Copy link
Contributor

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

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