Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0018761
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] B. User interfaceminoralways2011-10-11 18:272011-11-04 13:58
Reporterdmitry_mezentsevView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revisionf9b05776d657
ProjectionnoneETAnoneTarget Version3.0MP5
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0018761: There is no way to cancel new record in the form view using menu buttons

DescriptionNow if you press new in a form and then want to get rid of this new record you need to press Cross icon on the form itself or Esc on the keyboard but Cancel button in the menu is not active (see screenshot).

It is confusing because:
* I personally without this button usually start clicking on another one - save and close and get an error as a response.
* If the record receives all required for it to be saved parameters then pressing Cross does not delete it but closes and saves it. That is a quite a different behavior to cancelling the record by the same button (see above) if no info is introduced into the form.

I also mentioned that the same scenario in grid works as I would expect it :-)
* Cancel button appears.
* If grid contained Cancel button
Steps To Reproducelive.builds.pi (11.10.2011)
* New Sales Invoice
* Mention that Cancel button is not active (see screenshot).
* Enter BP.
* Cancel button appears but if you press it only BP field is cleaned but record is still present.

Repeat the same steps in the grid to compare.
Proposed Solution* Cancel in a form should work like Cancel in a grid for freshly created New form and for the form with some parameters entered - delete it if was not saved before.
* Close cross in a form should work like "Save and close" button in a menu.
TagsNo tags attached.
Attached Filespng file icon 1034311.png [^] (84,311 bytes) 2011-10-11 18:27


png file icon littleMissX.PNG [^] (1,121 bytes) 2011-10-14 10:41


png file icon saveandclose.PNG [^] (75,436 bytes) 2011-11-02 13:39


png file icon close.PNG [^] (44,480 bytes) 2011-11-02 13:40

- Relationships Relation Graph ] Dependency Graph ]
has duplicate defect 00188513.0MP5 closedmtaal Save buttons must be disabled when you can´t save 

-  Notes
(0041821)
rgoris (developer)
2011-10-14 09:36

Agree with Dmitry. Although technically creating a new form does not have any data to be canceled yet (as no fields have been filled in yet), for the user canceling means "abort my current action and go back to where i came from". Pressing the Cancel button to kill a new form is very obvious behavior that must be supported.
(0041822)
rgoris (developer)
2011-10-14 09:53

So here is the proposed solution
1) let Cancel button kill the new window
2) keep Close button as it is now (kills the new window)
3) let Save&Close button also kill the new window (note that only the little X should be enabled, not the floppy disk icon)
4) Disable the Save button (until something really has changed in the form)
(0041825)
dmitry_mezentsev (developer)
2011-10-14 10:37

Rob,

Sorry, don´t fully understand you.
>1) let Cancel button kill the new window
>3) let Save&Close button also kill the new window (note that only the little X should be enabled, not the floppy disk icon)
In this case we will have 2 buttons enabled for doing the same? Why? Why couldn't we let Save&Close button to work like it is now but just not show it (as you mentioned) if there are no changes in the form?
(0041828)
rgoris (developer)
2011-10-14 10:43

Dmitry, the button Save&Close has two appearances: save&close (both floppy and x are enabled) and close (only the x is enabled, see image attached).

In the x-only case it does the same as the big X (close button) in the top right of a window. Right now, this also kills a new window, which is good. So the little x should do the same.

Also, as YOU told me you always clicked the save&close button, we´ll have one more happy user :-)
(0041832)
dmitry_mezentsev (developer)
2011-10-14 11:27

> Also, as YOU told me you always clicked the save&close button, we´ll have one more happy user :-)
I was primarily clicking on it because Cancel button was not there. If Cancel is there and Save&Close is disabled :-) - I would not.

But what is strange for me as I mentioned above is to have 2 active buttons just near to each other (Save&Close and Cancel) that do the same. For me not very clean but won´t fight for this.
(0041833)
rgoris (developer)
2011-10-14 11:43

Could be but does not do any harm.

More important is that the big X (top right of window) and the small x (toolbar button)always do the same.
(0041841)
dmitry_mezentsev (developer)
2011-10-14 15:48

