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

Layout issue with oe_inline #23914

Closed
wants to merge 2 commits into from

Conversation

msh-odoo
Copy link
Contributor

@msh-odoo msh-odoo commented Mar 28, 2018

Task: https://www.odoo.com/web#id=31003&view_type=form&model=project.task&action=333&active_id=131&menu_id=4720

Pad: https://pad.odoo.com/p/r.631015720100a9d380f3446249f0daf9

Glitch is created because input element will have padding of 1px while select element will not have any padding, so to have right alignment added 1px padding on the select element(let's keep padding on input element and make select element aligned with input)

@C3POdoo C3POdoo added the RD research & development, internal work label Apr 3, 2018
@vas-odoo vas-odoo force-pushed the master-web-selection-field-vas branch 2 times, most recently from 0a8a1d6 to 89d016f Compare April 11, 2018 12:16
@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch from 2344bcd to 9f7b842 Compare April 17, 2018 09:31
@vas-odoo vas-odoo force-pushed the master-web-selection-field-vas branch 2 times, most recently from 4dd449f to a9ed072 Compare April 23, 2018 13:16
@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch from a9ed072 to aa67069 Compare April 24, 2018 09:21
@vas-odoo vas-odoo force-pushed the master-web-selection-field-vas branch from aa67069 to f64476c Compare May 9, 2018 11:23
@vas-odoo vas-odoo force-pushed the master-web-selection-field-vas branch from f64476c to eea2792 Compare October 22, 2018 09:18
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses seen 🙂 labels Oct 22, 2018
@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch from eea2792 to decbe43 Compare April 19, 2019 05:50
@robodoo robodoo added CI 🤖 Robodoo has seen passing statuses and removed CI 🤖 Robodoo has seen passing statuses labels Apr 19, 2019
@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch from decbe43 to 276ad94 Compare April 19, 2019 07:10
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Apr 19, 2019
@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch from b182449 to 12821e4 Compare April 19, 2019 07:18
@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Apr 19, 2019
Copy link
Contributor

@qsm-odoo qsm-odoo left a comment

Choose a reason for hiding this comment

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

@vas-odoo Could you rewrite the commit messages to follow the guidelines? Thank you ! :)

@qsm-odoo
Copy link
Contributor

@msh-odoo

@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch from 12821e4 to 931a381 Compare April 19, 2019 11:34
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Apr 19, 2019
@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch 7 times, most recently from ab8b0d5 to 291f5b2 Compare April 19, 2019 12:19
@msh-odoo
Copy link
Contributor Author

@qsm-odoo done changes in commit message

Copy link
Contributor

@qsm-odoo qsm-odoo left a comment

Choose a reason for hiding this comment

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

@msh-odoo I am not sure what changed... please check back the slides we made:

  • 72 characters max per line
  • No uppercase letter in the title
  • "generic" is not an app
  • task-31003
  • ...

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Apr 19, 2019
@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch from 291f5b2 to 4e2cec8 Compare April 19, 2019 13:40
@robodoo robodoo removed the CI 🤖 Robodoo has seen passing statuses label Apr 19, 2019
@msh-odoo msh-odoo force-pushed the master-web-selection-field-vas branch from 4e2cec8 to 7e24931 Compare April 19, 2019 13:42
vas-odoo and others added 2 commits April 19, 2019 19:12
This will fix UI glitch for fields added in form view side by side

to get fields on same line and also to get space between
two fields use o_row oe_inline is used to put two elements
on same line but it is used mostly where 1 element is field
and second one is static string maybe in span, oe_inline on
two field creates glitch in UI, to have two fields or divs on
same line we should use o_row class, o_row class meant to do that

task-31003
This commit will fix UI glitch when seletion field and
any other field kept on same line

Using o_row we can put two divs, two inputs side by side but
if we have case where from two elements 1 is input and second one
is selection widget then we will have glitches in UI,
this commit will fix that glitch

task-31003
@msh-odoo
Copy link
Contributor Author

@qsm-odoo I hope you'll be happy now

@qsm-odoo
Copy link
Contributor

@msh-odoo Thanks
@robodoo r+ rebase-ff

@robodoo
Copy link
Contributor

robodoo commented Apr 19, 2019

Merge method set to rebase and fast-forward

@robodoo robodoo added the CI 🤖 Robodoo has seen passing statuses label Apr 19, 2019
@robodoo
Copy link
Contributor

robodoo commented Apr 19, 2019

Linked pull request(s) odoo/enterprise#2087 not ready. Linked PRs are not staged until all of them are ready.

@JKE-be
Copy link
Contributor

JKE-be commented Apr 21, 2019

@robodoo retry

@robodoo
Copy link
Contributor

robodoo commented Apr 21, 2019

I'm sorry, @JKE-be. Retry makes no sense when the PR is not in error.

pniederlag pushed a commit to pniederlag/odoo that referenced this pull request Apr 21, 2019
This commit will fix UI glitch when seletion field and
any other field kept on same line

Using o_row we can put two divs, two inputs side by side but
if we have case where from two elements 1 is input and second one
is selection widget then we will have glitches in UI,
this commit will fix that glitch

task-31003

closes odoo#23914

Signed-off-by: Quentin Smetz (qsm) <qsm@odoo.com>
@robodoo
Copy link
Contributor

robodoo commented Apr 21, 2019

Merged, thanks!

@robodoo robodoo closed this Apr 21, 2019
@rdeodoo rdeodoo deleted the master-web-selection-field-vas branch April 21, 2019 17:40
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants