Cmake build: Make QT5-compilation be default

(cherry picked from commit c971974f331c249fa4d57953d8d93c107c154024)

Merge branch '2.4.x' into 2.4.1-devel

Put new citation after selected item rather than at end of list.

Fixes bug #12940.

Patch from Daniel.

(cherry picked from commit 40ae8644f9f81576033ff1f8f56428946029b4f4)

XHTML: export paragraph IDs also for list items, but not for the opening tag of the list.

The IDs are used to link to specific elements (for instance, the User's guide nomenclature).

The beginning of the list (<ul> or <ol>) has the same magic ID as the first item of the list.

Make comment clearer.

InsetInfo: no need to output + explicitly in a shortcut, it is already contained in the sequence.

One more

Status for last few commits

Fix display of math hull inset in RTL context

The inversion of margin was done in the wrong way for RTL support.

(cherry picked from commit b469c142ad579d3338fc81ab3e1c6ee7b4e270ec)

Fix bug #12934

(cherry picked from commit 6484dce51b46f8fc0134a7532efb08401bba3c53)

Set buffer when creating a MathData from parts of another one

This is triggered by selecting part of an equation under Xcb platform
(x11). Indeed, the current selection has to be sent to the X11 server
and BufferView::requestSelection() is called. Eventually
cap::grabSelection() is the function that creates this partial
MathData object.

Fixes ticket #12947.

(cherry picked from commit ed2f3cb9cfa1e871d48a92d41791a77bd218ab67)

Try to make sure that math insets have a properly set buffer_ member

Set the buffer of contents that is added to a MathData object through
MathData::insert() (both versions)
MathData::push_back()
asArray()

Also in math macros, initialize look_ with the relevant buffer.

This reduces the number of insets hat do not have a proper buffer.

See #13050 for discussion of this issue.

(cherry picked from commit f3c5ff9cb72c5231f1e1e81452e67d6f12dadecb)

Fix bug #11796.

Toolbars are now locked by default. Unlocked state should be saved
by Qt as a session variable.

Patch from Daniel.

(cherry picked from commit 42a8463aab6100e8209caa8cd137d3c500623729)

Fix bug #8856.

Show values of cross-references in tooltips.

(cherry picked from commit 2a1c3dce57da0ad25451fc61f413ebbeba2448fe)

Fix bug #12981

(cherry picked from commit 61181a39778ed5bcd7849ab388c402ab3f88d5cd)

Reset require_fresh_start_ after buffer has been viewed/exported

The LFUN buffer-reset-export states to start afresh the export for the
next run, but in fact, the boolean was never reset, so that this was
done for all subsequent runs unless the buffer had been closed and
reopened.

Now the lfun does what it states and only acts on the next pass.

(cherry picked from commit e1cb15ee5d9377cef6e3683a6d0bb68eb9b379c1)

Update sk.po

Amend 6c3447c8: FindAdv: sometimes a space is added on some math symbols

Restrict the removing to math insets

(cherry picked from commit f248b003a9f7b8b16c85c04ae37b3c5e27a4a8d6)

FindAdv: sometimes a space is added on some math symbols

For example '\int '.
Should fix regression #13070
Spotted  by Alexander Dunlap

(cherry picked from commit 4730ec78f544b6fd51defd063e52955554b14e89)
(cherry picked from commit 6c3447c8a62d1ed9c8e17732f991296123c69efb)

Begin 2.4.1 development

Minor update

Amend 4730ec78: FindAdv: sometimes a space is added on some math symbols

Restrict the removing to math insets

New status file, and status for last commit

FindAdv: sometimes a space is added on some math symbols

For example '\int '.
Should fix regression #13070
Spotted  by Alexander Dunlap

Wininstaller Qt6.7 changed file name

(cherry picked from commit a6d0d7ea92b686fc102b05970830ee11fc51e47b)

Remerge strings.

New 2.4.0 release

Cmake build: Initialize variable 'CMAKE_INSTALL_PREFIX' before use

Spotted by Eugene Chornyi <yu_jin@lyx.org>

InsetNote: pass on inset-modify lfuns addressed at other insets (#13068)

(cherry picked from commit ae1ddcbda0615b742184f4263cd1ad550eb38c32)

Update status

Address Qt 6.7 deprecation warning

(cherry picked from commit 44633c7f729e79ea023f2a5ccbbdd0002411619f)

Initialize is_long_tabular before updateIndexes() (which employs it)

(cherry picked from commit 10b5f03d95dca8f3e8c225aa2deca4d473f8b169)

Update aastex63.layout according to https://journals.aas.org/aastexguide/#acknowledgments

(cherry picked from commit d3d8b3d748f03ebafd477570291264961e22c7c4)

*RELEASE-NOTES: add cprotect

Return a reference, not a copy.

Stupid mistake from 2009. Thanks to Scott and JMarc for tracking
it down.

(cherry picked from commit de5f63eeb398046027180162272cc83099fdef55)

Amend 207eaeee9071cb

The upstream encoding needs to be updated after this routine
(see comment in OutputParams.h:183ff.)

(cherry picked from commit 2d95f8fc160d6ecc63acb81ffac16a99339083b2)

Document NeedCProtect -1 InsetLayout option

Amend a716fb96b53447

(cherry picked from commit af59551b1b0c7d7e95c70ae4db156cba7ac0cefe)

NeedCProtect -1 also for InsetLayouts

(cherry picked from commit a716fb96b534475759799b195a7edaeb049fbcf6)

Document NeedCProtect -1

(cherry picked from commit ea004cdb28e9b440347a9487d127e55765940c79)

Amend 207eaeee9071cb

FragileFrame, as opposed to normal frame, supports cprotect (and needs
it in the usual circumstances)

(cherry picked from commit eeb3f9e719d15c4117a510e8a214d1c1b2c5a33c)
(cherry picked from commit 50cd148bd260222e30b6e429d135c45f8c2f1f14)

Introduce NeedCProtect -1 layout option

It turns out beamer frame does not allow \cprotect and errors if it is
used. Hence we need to prevent it in this context entirely.

(cherry picked from commit 207eaeee9071cb828a2ab7f4680f8ff92e379af8)
(cherry picked from commit f05fd787548e8b3082051b7d45bb4119f97a55dc)

Amend(1) 5eaa03a1, Adapt to recognise AC_INIT empty buildtype

Cmake build: Adapt to recognise AC_INIT empty buildtype

Update build file

2.4.0!

      Close notes about status of translation

   Typo

    Translate documentation of NeedCProtect

Document NeedCProtect -1

Amend 207eaeee9071cb

FragileFrame, as opposed to normal frame, supports cprotect (and needs
it in the usual circumstances)

(cherry picked from commit eeb3f9e719d15c4117a510e8a214d1c1b2c5a33c)

Introduce NeedCProtect -1 layout option

It turns out beamer frame does not allow \cprotect and errors if it is
used. Hence we need to prevent it in this context entirely.

(cherry picked from commit 207eaeee9071cb828a2ab7f4680f8ff92e379af8)

Status

Consider CT in dash mechanism (#13062)

(cherry picked from commit fe4f0dbf0b37c413994bbfa780a833b1c041a433)

Remerge strings for 2.4.0 release

Update LFUNs

  Typo

    Last review of French Tutorial manual before 2.4.0

   Last review of French Math manual before 2.4.0

     Last review of French Math manual before 2.4.0

Set buffer_ correctly when inseting a math macro over a selection

The code that handles this special macro insertion (in
Cursor::mathMacroClose())) is very weird: it inserts the contents of
the macro _after_ the macro and later, since the macro is 'greedy', it
will eat this contents and put it in its first macro cell.

Does it make sense to you? It does not ake sense to me either. Anyway,
this is the point where one should make sure that the buffer is set.

Fixes ticket #12682.

(cherry picked from commit b45a957929a78cefc729a0fa984214eb83357d42)

            Last review of French Intro manual before 2.4.0

  Last review of French Customization manual before 2.4.0

    Check spelling of the French EmbeddedObjects manual

    Translation of French Additional manual completed

Sanitize cursors after a buffer has been reloaded

When a buffer is reloaded, its content may remain the same, but the
memory allocation is new, so that the inset pointers in cursors are
now wrong. This requires to sanitize the cursors held by the buffer
views.

Before the biginset branch, some full metrics computation call that is
now removed probably did that as a side effect. Now we have to be more
precise.

To this effect, introduce WorkAreaManager::sanitizeCursors() and use
it in Buffer::reload().

(cherry picked from commit c1fd622c51752d790576600f5911813ff8dac3fa)

small changes backported from master

   Typo

    Remove non supported Article classes in UserGuide

* metainfo.xml - homepage missing

Document biginset branch

Status

     Update fr.po

Update sk.po

* cs.po

de.po

remerge strings

Handle the case of outdated chkconfig.ltx in user directory

(cherry picked from commit b8ff824a4f4ab4d3e213b55bf8117fd95b7b9657)

Status update

gbrief-2 loads babel internally. So don't load it ourselves.

This makes the class fully functional with non-TeX fonts (it uses
\babelfont for the purpose). Before, docs were compiling but the
output was wrong.

(cherry picked from commit c3c96a446b32ef77927b5e06436c2668d95dfd6d)

g-brief loads babel internally. So don't load it ourselves.

This makes the class fully functional with non-TeX fonts (it uses
\babelfont for the purpose). Before, docs were compiling but the
output was wrong.

Convert $${python} placeholder for graphic conversions

(cherry picked from commit cdcaf0e7b6cc45bc74175667c390c3148c4730e9)

Cmake install: Forgotten to install tabletemplates.

(cherry picked from commit d05e295ddc22c65183a203e97240898a055d2d12)

Cmake install: Forgotten to install tabletemplates.

(cherry picked from commit d05e295ddc22c65183a203e97240898a055d2d12)

     French UserGuide: check spelling and remove no more exiting article classes

#13049 add window activation for preferences to avoid it be hidden by main window on mac after alert prompts

(cherry picked from commit 4c1db7cad866d99af2f2474d8467cc8581eb0eef)

   Typos in Additional.lyx, seen while translating in French

   Typo in fr.po

   Typos iun Franch Additional manual

    Update translation of the French Additional manual, section 4.12 missing

* another update fo zh_CN.po

Always repaint the gray area below main inset

Now that SingleParUpdate does not always lead to a full screen update
when the height of the paragraph changes (see new behavior of
updateMatrics(bool)), it is necessary to make sure that the grey area
below the main page is always repainted.

(cherry picked from commit 1a11abe4394272f521cd63993e426c136e0e9b6c)

Do not compute metrics at each preview when loading file

With the branch-test.lyx file from #12297, loading takes forever when
previews are activated. This is because each preview element causes a
full screen metrics recomputation.

This commit just skips these calls and only does one when all previews
have been obtained. As a result, computing the previews takes 1 second
instead of 25 seconds on branch-test.lyx.

Part of bug #12297.

(cherry picked from commit 244969330108a89f4be93d3b2eb9024bdb756204)

Improve the code that limits scrolling at top/bottom

The most visible part of this commit is the move of part of
BufferView::updateMetrics to a new TextMetrics::updateMetrics. This
new method makes sure that metrics are known for all visible paragraphs
(starting from anchor), and that the positions of the paragraphs have
been recorded.

This method is called up to 3 times in BufferView::updateMetrics:
* unconditionally, to update all visible metrics,
* then, if the bottom of the document is visible and too high, after
  updating the anchor ypos,
* and similarly if the top of the document is visible and too low.

This fixes for example the case where one jumps to Section 5.3 at the
end of Tutorial and 'scroll_below_document' is false.

Some now redundant code is removed from BufferView::scrollToCursor.

The anchor-setting code in BufferView::draw is not clearly useful, but
left here just in case. It generates a debug warning, though.

Part of bug #12297.

(cherry picked from commit f15d2ebf3819913114ab93d1ff7e140cb26b03d5)

Implement quick scroll

Replace flag parameter for updateMetrics() by a `force' boolean. When
it is false, the method keeps the metrics of paragraphs that are still
visible in WorkArea instead of computing everything afresh. All it has
to do is update their positions.

Add code to updateMetrics() to update the value of the anchor pit/ypos
(similar to the one in draw()).

Update processUpdateFlags() to use this when update flag is ForceDraw.

Modify scrollDocView() to just change the anchor paragraph position
when the scrolling operation would re-use some of the existing
paragraphs.

The time needed to update the metrics when scrolling with mouse in the
branch-test.lyx document is now divided by 20!

Part of bug #12297.

(cherry picked from commit 08010c6a5e425b3f2d0d625536e3a571c90a0482)

Introduce new helpers ParagraphMetrics::top/bottom

This avoids code with position/ascent/descent that is difficult to follow.

No change in function intended.

(cherry picked from commit 0b6105b9245350e428c73deee88af2cd7c0d4732)

In the no-draw phase, do not cache the positions of not visible insets

This can make a big difference for a very large branch that contains
lots of equations.

This is complementary to the previous patch, since instead of reducing
the number of calls to updatePosCache, we make it faster.

In the same test of scrolling with mouse wheel through the
branch-test.lyx document, one finds a 23% improvement for
BufferView::updateMetrics().

Part of bug #12297.

(cherry picked from commit 7f85024f80601b15634fb5e771bba51435ad429f)

Remove some redundant calls to updatePosCache

The setting of insets positions was done twice in updateMetrics.
When one of the paragraph is a huge branch, this can be very expensive.

This leads to a 17% improvement on updateMetrics time on a scrolling test.

Part of bug #12297

(cherry picked from commit d19ade9a611d3ecf6840c5eb43291cb268ad6f4f)

Enable Update::SinglePar in nested insets too

The idea of single par update is to try to re-break only the paragraph
containing the cursor (if this paragraph contains insets etc.,
re-breaking will recursively descend).

The existing single paragraph update mechanism was tailored to work
only at top level. Indeed changing a paragraph nested into an inset may
lead to larger changes.

This commit tries a rather naive approach that seems to work well: we
need a full redraw if either

1/ the height has changed
or
2/ the width has changed and it was equal to the text metrics width;
   the goal is to catch the case of a one-row inset that grows with
   its contents, but optimize the case of typing in a short paragraph
   part of a larger inset.

NOTE: if only the height has changed, then it should be
  possible to update all metrics at minimal cost. However,
  since this is risky, we do not try that right now.

Part of bug #12297.

(cherry picked from commit 9a96726bcd06d565c3027011fea954656aa46668)

Defer construction of temporary cursor.

We were doing this on every mouse movement. I.e., for every single
pixel the mouse traverses. It can be expensive if there are a lot
of math insets.

(cherry picked from commit 4e8aa89eb96cd73e6630deea6fd6b9d3039aab6c)