LibreOffice 4.0.2.2 on Windows XP: 1. In cell A1 type in: 100 2. In cell A2 type in: =A1*10 3. Click on cell A2 (there is a formula and will make it easier for next steps). 4. Tools | Goal Seek. 5. Formula cell contains $A$2 (because of step 3). Leave this field as default. 6. Target value type in: 2000 7. Variable cell type in: A1 8. Click on OK button. 9. Dialog appears with text: "Goal Seek successful. Insert result (200) into current cell?" Now the problem: Dialog reports that it would like to insert result in "current cell" witch in my case is A2. The problem is the term "current cell". I understand term "current cell" the cell that is bordered (marked) and the name is displayed in Name Box (drop-down selection above cell A1). In my case "current cell" is A2 (from step 3). This message "current cell" is wrong. Because value is ALWAYS inserted in "Variable cell", not "current cell". Expected: I think dialog instead of: Goal Seek successful. Insert result (200) into current cell? should display something like: Goal Seek successful. Insert result (200) into Variable cell?
Hi, Thanks for reporting. I totally agree with your comments about the "Current cell" label. Therefore I mark this as NEW. I mark this bug as trivial low following our prioritizing flowchart https://wiki.documentfoundation.org/images/0/06/Prioritizing_Bugs_Flowchart.jpg Kind regards, Joren
Maybe we should even remove any reference to "type" of cell where data will be inserted. Probably better instead of: Goal Seek successful. Insert result (200) into current cell? use: Goal Seek successful. Result (200). Apply setting to spreadsheet?
Patch is under review: https://gerrit.libreoffice.org/#/c/3291/
(In reply to comment #2) > Goal Seek successful. Result (200). Apply setting to spreadsheet? I don't like the word 'setting' What about " Apply entry to spreadsheet? " or "Apply value to spreadsheet? " or simply "Apply to spreadsheet? "
@Cor Nouws, I understand you point and I agree with you. "Apply to spreadsheet?" --> I love this one, it is simple and tells user what will happen. I vote for this. Other two: - "entry": it is actually not an entry it is calculated value, - "value": no need for additional word
One more detail we should probably talk about. When current dialog "Goal Seek successful..." appears it offers two buttons Yes/No. I don't like Yes/No questions. It forces user to read all of the dialog text and it misunderstood it can lead to undesired results. My suggestion is: 1. change Yes button label to: Apply 2. change No button label to: Cancel
Joren De Cuyper committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d06180200195d4c20a58d989f9d2c16cb1989688 fdo#63290 UI: Goal Seek - incorrect term "current cell" The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
The commit does not take comment 4 into account at all.
(In reply to comment #8) > The commit does not take comment 4 into account at all. Yes, I see that. I didn't had the time to push another change for review. I only pushed my change to gerrit to ping the UX advice list. After that I saw Cor proposed a better change. Until now next version of 4.0.x version will not be affected (only 4.1 master version), so no worries about that :-). I'm sorry for the noise and I'll push the right string changes to gerrit soon. Kind regards, Joren
Reaction of Stefan Knor at the patch currently under review (https://gerrit.libreoffice.org/#/c/3338/): Hm, that message sounds a bit odd. How about Goal seek successful. Insert result ($result) into spreadsheet? Any ideas/agreements/improvements/...?
Thanks for adding that in there :) grofaty, can you explain why we should change the verb from "insert" to "apply"? I admit, "insert" is maybe not the most fitting verb, but "apply" seems worse. Maybe an alternative: "replace value"? Looking at this more closely, why should we use the term "spreadsheet"? The obvious term seems to be to use "variable cell", no? This is the term that is already used in this dialogue for this particular cell. I don't think we need emphasise when we're successful – it can be implied. So, new proposal: _____________________________________________________________________ LibO Calc Result of goal seek: $result [line break] [line break.] Do you want to insert the result into the variable cell? [ Insert ] [ Cancel ] _____________________________________________________________________ Joren, as you are at it: could you also look into updating the dialogue of a failed goal seek? It's pretty easy to get that, e.g. by using =(A1*RAND()) as the formula. Proposal for that: _____________________________________________________________________ LibO Calc Goal seek unsuccessful. [line break] [line break.] Do you want to insert the closest result ($result) into the variable cell anyway? [ Insert ] [ Cancel ] _____________________________________________________________________
@Astron, I understand your point. My main complain was "current cell", so using "variable cell" is more appropriate. Lets use your "Insert" suggestion instead of my suggestion "Apply", it sounds to me as a better solution. I love the solution you have provided (line brakes etc). Maybe just one additional suggestion, instead of "Do you want to insert..." just use "Insert..." so without "Do you want to". The logic would be the same as for example closing document without saving, it is displayed message "Save changes to..." and not "Do you want to save changes...". Less words to read the better and also to have whole product the same question logic. One additional suggestion. When Goal Seek dialog opens there is no icon. If you look at close without saving dialog there is question icon. I think we should add the "question" icon to the left of the dialog. It looks more professional.
(In reply to comment #11) > Thanks for adding that in there :) > > grofaty, can you explain why we should change the verb from "insert" to > "apply"? I admit, "insert" is maybe not the most fitting verb, but "apply" > seems worse. > Maybe an alternative: "replace value"? It isn't that easy to convert the action buttons of the dialog. I pushed a reworked patch to gerrit for review which doesn't change anything to the buttons. They are "hard coded" and depends on the MessBox. There are no RET_INSERT buttons, so this wouldn't be an easy conversion. I'll have a look at it later. > So, new proposal: > _____________________________________________________________________ > LibO Calc > > Result of goal seek: $result [line break] > [line break.] > Do you want to insert the result into the variable cell? > > [ Insert ] [ Cancel ] > _____________________________________________________________________ > What about the "Goal Seek successful" line? I didn't delete it on my current patch. (In reply to comment #12) > I love the solution you have provided (line brakes etc). Maybe just one > additional suggestion, instead of "Do you want to insert..." just use > "Insert..." so without "Do you want to". Done > One additional suggestion. When Goal Seek dialog opens there is no icon. I'll also have a look at this. But this may also be a property of the MessBox, so mabye this isn't altered easy too.
(In reply to comment #13) > What about the "Goal Seek successful" line? I didn't delete it on my current > patch. I don't think we absolutely need this – there's a result, so we're successful. (But maybe I am assuming too much – we can keep this in.) One thing I'd change from yesterday night is that I'd actually replace the "unsuccessful" with "failed." But it should be alright anyway. Ugh ... I notice, something else went wrong with my failure message: "insert the closest result" is of course bogus if we don't have a result, that should remain value, as it was before.
Joren, I uploaded it my changes from the previous comment (hope that's ok), so we don't have to do too many roundtrips – if you're happy with this, let's just push it.
(In reply to comment #15) > Joren, I uploaded it my changes from the previous comment (hope that's ok), > so we don't have to do too many roundtrips – if you're happy with this, > let's just push it. +1
Joren De Cuyper committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=af4a947174aafe669c129c54c304caa43d90eac7 fdo#63290 - UI: Goal Seek - incorrect term "current cell" The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
I think we can close this one. I'll still have a look at the buttons; but that wasn't the initial problem for this bug report. Therefore I mark this as RESOLVED FIXED, and will have a look at the buttons later. Kind regards, Joren
Lior Kaplan committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=dbc3b2544acffcf9d8f1f14c3b6fce5c80e0493e fdo#63290 related: cleanup stings defines due to removal The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Migrating Whiteboard tags to Keywords: (ProposedEasyHack -> needsDevEval, SkillCpp ) [NinjaEdit]