Amend b8502a3ea2735c4

Fix glitch in index reversion routine

Indicate see[also] refs in label and outliner

Adapt all German and English manuals to new index features

Do not unnecessarily reset the outliner to TOC on internal model changes

The activated() signal is only issued on user changes, what we want here,
not internal model changes (as with currentIndexChanged())

Remove wrong count comment

Workaround for advanced search in 'uncover'-inset.

Allows for matches inside style Uncover if used
Settings->Adhere to search string formatting
This does not cover all of #12594 though.

Update sk.po

lyx2lyx: fix index reversion

This was only a warning, but it made the tests fail

          Update fr.po

German documentation of new index features

de.po

Fix output of subentry with sortkey

UserGuide: use new features for all index entries

Handle empty index subentries (#7820)

InsetIndex: hide printTree behind a LYX_INSET_INDEX_DEBUG flag

The change ensures that the function is not seen by the compiler when
the debug mode is not enabled. The flag is not set by default, because
it might have a significant performance impact for large indices.

The reason for the flag is that the default configuration under Linux
forbids the definition of unused functions.

Update sk.po

Update tex2lyx tests

Merge branch 'features/indexmacros'

Document new inset features

Typo

Rename subindex to subentry

Makes more sense, and distinguishes it better from printsubindex

Display subinsets in inset button and outliner

Enter subindex on Enter in index

Add Christopher Hillenbrand to the credits

Add some comments

          Update fr.po

Add last missing bit to insetindexmacros: file format change and lyx2lyx

The feature is now ready for primetime!

format change stub (to be updated before merge)

Enable pagination property with inset sortkey

Update sk.po

Merge branch 'master' into features/indexmacros

de.po

Minor improvements to the screen width UI

* fix order of input and unit widgets
* remove redundant "Screen used" label
* Add tooltip

Whitespace

        Update fr.po

Revert "WIP: refactor Systemcall"

This reverts commit c8015878e1375f83574a8497faadd7e8c6aa8859.

Committed by mistake obviously!

constify

WIP: refactor Systemcall

**WARNING; only compilation has been ested; even that does not work**

The goal of this commit is to use the list-based API to
QProcess::start, to avoid annoying syntax issues (see issues with
pasting from LaTeX).

* Create a new latexEnvironment() in filetools.h that returns a
  map<string, string> containing the variables and their values.

* Rewrite parsecmd() so that it returns a QStringList of tokenized parameters.

* Use this in startProcess. This is the part is is not finished yet.

Obviously, this will not be possible to get right for 2.4.0.

adjust text display width in windowed mode

Remove unused method

Revert "Always remove selection after cursor up/down"

This reverts commit 008a0825e8d67b9c0a3d31b906f04cd7b8fd18a8.

This commit was part of bug #12310. The goal was to remove the
selection when using "up" at the start of selection (so that the
cursor cannot move). Unfortunately, it creates navigation issues for
insets inside tabulars.

Since the fix was minor, the safest decision is to revert.

The part of ticket #12310 that resets selection with
char-forwars/backward when moving is not possible still works.

Fixes bug #12593.

de.po

Sync with master

Always qualify calls to std::move()

This quashes a new warning in clang++ 15, when std::move() (the
one-parameter version) is used as simply move(). There is a strong
recommendation from WG21 to avoid that.

Details here: https://reviews.llvm.org/D119670

It might be that we should not use that many move()s. I am not
competent to decide on that.

I also used this occasion to get rid of a spacial casing for C++11
that does not seem necessary after all.

GuiLog: don't miss package warnings for packages with dashes

(e.g. scrlayer-scrpage)

       Update fr.po

Only load subscript package with LaTeX < 2005/12/01 (#12387)

Need cellvarwidth also in left-aligned cells if they use newlines (#12587)

GuiLyXFiles: differentiate user and system folders

GuiLyXFiles: Hide empty categories on filtering (#12584)

GuiLyXFiles: Visually distinguish folders from files (#12584)

Preview: do not overwrite fg/bg color names that beamer uses

Rename them to lyxfg/lyxbg

Fixes bug #12563.

Fix inline completion prefix for macro insets

As it was, the prefix was empty when completion popups were disabled.

Related to bug #12581.

Disable completion when document is read-only.

Being able to insert completion in a read-only document is definitely
a bad idea.

This patch acts both on indication of completion and on insertion
itself.

Fixes bug #12582.

Show branches from master document in branch inset dialog

Completion: handle undo in insets' insertCompletion methods

I mathed, undo should record the underlying inset on completion,
whereas in test recording the paragraph is enough.

Therefore the recordUndo() in GuiCompleter::tab is removed, and the
relevant recordUndo/recordUndoInset are used at the right places.

As a further cleanup, the parameter `finished' of
Text::insertCompletion is removed as it is useless.

Fixes bug #12581.

Whitespace

Fix bug #12580

This amends [4a7a1935/lyxgit]. Before 4a7a1935 it was assumed
that a labeled row was also numbered. Hence, when unnumbering
a row a label was also deleted. We now have to do this explicitly.

ctests: ignore zh_CN/Tutorial that time out

Jürgen pointed out that it might be due to this issue:
  https://tex.stackexchange.com/questions/548139/

and that we might want to report it here:
  https://www.nongnu.org/cjk/

ML discussion was here:
  https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg217924.html

Fix gcc-12 warning

From Kornel:

>  /usr2/src/lyx/lyx-git/src/Paragraph.cpp:1931:1: warning: control reaches end of non-void
>  function [-Wreturn-type]
>
> This is with gcc-12, compiled in debug mode.

Fix line breaking at trailing space

With a string like "abc def [inset]", it should be possible to break
between the "def" and the inset. A case of interest is when "abc def "
is too long, but "abc def" is not.

To this end, tweak the test that look whether breaking was successful:
the right width to look at is nscp_width, the width without the final
space.

vocabulary

Fixup 572b06d6: reduce cache size for breakString

This was necessary only due to bug #12534.

Reduce the size from 10Mb to 1Mb. Profiling shows that the cache hit rate
is the same.

Disable use of QPA_XCB if using QT6

Update sk.po

Update it.po

Amend f426470a

The synctex option can also be specified by a single dash and
any value different from zero will do.

Unify wording for statistics

Use 'no blanks' uniformly.

Also remove separator in zoom menu.

Disentangle status bar and zoom menu

- Moves the zoom menu to the zoom value

Update sk.po

* doxy

de.po

Simplify status bar context menu

Remove "Show" prefix.

Remove "(With Blanks)".

        Update fr.po

MathML: amend 26300c565f as discussed in bug 12513

Contributed by lynx

MathML: should display "Text" MathFont using proper <mtext> tag

Fixes bug 12397

Contributed by lynx

InsetTabular: fix a typo

InsetTabular: move more helper methods to private

InsetTabular: replace a Boolean parameter to choose between XHTML and DocBook by an enum class

The only goal is to improve code readability: this enum class is strictly equivalent to a Boolean, with the same meaning as the previous is_xhtml arguments (arbitrary choice).

InsetTabular: move XML helper methods to private

They are not needed outside InsetTabular.

DocBook: amend 50374e38e1

All table rows were generated in the same XML dialect, irrespectively of the caller context (XHTML or DocBook).

Small fixes to fr.po related to status bar menu

Fix a warning related to comparing different signedness

LyXHTML: implement index ranges

LyXHTML: implement multiple indices

DocBook: amend 34ea4080

@ for sorting is implemented at 34ea4080, but the user was still shown a warning (now removed).

DocBook: add test case for multiple indices

DocBook: add missing test case for index

Use auto const & when possible to avoid copies

In several range-based for loops, implicit copies are made. Remove
that when possible, and try to shut converity up otherwise.

Fixes issues found by coverity.

       Update fr.po

Update sk.po

Update stats display if a stat is (un)selected

ctests: uninvert a utf8x test

The compilation succeeds now, although the behavior might be
different: unless we load ucs explicitly it will proceed with
utf8 instead of utf8x if I understand correctly.

See here for more details:
https://github.com/latex3/latex2e/issues/833#issuecomment-1219535732

ctests: invert EuropeCV tests

These might not be fixed for a while:

  https://github.com/gsilano/EuropeCV/pull/33

de.po

Another inset font switch fix

Polish status bar statistics

Load ucs before utf8x inputenc with recent ucs versions

Otherwise inputenc falls back to utf8 option, which breaks documents

fix compiler warning