Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0019807 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] B. User interface | minor | always | 2012-02-18 01:31 | 2012-03-02 15:38 | |||
| Reporter | dbaz | View Status | public | |||||
| Assigned To | dbaz | |||||||
| Priority | high | Resolution | fixed | Fixed in Version | 3.0MP9 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | c5f71c891935 | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Review Assigned To | ||||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0019807: [P&E] Delete Icon has not "hover" styles | |||||||
| Description | Delete Icon has not "hover" styles, so when you mouseover/press it you have not visual notification | |||||||
| Steps To Reproduce | As "Openbravo Admin" * Go to "Payment Term" * Create New - "Search Key": 30-60-90 - "Name": 30-60-90 * Save * Go to "Lines" * Create New - "Percentage Due": 25 - "Rest": un-checked * Save * Create New - "Percentage Due": 25 - "Rest": un-checked * Save * Go to "Sales Invoice" * Create New - "Organization": same organization you used in the created "Payment Term" - "Business Partner": any - "Payment Term": 30-60-90 * Save * Go to "Lines" * Create 2 news with any product and Quantity 100 * "Complete" the "Sales Invoice" * Click "Modify Payment In Plan" Check that the delete icon (rubbish bin) has not any visual at all | |||||||
| Proposed Solution | There are two solutions: * Fast one: add changes in the icon and a cursor:pointer to the <img> itself. Cons: since all the cell is clickable, it should be strange, that if you click outside the <img> but still inside the cell, the action is performed. This is because the delete column is made using Smartclient standard way * Elaborated one: remove Smartclient's delete column code, and include a button using "createRecordComponent" which permorms the deletion action. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files |  ob-pick-and-execute-view.js [^] (10,508 bytes) 2012-02-18 01:36 | |||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|
 Relationships |
|
Notes |
|
|
(0045250) dbaz (developer) 2012-02-18 01:35 |
Attached a diff for the "Elaborated" solution. But it doesn't work when adding more rows. Since the way to fix it is not trivial and it seems that it would require Smartclient support, and since the issue impact with "Fast" solution is not so big, the "Fast" solution is finally implemented. If the issue returns, the "Elaborated" solution will be retried. |
|
(0045251) hgbot (developer) 2012-02-18 01:38 |
Repository: erp/devel/pi Changeset: c5f71c8919355f5610a1464d9b7f2883bc365778 Author: David Baz Fayos <david.baz <at> openbravo.com> Date: Sat Feb 18 01:37:35 2012 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/c5f71c8919355f5610a1464d9b7f2883bc365778 [^] Fixed issue 19807: [P&E] added mouse effects to 'Delete' button --- M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-pick-and-execute-view.js M modules/org.openbravo.client.application/web/org.openbravo.userinterface.smartclient/openbravo/skins/Default/org.openbravo.client.application/images/process/iconDelete.png A modules/org.openbravo.client.application/web/org.openbravo.userinterface.smartclient/openbravo/skins/Default/org.openbravo.client.application/images/process/iconDelete_Down.png A modules/org.openbravo.client.application/web/org.openbravo.userinterface.smartclient/openbravo/skins/Default/org.openbravo.client.application/images/process/iconDelete_Over.png --- |
|
(0045261) AugustoMauch (manager) 2012-02-20 11:01 |
Code reviewed and verified on live builds, revision #5e8637a2a10b65ccaa75b357399c0463d025703e |
|
(0045888) hudsonbot (developer) 2012-03-02 15:38 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2012-02-18 01:31 | dbaz | New Issue | |
| 2012-02-18 01:31 | dbaz | Assigned To | => dbaz |
| 2012-02-18 01:31 | dbaz | Modules | => Core |
| 2012-02-18 01:32 | dbaz | Proposed Solution updated | |
| 2012-02-18 01:35 | dbaz | Note Added: 0045250 | |
| 2012-02-18 01:36 | dbaz | File Added: ob-pick-and-execute-view.js | |
| 2012-02-18 01:38 | hgbot | Checkin | |
| 2012-02-18 01:38 | hgbot | Note Added: 0045251 | |
| 2012-02-18 01:38 | hgbot | Status | new => resolved |
| 2012-02-18 01:38 | hgbot | Resolution | open => fixed |
| 2012-02-18 01:38 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/c5f71c8919355f5610a1464d9b7f2883bc365778 [^] |
| 2012-02-20 11:01 | AugustoMauch | Note Added: 0045261 | |
| 2012-02-20 11:01 | AugustoMauch | Status | resolved => closed |
| 2012-02-20 11:01 | AugustoMauch | Fixed in Version | => 3.0MP9 |
| 2012-03-02 15:38 | hudsonbot | Checkin | |
| 2012-03-02 15:38 | hudsonbot | Note Added: 0045888 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |