Fix broken Apple speller interface

- restore spell checker result LEARNED_WORD - it's required for Apple speller engine
- avoid use of "unlearn" of known dictionary word - it doesn't work with paragraph based spell check

Additional check for empty word to avoid potential crash if no WordLangTuple is assigned by spell check

Allow removing words from the personal dictionary, that weren't previously added.

"Cancel Background Process" -> "Cancel Export"

"Cancel Export" sounds more user friendly.

Discussed briefly on ML here:

  https://www.mail-archive.com/search?l=mid&q=20221122200127.th3qwgpimqndksmd%40gary

RELEASE-NOTES: document clicking on red x

New functionality introduced at 869247b6.

Rm dup "Float" in "Insert>Float>Figure Wrap Float"

Change menu options "Figure Wrap Float" to "Wrapped Figure"
and "Table Wrap Float" to "Wrapped Table".

Improve replacement for qDegreesToRadians

This is for Qt 5.0.X compatibility.

Update sk.po

makefile - forgotten file

* postats.py

Centralize cancelation of export

An export can be canceled by clicking on the status bar icon or by
executing export-cancel (e.g., via Document > Cancel Background
Process).

The centralized code kills the script and updates the status bar
icons.

No change in functionality intended.

Related commits: 70a71a82 and 01abab9a.

After export-cancel, hide busy and cancel buttons

This is consistent with 70a71a82 (which is about the code path
through clicking on the 'x' status bar icon).

In the next commit I'll make a function to centralize this sequence
of killing a script and emitting the scriptKilled() signal.

After confirm cancel, hide busy and cancel buttons

