|
Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
| Summary: | Tools>Customize>Keyboard: Wrong "Can't" in explication for CTRL+Z, CTRL+Y | ||||||
|---|---|---|---|---|---|---|---|
| Product: | General | Reporter: | raindrops <na1000> | ||||
| Component: | ui | Assignee: | AOO issues mailing list <issues> | ||||
| Status: | CONFIRMED --- | QA Contact: | |||||
| Severity: | Trivial | ||||||
| Priority: | P3 | CC: | avagula, issues, jsc, mseidel, pescetti, rafaella.braconi, rainerbielefeld_ooo_qa, samalipraveena | ||||
| Version: | OOo 2.0.2 | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | All | ||||||
| OS: | All | ||||||
| See Also: | https://bz.apache.org/ooo/show_bug.cgi?id=84060 | ||||||
| Issue Type: | DEFECT | Latest Confirmation in: | 4.1.0-dev | ||||
| Developer Difficulty: | Simple | ||||||
| Issue Depends on: | |||||||
| Issue Blocks: | 84060 | ||||||
| Attachments: |
|
||||||
|
Description
raindrops
2006-04-12 15:33:08 UTC
Reproducible. Reassigned. This affects also localizations. and obviously wrong component, as this affects all applications, not only graphics This issue blocks ET translation issue 84060 cd: We want to revise the accelerator configuration dialog for OOo 3.0. I will take care of this issue. Can we have these 2 strings fixed for 2.4? cd->coni: I am sorry but we already passed UI and translation freeze date. cd->pb: Please take over. The default strings for both commands should be changed to "Undo" and "Restore". It's an old problem that we use something like "Can't "+command to reflect the initial state that both commands are not enabled. I think that we should change that. Please find a solution with user experience for OOo 3.0. vagula->cd: there is some misunderstanding.. the list in customize dialog/keyboard seems to be static and I dont believe that strings in this dialog are reused somehow, at least playing around with different states of ondo/redo possibilites does not change anything in this dialog (btw, one notice more: shortcut ctrl-shift-y has no assignment in keyboard-area, but it really has one - Repeat) pb: to busy to fix in 3.0 -> 3.x. Please take over Still Reproducible with server installation of "AOO 4.1.0-dev – English UI / German locale - [AOO410m14(Build:9760) - Rev. 1583418_2014-04-02]" on German WIN7 Home Premium (64bit)", own separate user profile. Created attachment 83948 [details]
Patch to update GenericCommands.xcu file
Modified GenericCommands.xcu file as mentioned in Bugzilla - removed can't from both Undo and Restore.
Thanks, works for me in that context, but it has side-effects. "Undo" is not an ordinary menu item, since it changes to "Can't Undo" when the undo operation is impossible (for example, immeditely after you start OpenOffice the "Edit" menu says "Can't Undo"). One would have to check where this "Undo" -> "Can't Undo" replacement is done, and see if it possible to fix the command in this context (i.e., Tools - Customize - Add - Edit) while still preserving the dynamic "Undo"/"Can't Undo" change. Alternatively, we could drop the dynamic "Undo"/"Can't Undo" change altogether: but still this requires that we find and disable the code that dynamically changes it and that we quickly discuss and approve the change on the dev list. Thank you so much for your review. Yes, you're right. I thought the code i changed would only affect customize dialog part, but it changed the edit->Undo as well. I too noticed that it changes the edit->Undo/can't Undo option as well, but it is disabled (pale text, not bold one) when undo can't be selected. Once the user types something, it changes to bold. So i left it as such. Even if the user selects it, nothing will happen. Will this not be enough? The string is used in all user interface elements, that is, menu items and toolbar items. IMHO it is a good indicator to the user that she/he cannot Undo/Redo, and I guess this is used by screenreaders and other accessibility tools where users cannot see the icons being disabled. I modified can't undo to undo in two places in GenericCommands.xcu file. one in node Undo, and another in Undoaction. Maybe i'll try changing one back to Can't Undo. Many menuitems like cut, copy, paste are disabled when you can't do these operations. Only when you type in something do they get changed to bold again. disabled->enabled and vice versa. Their text aren't changing to Can't cut->cut or vice versa. Hi Andrea, I think the change i did is correct. This .xcu file is the only place from where both edit menu (take Can't Undo) and customize dialog take the text for Undo (Can't Undo) and Can't Restore. If you change it here, it will affect both edit menu and customize dialog. However, the code is behaving correctly. As you said,"One would have to check where this "Undo" -> "Can't Undo" replacement is done, and see if it possible to fix the command in this context (i.e., Tools - Customize - Add - Edit) while still preserving the dynamic "Undo"/"Can't Undo" change." The original code behaves like this: the edit menu is not changing from "Undo" -> "Can't Undo" every time. Instead, it is disabled "Can't Undo" in the beginning because of the "Can't Undo" text in .xcu file. Then, when you type in some text, it changes to enabled Undo "typing that text". when you remove the text, it changes to disabled "Undo typing that text" enabled "Redo typing that text". So my fix is working correctly. No need for any more code change. on the first look it seems to be ok, to simply change the xcu file. The menu entry is disabled and changed afterwards as you describe. But Ariel's comment is valid and very important and have to be checked before we change anything. Reset the assignee to the default "issues@openoffice.apache.org". |