very, very small problem with location of buttons in find and replace

Kornel Benko kornel at lyx.org
Tue Mar 18 07:49:04 UTC 2025


Am Mon, 17 Mar 2025 23:27:09 +0100
schrieb Pavel Sanda <sanda at lyx.org>:

...
> BTW Kornel, when I tried to test this in master, it seems that Adv Find is broken there.
> If I just follow steps in MWE of the bug above the search errorneously triggers
> the dialog of reach end of document, although it should just land on the first 
> occurence. Can you check?
> 
> Pavel

I have no idea. Bisect leads to
commit 9c443d96
Author:     Jean-Marc Lasgouttes <lasgouttes at lyx.org>
AuthorDate: Wed Sep 4 15:48:38 2024 +0200
Commit:     Jean-Marc Lasgouttes <lasgouttes at lyx.org>
CommitDate: Thu Sep 5 16:57:35 2024 +0200

    Cleanup Page Up/Down code
    
    Since BufferView::scroll() does not correct scrolling amount at
    top/bottom anymore, remove its return value and add a new one to
    BufferView::updateMetrics(bool) that returns this correction.
    
    Rely on updateMetrics(false) to set the metrics straight after moving
    the anchor y position.
    
    Use assert_in_view = false when calling TextMetrics::editXY() because
    this parameter does not work as advertised: if an inset not totally
    visible, the code will not try to go inside it to look for a smaller
    row that is totally visible.
    
    Two small (hopfully useful) modifications:
    - take into account the cursor X target instead of its screen position
    - adapt the cursor position depending on
      lyxrc.mac_like_cursor_movement when hitting start or end of
      document.

src/BufferView.cpp
src/BufferView.h

	Kornel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: Digitale Signatur von OpenPGP
URL: <http://lists.lyx.org/pipermail/lyx-devel/attachments/20250318/82a3bfb2/attachment.sig>


More information about the lyx-devel mailing list