Notes |
|
(0037692)
|
rgoris
|
2011-05-30 11:15
|
|
Still persists as of today. |
|
|
|
I also saw it (in Purchase Invoice, Sales Invoice). The thing is that pressing Edit on next row does not actually puts it into the edit mode (no fields are selected to be edited).
At the same time for example in the Sales Order window I cannot reproduce it, it works correctly.
Interesting thing here is that, for example, for Purchase Order from the videos Edit Button should not appear at all because it is already processed and all the fields are read-only (if you switch to the Form view - you cannot edit document). |
|
|
(0037721)
|
mtaal
|
2011-05-30 13:10
|
|
I think that's also what goes 'wrong' here. The row is in edit mode but you can't edit any of the fields. So therefore it gives this impression. Maybe the solution is to give a warning if none of the form fields are editable and then go back to non-edit state.
The 'problem' is that we don't know if the fields are editable until we actually open/start the form. So therefore the edit button is shown. It is a feature request for MP1 to support non-editable taking into account the state of the object.
The video shows also a real error (that's what the issue is for), that you can have multiple rows in edit mode and cancel does not work. |
|
|
(0037762)
|
hgbot
|
2011-05-31 08:45
|
|
Repository: erp/devel/pi
Changeset: 696fc38dd3e9471df130c956f18b820433062741
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue May 31 08:44:52 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/696fc38dd3e9471df130c956f18b820433062741 [^]
Fixes issue 17397: Grid Editing: cannot cancel editing of rows
---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
M modules/org.openbravo.userinterface.smartclient/web/org.openbravo.userinterface.smartclient/js/ob-smartclient.js
---
|
|
|
|
|
|
(0037866)
|
rgoris
|
2011-06-01 09:49
|
|
Problem persists in today´s live build. It was pushed last night so it should have been visible by now. Please check, i also added another screenshot. |
|
|
(0037868)
|
mtaal
|
2011-06-01 10:08
|
|
|