>1) let Cancel button kill the new window
>3) let Save&Close button also kill the new window (note that only the little X should be enabled, not the floppy disk icon)

Did not want to continue but has just came across the case where it does not work OK.
There are some forms which after pressing new contain all the required info to be saved without the need to enter info (ex, Financial Account - Payment Method tab).
In this case Cancel as killing such form - makes sense, but Save&Close button will undermine its own meaning if it also kills it. So we should take this case into account in deciding where Save - cancels and where Save - saves :-).
(0041993)
mtaal (manager)
2011-10-23 00:13

Rob,
Any final comments on this one?

gr. Martin
(0042070)
rgoris (developer)
2011-10-24 15:57

Dmitry/Martin

Save&Close button can have two states:
1) It saves and closes (this is when both floppy disk and x are enabled in the button graphic)
2) It only closes (this is when only the x is enabled in the button graphic) because there is nothing to save (no changes). This does exactly the same as the Close button (X).

So in case of Financial Account - Payment Method tab (new record that has all the necessary field values at the start) the Save&Close button will be a real Save&Close button (with both floppy and x enabled in the graphic). To cancel you would click the Cancel button (or the big X close button top right). To save&close you would click the save&close button.

Note that clicking the big close (X) in the top right of the form works as follows: it always saves & closes UNLESS no changes have taken place: in this case it kills the window.
(0042084)
mtaal (manager)
2011-10-25 10:23

Changes to do:
- save button should be disabled if there are no changes
- save-close should be in close status if there are no changes
- cancel should be enabled if there are no changes, in this case it should abort the form.
(0042125)
hgbot (developer)
2011-10-26 19:36

Repository: erp/devel/pi
Changeset: f9b05776d6570f5074ff4e112b27ed44bf0f1969
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Oct 26 19:33:38 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f9b05776d6570f5074ff4e112b27ed44bf0f1969 [^]

Fixes issue 18761: There is no way to cancel new record in the form view using menu buttons
Changed when the cancel button is enabled (always in form mode) with different behavior
Only enable save if all required fields have been set and there are no errors in the form

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-statusbar.js
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/toolbar/ob-toolbar.js
---
(0042419)
rgoris (developer)
2011-10-31 17:26

Just tested on live builds pi and we´re almost there. Only (- save-close should be in close status if there are no changes) is not entirely doing what it should.

If you click the little x it should do the same as the big X (close the form even though it cant be saved)
(0042433)
hgbot (developer)
2011-11-02 10:26

Repository: erp/devel/pi
Changeset: 7445c86c656d83ff4075353bccfe10826442eabb
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Nov 02 10:25:34 2011 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/7445c86c656d83ff4075353bccfe10826442eabb [^]

Related to issue 18761: There is no way to cancel new record in the form view using menu buttons

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/toolbar/ob-toolbar.js
---
(0042445)
marvintm (developer)
2011-11-02 13:20

I see a problem with these changes: there are windows in which even if no change was done by the user after creating a record, the current values are correct.

In this case, the system behaves inconsistently. For example, in the Sales Invoice Lines tab, you can create lines without specifying anything. If you create a new line and then click on the close button, the line will not be saved. However, if you create the line, then change the quantity from 1 to 2, and then click on close, the line will be saved.

This is not consistent in my opinion.
(0042448)
rgoris (developer)
2011-11-02 13:38

marvintm: if you create a new record and the defaults are good (to be saved) and you hit the save&close button, then it will be saved. Note that the button will both have the floppy (save) and the x (close) enabled.

So, summarizing
If the form can´t be saved: only close (only little x enabled)
If the form can be saved: save&close (both little x and floppy enabled)

If only the little x of the save&close button is enabled, then its behavior is exactly the same as when clikcing the big X close button in the top right.
(0042450)
rgoris (developer)
2011-11-02 13:40
edited on: 2011-11-02 13:41

see attached images for the difference. In the last case (close) it should do the same as the close (big X) in the top right of the form, which is abort/cancel, return to grid without creating anything

