Apache OpenOffice (AOO) Bugzilla – Issue 49491
Double click on Calc statusbar does not show Navigator
Last modified: 2013-08-07 15:14:30 UTC
If you click on left side of statusbar (where is Page 1/3 text) in Writer You can reach the Navigator. If you click on left side of statusbar (where is Sheet 1/3 text) in Calc You can't reach the Navigator. It may be a bug? Isn't it?
It isn't. Writer is special case. Calc isn't page oriented but Writer is, so the Navigator opens as you can easily navigate the pages with it. Calc is Sheet based and therefore each Sheet is rechable by it's sheettab. Nvertheless I re-assign this to the requirements team as enhancement request. Frank
Navigator in Calc as use ful as in Writer :o) Same element in different part of OOo should work as in others...
Can someone hep out where can I found this part in the Writer?
Hello fans! sc\source\ui\view\tabvwsh3.cxx has this lines: case SID_STATUS_DOCPOS: { //! Navigator an-/ausschalten (wie im Writer) ??? //!GetViewData()->GetDispatcher().Execute( SID_NAVIGATOR, //! SFX_CALLMODE_SYNCHRON|SFX_CALLMODE_RECORD ); } break; it is very same in Writer: sw\source\ui\uiview\view2.cxx case FN_STAT_PAGE: { GetViewFrame()->GetDispatcher()->Execute( SID_NAVIGATOR, SFX_CALLMODE_SYNCHRON|SFX_CALLMODE_RECORD ); } break; Does anyone know anything why we not uncomment these lines in the Calc's source?
timar -> for FSF.hu build
Created attachment 34046 [details] Patch to enable Navigator from statusbar - I will test it in my builds
Hi, if you want this integrated, please write a small specification and get the approval for it from QA, Development and User Experience. Frank
Created attachment 34056 [details] draf version of specification - I hope it is enough :o)
I guess it is OK, although it only consists of an abstract. Please post to dev@specs.openoffice.org to get some people to approve it (or let them point out what is missing). Exp. I doubt that the people will accept you as representative of User-Experience, but why not give it a try :-) And it not reenabling the stuff (it was commented out from the very beginning..) PS: Did you know OOo has a spell-checker? <evil grin/>
FL-kami_: I really appreciate your work to get this change into the product. I think this change is reasonable, but unfortunately the current specification is incomplete and ambiguous. The "Abstract" section of a spec. has to contain just an abstract of the content not the detailed solution. The abstract gives the reader an idea about the content. Furthermore this section will be automatically processed to generate a What's New Guide, to document changes made in a version. The spec. should be added to the specs.openoffice.org project to be processed for the guide and it makes it easier to change something in the spec. without the need to create a new attachment, when the sec. is just linked to the issue and vice versa. So please add a "Detailed Specification" part to the specification and describe the solution/change in this part. I.e. adding a screenshot of the status bar with a marked double click area makes the description more unambiguous.
Added myself to cc list.
So, is the only thing holding this patch back is the spec?
Kohei -> Would you review the patch and fix it if you need? Also I have created a full spec. with more goals. I posted it to UX team. I will attach here If you want.
@kami_: sorry I didn't get back to you earlier. I tested your patch, but merely uncommenting those line will not launch the navigator by double-clicking the sheet status area. The code path never reaches there. This means two things: 1) that SID_STATUS_DOCPOS block can be safely removed for the sake of code cleanup. :-) 2) find another way to launch the navigator. (yikes!) Kohei
Created attachment 50134 [details] patch that works
setting issue type to PATCH.
It turns out that the .uno:StatusDocPos command gets dispatched to ScCellShell::Execute, not ScTabViewShell::Execute. So you have to put the handler code there to make it work.
re-assigning it to nn so the issue won't get lost.
Created attachment 50135 [details] a little cosmetic change to make it look more blended in.
Created attachment 50136 [details] Draft specification for statusbar behaviors - passed to UX list
nn: Can you integrate the Kohei's patch. I think the best way to open new issue for the advised other changes. KAMI
I added the patch, for the navigator on the sheet number field, to CWS "dr68". There's issue 99730 about right-click on the page styles.
reassigning to QA for verification
Created attachment 63081 [details] TestCaseSpecification
verified in onternal build cws_dr68
Verified in DEV300m53 on WinXP Closing