[7.0 trunk] Float form input fields width problem

Bug #1051746 reported by tozm
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Web (MOVED TO GITHUB)
Fix Committed
Undecided
van der Essen Frédéric (OpenERP)

Bug Description

Something seems to be wrong with the web forms containing float fields. For example, sales order form contains Qty and Unit Price. Once the form changes into edit mode, the input field width is larger than the actual table column, and is actually partially obscured by the field in the adjoining column. This makes editing extremely difficult, and in fact renders the UI quite unusable, with no workaround.

Have tried in both Firefox 14.0.1 and IE with same result. Will attach a screen shot, but should be easy to reproduce?

Revision history for this message
tozm (tozm) wrote :

Attached screenshot shows the problem. See how the halo surrounding the Qty field actually extends beyond column width and is obscured by the listbox for UoM. Being right-justified input, you can't see what yopu are typing. Similarly the unit price field seems to have the wrong width in input mode. At least in that case you can see it all as it sits on top of the total column.

Revision history for this message
tozm (tozm) wrote :

Note that this problem is low level.... not confined to just the Sales Order screen, that was just an example.

Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello,

I have checked your Issue with latest trunk, but I did not face any view problem. It's seems fine. That's why I have attached the video. would you please check it and notify us where you have faced the problem.

Thanks and waiting for your reply!

Changed in openerp-web:
status: New → Incomplete
Revision history for this message
tozm (tozm) wrote :

Am on the latest trunk.

Can you try it again on the Sales Order screen. I see the problem there, but not on the Invoicing screen as shown in your video.

Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello,

As per your comment#2 "*** ...the Sales Order screen, that was just an example". But Sales order view now changed. That's why I am take an example of sale Invoicing line. And It's work fine as per the bug point of view.

Moreover, sale order functionality are change in latest trunk. I have attached the video for your reference. So would you please try with latest code. currently I am close this Issue. If you face the problem again than without hesitation reopen this Issue.

Revision
------------
server : 4440
addons : 7471
web : 3073

Thanks for understanding!

Changed in openerp-web:
status: Incomplete → Invalid
tozm (tozm)
Changed in openerp-web:
status: Invalid → New
Revision history for this message
tozm (tozm) wrote :

I synced up again this morning with trunk and I am still seeing the same issue.

Since I observed you used Opera, I also installed Opera to rule out client-side environmental differences and saw the same issue.

Can you please do two things:

(a) Clear your browser cache to make sure you haven't got a cached version of the css hanging around... this problem may have been introduced relatively recently if work has been going on in this area.

(b) Post a video of doing a fresh sales order entry. I would like to compare exactly what you see with what I see.

Revision history for this message
tozm (tozm) wrote :

I believe I have finally found the cause of the problem....

In base.css, there is a missing width attribute in the following style def:

.openerp .oe_list.oe_list_editable.oe_editing .oe_form_field.oe_form_field_float input, .openerp .oe_list.oe_list_editable.oe_editing .oe_form_field.oe_form_view_integer input {
  text-align: right;
}

It should read:

.openerp .oe_list.oe_list_editable.oe_editing .oe_form_field.oe_form_field_float input, .openerp .oe_list.oe_list_editable.oe_editing .oe_form_field.oe_form_view_integer input {
  text-align: right;
 width:100%
}

All good now, but would be nice to see this included in trunk ASAP. (Have privately emailed fva about this.)

Changed in openerp-web:
assignee: nobody → van der Essen Frédéric (OpenERP) (fva-openerp)
Revision history for this message
van der Essen Frédéric (OpenERP) (fva-openerp) wrote :
Changed in openerp-web:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.