(0042451)
marvintm (developer)
2011-11-02 15:58

I didn't explain myself well enough. After the changes Martin did, the following happens:

- If you create a Sales Invoice Line in form view, and immediately click on the gray "x" button (the one shown in the first screenshot), the line is _not_ saved
- If you create a Sales Invoice Line in form view, change the quantity to 2, and click on the gray "x" button, the line IS saved.

This behaviour complies with the description Martin wrote in the comment 0042084, but in my opinion is clearly inconsistent, because the line was perfectly correct in both cases, and therefore it should have been saved in both.
(0042456)
rgoris (developer)
2011-11-02 17:03

Correct but this has is related to the behavior of the form-close (big gray X) button and not the little x.

The little x now tries to save, this is not good. So first let´s fix this behavior (last image). The little X must cancel/abort the form creation.

The other case you mention can be fixed later, can you open a separate issue for that?
(0042571)
hudsonbot (developer)
2011-11-04 13:56

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/d10da320df2a [^]

Maturity status: Test
(0042618)
hudsonbot (developer)
2011-11-04 13:58

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/d10da320df2a [^]

Maturity status: Test

- Issue History
Date Modified Username Field Change
2011-10-11 18:27 dmitry_mezentsev New Issue
2011-10-11 18:27 dmitry_mezentsev Assigned To => rgoris
2011-10-11 18:27 dmitry_mezentsev File Added: 1034311.png
2011-10-11 18:27 dmitry_mezentsev Modules => Core
2011-10-14 09:36 rgoris Note Added: 0041821
2011-10-14 09:36 rgoris Assigned To rgoris => mtaal
2011-10-14 09:36 rgoris Status new => scheduled
2011-10-14 09:36 rgoris Target Version => 3.0MP5
2011-10-14 09:53 rgoris Note Added: 0041822
2011-10-14 09:54 rgoris Issue Monitored: rgoris
2011-10-14 10:37 dmitry_mezentsev Note Added: 0041825
2011-10-14 10:41 rgoris File Added: littleMissX.PNG
2011-10-14 10:43 rgoris Note Added: 0041828
2011-10-14 11:27 dmitry_mezentsev Note Added: 0041832
2011-10-14 11:43 rgoris Note Added: 0041833
2011-10-14 15:48 dmitry_mezentsev Note Added: 0041841
2011-10-23 00:13 mtaal Note Added: 0041993
2011-10-23 00:13 mtaal Status scheduled => feedback
2011-10-24 15:57 rgoris Note Added: 0042070
2011-10-25 10:23 mtaal Note Added: 0042084
2011-10-25 10:23 mtaal Status feedback => new
2011-10-26 18:59 mtaal Relationship added has duplicate 0018851
2011-10-26 19:36 hgbot Checkin
2011-10-26 19:36 hgbot Note Added: 0042125
2011-10-26 19:36 hgbot Status new => resolved
2011-10-26 19:36 hgbot Resolution open => fixed
2011-10-26 19:36 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/f9b05776d6570f5074ff4e112b27ed44bf0f1969 [^]
2011-10-31 17:26 rgoris Note Added: 0042419
2011-11-02 10:26 hgbot Checkin
2011-11-02 10:26 hgbot Note Added: 0042433
2011-11-02 13:20 marvintm Note Added: 0042445
2011-11-02 13:38 rgoris Note Added: 0042448
2011-11-02 13:39 rgoris File Added: saveandclose.PNG
2011-11-02 13:40 rgoris File Added: close.PNG
2011-11-02 13:40 rgoris Note Added: 0042450
2011-11-02 13:41 rgoris Note Edited: 0042450 View Revisions
2011-11-02 15:58 marvintm Note Added: 0042451
2011-11-02 17:03 rgoris Note Added: 0042456
2011-11-03 10:03 rgoris Status resolved => closed
2011-11-04 13:56 hudsonbot Checkin
2011-11-04 13:56 hudsonbot Note Added: 0042571
2011-11-04 13:58 hudsonbot Checkin
2011-11-04 13:58 hudsonbot Note Added: 0042618


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker