Bug 112437 - Go to page toolbar control
Summary: Go to page toolbar control
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
6.0.0.0.alpha0+
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: topicUI
Depends on:
Blocks: UNO-Command-New Navigator Go-To-Page
  Show dependency treegraph
 
Reported: 2017-09-17 00:45 UTC by Jim Raykowski
Modified: 2023-12-03 19:15 UTC (History)
10 users (show)

See Also:
Crash report or crash signature:


Attachments
screenshot of Go To Page toolbar control (11.47 KB, image/png)
2017-09-17 00:45 UTC, Jim Raykowski
Details
screenshot (135.74 KB, image/png)
2023-11-27 19:22 UTC, BogdanB
Details
demo of proposed toolbar control .uno:Go2Page (1.16 MB, video/x-matroska)
2023-12-01 10:26 UTC, Jim Raykowski
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jim Raykowski 2017-09-17 00:45:51 UTC
Created attachment 136292 [details]
screenshot of Go To Page toolbar control

I have made a Go To Page toolbar control. The attached screenshot shows it added  to the Findbar control through use of the customize option.
Comment 1 Heiko Tietze 2017-09-18 13:50:13 UTC
Great work, Jim. Guess everything was tracked under bug 79167 - otherwise you would have asked a question. So looking forward your next patch :-).

*** This bug has been marked as a duplicate of bug 79167 ***
Comment 2 Yousuf Philips (jay) (retired) 2017-09-18 17:00:16 UTC
A jump to page number toolbar control was created for the print preview toolbar (bug 80657), but it was a simple text field and not a spinbox. So please do create this control (might be useful if both toolbar controls shared code), though i'm doubtful that the find toolbar is a suitable place for it.

@heiko, @stuart, @cor: what is your take?
Comment 3 Jim Raykowski 2017-09-18 23:06:41 UTC
I created it to replace the spinbox page control in Navigator panel but like the 'Navigate By' and 'Previous/Next Navigate By' controls it will work in other toolbars. The Navigator panel toolbar is able to use this 'Jump to/Go to/Move to Page' control as well as the 'Navigate By' dropdown  and 'Previous/Next controls' that are part of the Find tool bar. I used the customize toolbar feature to add this control to the find bar for demo purposes.

Yes, I believe the current jump to page toolbar control only works in the print preview toolbar. I custom added to the Find tool bar and it didn't function as expected so I looked at the code and saw that it is intended for the print preview toolbar.
Comment 4 Yousuf Philips (jay) (retired) 2017-09-19 11:37:34 UTC
(In reply to Jim Raykowski from comment #3)
> I created it to replace the spinbox page control in Navigator panel

Sweet.

> I used the
> customize toolbar feature to add this control to the find bar for demo
> purposes.

Oh okay. It would be great to have it in .uno command form and functional everywhere, so that it can be added to any toolbars and hopefully it will wind up in one of the notebookbar implementations.

> Yes, I believe the current jump to page toolbar control only works in the
> print preview toolbar. I custom added to the Find tool bar and it didn't
> function as expected so I looked at the code and saw that it is intended for
> the print preview toolbar.

Thanks for the info.
Comment 5 Jim Raykowski 2017-09-27 08:00:11 UTC
Hello All,
 
I did a commit for this about a week ago at a time when gerrit was having troubles. The commit resulted in "No Builds Executed". There was a notice saying "If one of your Gerrit patch sets was affected by the purge, it should have received a "No Builds Executed" comment from Jenkins now. You can manually re-submit it by doing a rebase."

I read about git rebase and did some testing but would appreciate some help with this.

The link to the commit is
https://gerrit.libreoffice.org/42492
Comment 6 Jim Raykowski 2017-10-07 00:58:58 UTC
Thank you Yousuf for doing the rebase. Unless instructed different, I will wait for this to be merged before committing work done on tdf#89566 as it involves use of this control.
Comment 7 Xisco Faulí 2018-01-22 10:56:04 UTC
Dear Jim Raykowski,
This bug has been in ASSIGNED status for more than 3 months without any
activity. Resetting it to NEW.
Please assigned it back to yourself if you're still working on this.
Comment 8 Jim Raykowski 2018-05-10 02:18:17 UTC
  (In reply to Xisco Faulí from comment #7)
> Dear Jim Raykowski,
> This bug has been in ASSIGNED status for more than 3 months without any
> activity. Resetting it to NEW.
> Please assigned it back to yourself if you're still working on this.

Xisco, sorry about the delayed response. This wasn't accepted during review. There is already a toolbar control that does this using a different approach than the one I took.
Comment 9 BogdanB 2023-11-27 19:22:33 UTC
Created attachment 191068 [details]
screenshot

Is needed another way to jump to a certain page?
1). We have just a click and insert a number (like in the screenshot in the bottom).

2). We have in the Navigator the page number, so we can jump from one page to another very easy, or just insert the page number (like in the screenshot on the right).

Screenshot from
Version: 7.6.2.1 (X86_64) / LibreOffice Community
Build ID: 56f7684011345957bbf33a7ee678afaf4d2ba333
CPU threads: 16; OS: Linux 6.2; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded
Comment 10 Jim Raykowski 2023-11-27 21:10:10 UTC
+1 from me for closing this.
Comment 11 V Stuart Foote 2023-11-28 12:17:07 UTC
+1 resolved with current Navigator widget and "Go to Page..." dialog links from UI.
Comment 12 V Stuart Foote 2023-11-28 12:29:59 UTC
(In reply to V Stuart Foote from comment #11)
> +1 resolved with current Navigator widget and "Go to Page..." dialog links
> from UI.

Though Jim's work on https://gerrit.libreoffice.org/c/core/+/42492/ to establish an UNO command GO_TO_PAGE seemed on track. And took care of similar unfinished work on bug 80657 by Gülşah Köse in https://gerrit.libreoffice.org/c/core/+/19335/

@Jim the UNO for Toolbar, Menu or Keyboard assignment seems worthwhile...
Comment 13 Jim Raykowski 2023-12-01 02:52:35 UTC
Working WIP patch of renewed effort:
https://gerrit.libreoffice.org/c/core/+/160191

One benefit of this control is it is able to replace the go to page control in the Writer Navigator.
Comment 14 Jim Raykowski 2023-12-01 10:26:25 UTC
Created attachment 191175 [details]
demo of proposed toolbar control .uno:Go2Page
Comment 15 V Stuart Foote 2023-12-01 13:27:08 UTC
(In reply to Jim Raykowski from comment #14)
> Created attachment 191175 [details]
> demo of proposed toolbar control .uno:Go2Page

Very cool!  Any advantage to having each instance (TB, Findbar, SB Navigator) in sync?  I guess probably not, how often would user have 2 or 3 of the flavors of the control enabled... normally only going to be 1.
Comment 16 Heiko Tietze 2023-12-01 13:31:00 UTC
Not a fan of cluttering the quick find bar. Toolbar and Navigator are fine, and customization capability too, of course.
Comment 17 V Stuart Foote 2023-12-01 13:48:03 UTC
(In reply to Heiko Tietze from comment #16)
> Not a fan of cluttering the quick find bar. Toolbar and Navigator are fine,
> and customization capability too, of course.

Agree, but expect for the Find bar the GotoPage combobox would by default be not-visible, like the Navigate By Previous/Next control arrows are currently. But it would be there on the Visible button pane to enable if user prefers.

Perfect in the SB Navigator deck.

Coin flip on listing it as a not-visible button on the Standard TB by default...

In any case nice work, thanks Jim!
Comment 18 Jim Raykowski 2023-12-02 00:13:04 UTC
I added the control to the standard toolbar and findbar only for demonstration purpose to show that it can be added to any toolbar in Writer including toolbars in sidebar panels.

It would be nice to have Caolán's thoughts on the replacement of the current Navigator Go to Page control by this UNO control.
Comment 19 Caolán McNamara 2023-12-03 19:15:27 UTC
oh, I don't have any special insight here. I don't have any specific concerns if that's the question.