[18.0][IMP] product_contract - add class o_row to auto_renew fields

[anthonissen-a]

Added class="o_row" to the auto-renew divs to ensure consistent layout and styling with the recurrence section.

[OCA-git-bot]

Hi @sbejaoui,
some modules you are maintaining are being modified, check this out!

[mamcode]

Thanks for this improvement @anthonissen-a
At the code level, I understand that this improvement adds consistency in layout and styling with the recurrence section, but visually I can't see any changes. In any case, I believe this is the correct way the code should be.

[anthonissen-a]

Hi @mamcode, thanks for the review! In one of our projects, I had to make these fields readonly, and that's where the styling difference becomes noticeable. You can see the contrast in the screenshots below, with and without class="o_row" applied :

[mamcode]

@anthonissen-a Thanks, now it's clear to me.

[yvaucher]

Seems consistant with the rest of the module. (I wonder what happens on the responsivness though)

[yvaucher]

/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-1318-by-yvaucher-bump-patch, awaiting test results.

[OCA-git-bot]

@yvaucher your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-1318-by-yvaucher-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.