search

OCA / multi-company

GNU Affero General Public License v3.0
101 stars 281 forks source link

[IMP] company_dependent_flag: icon position improved on company dependent fields #569

Closed dora-jurcevic closed 8 months ago

dora-jurcevic commented 8 months ago

CSS style modified in order to display the building icon on the same line as the input field on the form. The two elements (the icon and the field) are now better aligned

BEFORE Screenshot from 2024-01-16 14-25-18

AFTER

Screenshot from 2024-01-16 14-24-11

bealdav commented 8 months ago

Thanks a lot @legalsylvain you're right it's better to be near label but it's harder and require more code because label is either near to the field or either before with <div><label for="field"/></div>.

Then this patch is a kind quick and welcome fix with very few code, then sure.

Other behavior needs other PR, you're right, but code could be more tricky

legalsylvain commented 8 months ago

/ocabot merge patch

OCA-git-bot commented 8 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-569-by-legalsylvain-bump-patch, awaiting test results.

OCA-git-bot commented 8 months ago

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

bealdav commented 8 months ago

Thanks a lot @legalsylvain