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

[FIX] base: restore address display alignments #34371

Closed

Conversation

qsm-odoo
Copy link
Contributor

Commit 0c6e0d6
used flex to properly align address information and the fa icons. The
problem is that using flex prevents the parent div to ask for the
address information to be center/right aligned. This is the case for
the job pages in internal.

This commit uses display: inline-block on the top-level element of the
address to solve the problem.

opw-2000947

Commit odoo@0c6e0d6
used flex to properly align address information and the fa icons. The
problem is that using flex prevents the parent div to ask for the
address information to be center/right aligned. This is the case for
the job pages in internal.

This commit uses display: inline-block on the top-level element of the
address to solve the problem.

opw-2000947
@qsm-odoo qsm-odoo added RD research & development, internal work Website labels Jun 26, 2019
@qsm-odoo qsm-odoo self-assigned this Jun 26, 2019
@qsm-odoo
Copy link
Contributor Author

@robodoo r+

robodoo pushed a commit that referenced this pull request Jun 26, 2019
Commit 0c6e0d6
used flex to properly align address information and the fa icons. The
problem is that using flex prevents the parent div to ask for the
address information to be center/right aligned. This is the case for
the job pages in internal.

This commit uses display: inline-block on the top-level element of the
address to solve the problem.

opw-2000947

closes #34371

Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses merging 👷 and removed merging 👷 labels Jun 26, 2019
@robodoo
Copy link
Contributor

robodoo commented Jun 26, 2019

Merged, thanks!

@robodoo robodoo closed this Jun 26, 2019
@qsm-odoo qsm-odoo deleted the saas-12.2-fix-address-alignments-qsm branch June 26, 2019 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI 🤖 Robodoo has seen passing statuses RD research & development, internal work Website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants