[Libreoffice-commits] core.git: sw/source
Jim Raykowski (via logerrit)
logerrit at kemper.freedesktop.org
Fri Jan 15 00:03:39 UTC 2021
sw/source/uibase/uiview/viewmdi.cxx | 36 +++++++++++++++---------------------
1 file changed, 15 insertions(+), 21 deletions(-)
New commits:
commit 9598018e67dfc10c35022cbec4844363b7dd98ab
Author: Jim Raykowski <raykowj at gmail.com>
AuthorDate: Sat Dec 26 13:17:23 2020 -0900
Commit: Jim Raykowski <raykowj at gmail.com>
CommitDate: Fri Jan 15 01:02:58 2021 +0100
Make page navigation in move navigation handler consistent
Page navigation in the move navigation handler scrolls the view without
moving the cursor or unselecting selections. This patch makes page
navigation that results in document wrapping, scroll the view without
changing the cursor position or selections. This is useful, for example,
when find all search is made and page viewing is wanted that does not
destroy cursor selections.
Change-Id: Ie062eeba9a29c2b8a42605ed842d60244cc74147
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108380
Tested-by: Jenkins
Reviewed-by: Jim Raykowski <raykowj at gmail.com>
diff --git a/sw/source/uibase/uiview/viewmdi.cxx b/sw/source/uibase/uiview/viewmdi.cxx
index bfa8ec038f8a..348a9a0d3621 100644
--- a/sw/source/uibase/uiview/viewmdi.cxx
+++ b/sw/source/uibase/uiview/viewmdi.cxx
@@ -343,32 +343,26 @@ IMPL_LINK( SwView, MoveNavigationHdl, void*, p, void )
switch( m_nMoveType )
{
case NID_PGE:
- if ( bNext )
+ {
+ if (USHRT_MAX == rSh.GetNextPrevPageNum(bNext))
{
- if ( USHRT_MAX == rSh.GetNextPrevPageNum( true ) )
- {
- rSh.GotoPage( 1, true );
- SvxSearchDialogWrapper::SetSearchLabel( SearchLabel::EndWrapped );
- }
- else
- {
- PhyPageDown();
- SvxSearchDialogWrapper::SetSearchLabel( SearchLabel::Empty );
- }
+ const Point aPt(GetVisArea().Left(),
+ rSh.GetPagePos(bNext ? 1 : rSh.GetPageCnt()).Y());
+ Point aAlPt(AlignToPixel(aPt) );
+ // If there is a difference, has been truncated --> then add one pixel,
+ // so that no residue of the previous page is visible.
+ if(aPt.Y() != aAlPt.Y())
+ aAlPt.AdjustY(3 * GetEditWin().PixelToLogic(Size(0, 1)).Height());
+ SetVisArea(aAlPt);
+ SvxSearchDialogWrapper::SetSearchLabel(bNext ? SearchLabel::EndWrapped :
+ SearchLabel::StartWrapped);
}
else
{
- if ( USHRT_MAX == rSh.GetNextPrevPageNum( false ) )
- {
- rSh.GotoPage( rSh.GetPageCnt(), true );
- SvxSearchDialogWrapper::SetSearchLabel( SearchLabel::StartWrapped );
- }
- else
- {
- PhyPageUp();
- SvxSearchDialogWrapper::SetSearchLabel( SearchLabel::Empty );
- }
+ bNext ? PhyPageDown() : PhyPageUp();
+ SvxSearchDialogWrapper::SetSearchLabel(SearchLabel::Empty);
}
+ }
break;
case NID_TBL :
rSh.EnterStdMode();
More information about the Libreoffice-commits
mailing list