I noticed this post and decided to investigate: https://www.reddit.com/r/libreoffice/comments/sdycms/72_changed_where_i_land_when_using_navigation/ 1. Add heading 2. Double-click it in the Navigator The view scrolls, so the heading is in the middle. A bibisect with linux-64-7.2 points to https://git.libreoffice.org/core/commit/1df2581cfffc87386c6de7614793b2d664244e5a tdf#137838 SW: Add ability to create a hyperlink to a drawing object Adding Cc: to Jim Raykowski Was the change intentional or accidental?
Hi Buovjaga, I was not able to repro following the STR but looking at the bibisected patch I see what happened: -// force scroll to cursor position when navigating to inactive document -if(!bSel) +// assure visible view area is at cursor position + if (!m_pActiveShell->IsCursorVisible() && !m_pActiveShell->IsFrameSelected() && !m_pActiveShell->IsObjSelected()) The GotoOutline function brings the heading into view so it doesn't get force scroll to the top. Easy enough to revert to previous behavior: https://gerrit.libreoffice.org/c/core/+/129129
Thanks for the quick response. I confirm the patch puts back the old behaviour.
Jim Raykowski committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/b0434f62c17b69e10caf05ad85a60f1bffdb5238 tdf#147041 Always show navigated heading at top of visible view area It will be available in 7.4.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Should we cherry pick it to 7-3 so it gets into 7.3.1?
(In reply to Buovjaga from comment #4) > Should we cherry pick it to 7-3 so it gets into 7.3.1? Sure, I'll try. This is the first time I've cherry picked using gerrit. I've did it locally for fun. https://gerrit.libreoffice.org/c/core/+/129080
Jim Raykowski committed a patch related to this issue. It has been pushed to "libreoffice-7-3": https://git.libreoffice.org/core/commit/c6c343db72a97bb027ccb493cef6fa7f1025f119 tdf#147041 Always show navigated heading at top of visible view area It will be available in 7.3.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.