Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0017853 | Openbravo ERP | A. Platform | public | 2011-06-30 15:32 | 2011-07-06 02:43 |
|
| Reporter | rgoris | |
| Assigned To | mtaal | |
| Priority | high | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | 3.0MP1 | Fixed in Version | | |
| Merge Request Status | |
| Review Assigned To | |
| OBNetwork customer | |
| Web browser | |
| Modules | Core |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0017853: ESC to cancel does not work for read-only rows when edited |
| Description | Somehow this is logical but still not nice for the user. Since the row is readonly, nothing is changed and the edit mode never starts. Still, ESC should be the exit for the user. |
| Steps To Reproduce | Select a booked sales order
Press pen button to get into edit mode
Row cant be edited (because it is booked)
Now you want to press the ESC key to cancel
Nothing happens |
| Proposed Solution | One of the two solutions:
(1) Do not allow editing at all for readonly rows (hide pen icon, and Edit item in context menu)
(2) Do allow editing (well, sort of) but make sure ESC works.
(1) is much better but what I understood,much more work. So maybe implementing ESC for the time being is best. |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files |  ESC-to-cancel.png (139,272) 2011-06-30 15:32
https://issues.openbravo.com/file_download.php?file_id=4265&type=bug
|
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2011-06-30 15:32 | rgoris | New Issue | |
| 2011-06-30 15:32 | rgoris | Assigned To | => mtaal |
| 2011-06-30 15:32 | rgoris | File Added: ESC-to-cancel.png | |
| 2011-06-30 15:32 | rgoris | Modules | => Core |
| 2011-07-01 23:27 | hgbot | Checkin | |
| 2011-07-01 23:27 | hgbot | Note Added: 0038700 | |
| 2011-07-01 23:27 | hgbot | Status | new => resolved |
| 2011-07-01 23:27 | hgbot | Resolution | open => fixed |
| 2011-07-01 23:27 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/f0ea5cc1072f10bc6e5da02c3a5e9786883a2005 [^] |
| 2011-07-02 15:05 | rgoris | Note Added: 0038702 | |
| 2011-07-02 15:05 | rgoris | Status | resolved => closed |
| 2011-07-02 16:44 | hgbot | Checkin | |
| 2011-07-02 16:44 | hgbot | Note Added: 0038708 | |
| 2011-07-02 16:45 | hgbot | Checkin | |
| 2011-07-02 16:45 | hgbot | Note Added: 0038709 | |
| 2011-07-02 18:20 | hgbot | Checkin | |
| 2011-07-02 18:20 | hgbot | Note Added: 0038719 | |
| 2011-07-06 02:42 | hudsonbot | Checkin | |
| 2011-07-06 02:42 | hudsonbot | Note Added: 0038861 | |
| 2011-07-06 02:42 | hudsonbot | Checkin | |
| 2011-07-06 02:42 | hudsonbot | Note Added: 0038866 | |
| 2011-07-06 02:43 | hudsonbot | Checkin | |
| 2011-07-06 02:43 | hudsonbot | Note Added: 0038867 | |
| 2011-07-06 02:43 | hudsonbot | Checkin | |
| 2011-07-06 02:43 | hudsonbot | Note Added: 0038871 | |
|
Notes |
|
|
(0038700)
|
|
hgbot
|
|
2011-07-01 23:27
|
|
Repository: erp/devel/pi
Changeset: f0ea5cc1072f10bc6e5da02c3a5e9786883a2005
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Fri Jul 01 23:25:51 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f0ea5cc1072f10bc6e5da02c3a5e9786883a2005 [^]
Fixes issue 17853: ESC to cancel does not work for read-only rows when edited
---
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-grid.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
M modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
---
|
|
|
|
(0038702)
|
|
rgoris
|
|
2011-07-02 15:05
|
|
|
Excellent fixes again Martin :-) |
|
|
|
(0038708)
|
|
hgbot
|
|
2011-07-02 16:44
|
|
Repository: erp/devel/pi
Changeset: 7c5f9bf662ac8435b0e471313590bc17822c755c
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sat Jul 02 16:42:08 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/7c5f9bf662ac8435b0e471313590bc17822c755c [^]
Related to issue 17853: ESC to cancel does not work for read-only rows when edited
Added check on ctrl, alt, shft keys and return false
---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-grid.js
---
|
|
|
|
(0038709)
|
|
hgbot
|
|
2011-07-02 16:45
|
|
Repository: erp/devel/pi
Changeset: f16df639d37d2db3c58a10d22f5b2b5847b62e9a
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sat Jul 02 16:43:45 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f16df639d37d2db3c58a10d22f5b2b5847b62e9a [^]
Related to issue 17853: ESC to cancel does not work for read-only rows when edited, Related to issue 17836: Add Jump-to-selected-row in grid
Added check on ctrl, shft, alt and return false
Made changes for 17836 to have style controlled through the sin
---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
---
|
|
|
|
(0038719)
|
|
hgbot
|
|
2011-07-02 18:20
|
|
Repository: erp/devel/pi
Changeset: 50f8f74bf2be18346a944f758b09d58d2f07149d
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sat Jul 02 18:19:55 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/50f8f74bf2be18346a944f758b09d58d2f07149d [^]
Related to issue 17853: ESC to cancel does not work for read-only rows when edited
Repaired wrong key combination setting
---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-grid.js
---
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|