Cleanup mathes/BUGS (#3493)

This one is now also part of #12603.

Cleanup mathes/BUGS (#3493)

This one is now #12603.

Cleanup mathes/BUGS (#3493)

Removing. If this is still an issue, it should be described in much
more detail.

Cleanup mathes/BUGS (#3493)

Removing. M-c e has a different meaning nowadays
(tabular-feature delete-vline-left)

Please file a new report with an updated description if this is still
an issue.

Cleanup mathes/BUGS (#3493)

This one is now #12602.

Someone please check whether this is valid, and feel free to close.

Start to cleanup mathes/BUGS (#3493)

This one is implemented.

Add button to stop processing

(rather than having it stop by clicking on the spinner)

          Translation of new index features step 1

Do not allow nesting macro names in macro names

LyX removes a single backslash when it is inserted in a
macro name by the keyboard. However if it is followed by
another character before moving the cursor, it is retained
causing an assertion (bug #12601).

Substack should not be allowed to change columns

Disable the arguments append-column and delete-column of tabular-features.

The code is taken from InsetMathCases, with some changes
* no need to record undo here
* in dispatch, return is prefered to break, since we do not want to invoke
  InsetMathGrid::doDispatch.
Propagate these changes to InsetMathCases.

Cleanup of the InsetMathCases error messages to fit with other parts
of the code.

The handling of tabular-features in mathed needs to be unified somehow.

Based on a commit from lynx <lorenzobertini97@gmail.com>

Part of bug #12590.

ctests: typo in ignoredTests regex

Amends c94bb34c.

Comments

Do not allow pasting backslashes in macro names

Backslashes are not allowed in macro names and LyX
enforces this by not allowing entering them by keyboard.
However, it was possible to paste them and this may cause
crashes (see bug #12596).

Regenerate previews after zoom (#11919)

symbols: fix a type for \Im

Amend 48d9d01a: remove debug output

ctests: uninvert two xhtml tests

These exports previously had xmllint errors. These errors seem to
have been fixed thanks to the "features/indexmacros" merge.

Related to #10355.

Update the format for some doc pages

InsetIndex: when building the IndexNode tree, ignore IndexEntry when empty

The previous code always considered it had at least one term (an assumption that is wrong at least for doc/de/Math.lyx).

InsetIndex: enable escaping for terms in the index

The previous code explicitly disabled it and this behaviour caused problems (like "A&A" being output as-is, which is wrong in HTML: it should be "A&amp;A").

InsetIndex: improve formatting of resulting XHTML

There was no line feed before the closing </li> for entries, which was inconsistent with the other block tags.

Improve wording of RELEASE-NOTES entry

Drop C++17 if init-statements to support older compilers.

Fix compilation with gcc 4.9

It appears that gcc 4.9 does not implement the following part of C++11:
https://cplusplus.github.io/CWG/issues/1148.html

Therefore, we have to use a special case in C++11 mode that does an
explicit std:move.

With recent compilers (gcc >= 9), this leads in C++11 mode to a warning:

MetricsInfo.cpp: In member function ‘lyx::Changer lyx::MetricsBase::changeFontSet(const string&)’:
../../master/src/MetricsInfo.cpp:83:13: warning: redundant move in return statement [-Wredundant-move]
   83 |  return move(rc);
      |         ~~~~^~~~
MetricsInfo.cpp:83:13: note: remove ‘std::move’ call

Partly reverts commit fff28c57.

Merge branch 'breakspace'

This branch improves handling of spaces on display (see #10117):
* caret is correctly shown in the middle of double spaces in justified rows;

* sequence spaces are correctly shown at the end of rows before
  automatic row breaks.

Moreover, this branch:
* streamlines the code that handles spaces in row breaking

* improves display in Qt4 : although the improvements outlined above
  are not present in Qt4 for monospaced fonts, some dsplay glitches
  are resolved.

* improves performance for very long paragraphs (#12598).

Fix tex2lyx corner case (#12537)

(almost) Fix Qt4 support

When is it detected that our zero-width characters are not really
zero-width, avoid to use them altogether. Currently, this is known to
happen in Qt4 with monospace fonts.

This allow to remove the special Qt4 code in breaskString_helper.

Related to bug #10117.

CMake build: disable warn about not known policy

This only showed up with -DLYX_EXTERNAL_ICONV=OFF.

Use TextLayoutHelper in x2pos and pos2x

This leads to substantial code simplification.

No intended behavior change.

Related to ticket #10117.

Refactor GuiFontMetrics::getLayout

This allows to create a new version that takes an already built
TextLayoutHelper struct as parameter.

No intended change.

See discussion in bug #10117.

Handle multiple spaces at row break

In order to work around the Qt row breaking algorithm, which considers
multiple spaces as one at QTextLine break, we insert word_joiner unicode
characters beteween each pair of spaces.

The TextLayoutHelper class makes it easy to handle that.

Update Row::Element::rtrim() to only remove one space at row end.

Update support::countExpanders() to count all spaces, without special
handling for consecutive ones.

Fixes bug #10117.

Introduce a struct to handle docstring <-> QString index translation

This new code allows to get rid of some hardcoded stuff and will be a
foundation for handling of consecutive spaces.

This code can be used in the future for getTextLayout, x2pos, and
pos2x in order to streamline code. Actually some features as the naked
parameter are here to prepare for this second set of changes.

No change intended. The modified code is much clearer, though.

Related to bug #10117.

Revert "Fixup 572b06d6: reduce cache size for breakString"

10MB cache is not so large after all, and it can be helpful with
Scott's perverse examples.

This reverts commit 2abe7714078a0f772af2e3e2675d6f2b3c054e08.

Fixes part of bug #12598.

Fix expensive thinko: operator++ returns a reference, not a copy

This accounts for 45% of the row breaking time on the (huge) paragraph
of the MWE in bug #12598.

Fixes part of bug #12598.

Update sk.po

Revert "Try to be more pedantic wrt blank lines in lyx2lyx output"

This reverts commit 9ce0ff50c1a96b276fa1a74f358718bf571f3dfb.

Try to be more pedantic wrt blank lines in lyx2lyx output

Hopefully to the tests' please

Correct typos of "occurred" in warnings/errors

Change "occured" and "ocurred" to "occurred".

Fix pasting as LaTeX in documents with no modules

When there is no module, the old code would invoke tex2lyx with option
 -m
instead of
 -m ""
and an error would ensue.

This is because the QProcess command line parser used in Systemcall is
broken and does not recognize empty parameters as such. The solution
is to rewrite our parsecmd() parser to generate a list of parameters.
This is post-2.4.0 work.

The workaround used here is:

- set the list of modules to "," when it should be empty. In effect,
  this is a list of two empty modules.

- change tex2lyx to accept empty module names and ignore them; this is
  good in terms of robustness anyway.

Additionally, when there is no receiving buffer, set the defaults as
the BufferParams defaults instead of empty (this is cleaner, but
should not make a difference in practice).

In the long term, we should switch to use the QStringList-based API of
QProcess in Systemcall (see QTBUG-80640).

         Update fr.po

de/UserGuide: document index-tag-all

de.po

ctests: ignore a couple more roundtrip tests

Export fails after compilation with the following error:

  ERROR: CHAR: index 0 should be less than the length of the string

Guard some debug code

Amend f352a375 and 3bf1b97a.

Introduce index-tag-all lfun

This is a convenience function for indexing. It adds a copy of the
index inset under cursor after any word in the buffer that is equal
(case-insensitively) to the word preceding the index inset.

The function can be easily used to generate bad indexes (and I warn
about that in the UserGuide), but if used with care, it can also be
extremely convenient.

ctests: log a waste

ctests: ignore a couple fragile lyx2lyx tests

From Kornel [1]:

  Exporting to 2.3 format and exporting from the exported differ in extra lines and
  in changed sequence of 2 lines at begin.

  $ diff doc/fr/UserGuide.23.lyx fr/UserGuide.23.23.lyx

[1] https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg218252.html

Update RELEASE_NOTES

Show suggestions containing the input in the command buffer

Previously, only the suggestions starting with the current input were
shown.

Contains the following minor improvements:
- Add space to indicate when only one suggestion is found

- Select first item in suggestion-list in order to make selecting with
  arrow keys more intuitive

- Fix selection with Shift+Up/Down in text-field

Fix for bug #12572.

New lfun to copy index insets from ToC

The new function inset-insert-copy can only be invoked from the ToC.
It is currently only implemented for Index inset.

It is a special lfun because Inset::dispatch is called directly from
the ToC widget with cursor pointing to the inset, whereas the patch
happens in the workarea at caret position. This function cannot be
called directly.

Add an entry for this function in the toc context menu.

Fixes bug #4582.

When Inset::dispatch is called from TOC, process screen flags

This is needed because we are not using the complete dispatch
machinery here.

Part of bug #4582.

Introduce new function copyInsetToTemp

This is a version of copyInset that does touch the cut stack and the X
selection.

Part of bug #4582.

Fix another use of unqualified std::move

          Update fr.po

Amend 9be6a425, Workaround for advanced search in 'uncover'-inset.

Use getInset(i)->toString() instead of getInset(i)->plaintext()
if searching, so that we get the option of 'uncover' visible to the search engine.

Update sk.po

de.po

Add some options to index context menu

Adapt French docs to new index features

Do not \protect stuff in index when we postpone the index

\protect us not needed here and affects sorting

Fix page range without page format

ctests: log a waste and a success

A couple of the lyx2lyx tests were helpful, and a couple did not
lead to a fix. In particular, testing compilation was not helpful in
this case.

ctests: ignore some lyx2lyx tests that fail

From Jürgen [1]:

  These are old problems. One is related to a questionable nomencl change
  which we addressed in master but not in stable, see
  https://www.lyx.org/trac/ticket/10634#comment:2

  The other is due to CJK quotation marks which are not supported in 2.3.

We ignore, rather than invert, since no plans to fix these.

[1] https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg218237.html

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