Fix remaining path

Cmake export tests: Make test fail if there is some non-existant sub-file

doc files: fix some path and reference mistakes

also replace remaining occurrences of EPS images

Japanese EmbeddedObjects.lyx: fix image paths

also replace remaining occurrences of EPS images

Move some Cursor methods to CursorData

Basically, everything that does not depend on a BufferView should move
there. Some methods that do not seem to need a BufferView, like
selHandle or IdxFirst or push actually depend on it and could not be
moved.

This allows to simplify a few uses of recordUndo helpers.

- Move some methods to DocIterator: nextMath, prevMath, getPossibleLabel,
  getEncoding;

- Move some methods to CursorData: setCursor, setCursorSelectionTo,
  (setCursorTo|normal|reset)Anchor, (set|clear)Selection,
  sel(|ection)(Begin|End), selectionAsString, info, currentState,
  (mark|clear|check)NewWordPosition, fixIfBroken, sanitize, all undo
  related methods, reset, isInside, leaveInset, current mode;

- kill some unused methods: macromode, replaceWord, setScreenPos, touch,
  markInsert, markErase;

- Move code around to group things, and add a few comments (a lot remains to be done).

This changes lead to some related changes in other classes: removal,
change of parameter.

No intended change.

Fix comment

Allow multiple calls to processUpdateFlags before redraw

The goal of this commit is to ensure that a processUpdateFlags call
that requires no redraw will not override a previous one that did
require a redraw.

To this end, the semantics of the flag argument is now different: its
value is now OR'ed with a private update_flags_ variable. This
variable is only reset after the buffer view has actually been
redrawn.

A new Update::ForceRedraw flag has been added. It requires a full
redraw but no metrics computation. It is not used in the main code
(yet), but avoids to compute metrics repeatedly in consecutive
processUpdateFlags calls.

The process is now as follows:
- if flags is just None, return immediately, there is nothing to do.
- the Force flag is honored (full metrics computation) and replaced
  with ForceDraw.
- the FitCursor flag is honored and removed from the flags.
- the SinglePar update is added if ForceDraw is not in flags and only
  the current par has been modified.

The remaining flags are only then added to the BufferView update
flags, and the update strategy is computed for the next paint event.

Finally the dubious call to updateMacros in updateMetrics has been
removed for performance reasons.

de.po

Update sk.po

de.po

More accurate description of fixltx2e state.

   Add obsolescence warning in fixltx2e description (ticket 9361)

 Turn 'Do not load' math option to 'Automatic' (ticket 10661)

Implement ParamInfo::HANDLING_LTRIM and use it to ltrim InsetBibitem
output

See #9847.

Fix math mode for InsetMathMacro on output

After 6642152e, user macros were no longer wrapped in \ensuremath. In 2.2 and
before, InsetMathMacro behaved as follow:

* Textmode global symbols are wrapped in \text when in math.

* Other global symbols, and user macros, are wrapped in \ensuremath when in
  text.

* Undefined macros (ERT) are wrapped neither in \text nor in \ensuremath.

This is also consistent with the documentation of MathEnsurer in
mathed/MathStream.h.

This patch defines InsetMathMacro::currentMode() accordingly (respectively
TEXT_MODE, MATH_MODE and UNDECIDED_MODE) and uses it to determine the output.

After this patch, there is a mismatch between screen and pdf output for user
macros in \text. This is not a regression wrt 2.2 and is because linearization
does not satisfy currentMode() currently.

Additional.lyx: port today's changes

Account for shadowed system macros

Part of #10694.

Do not add deleted text to inset button label.

Fixes: #10775

Fix issue with change-tracked deleted display math

- If a display math not starting a new paragraph is deleted, the
  current \lyxdeleted macro (if any) must be closed and a new one
  started, otherwise the display math will be shifted up.

- Use \linewidth instead of \columnwidth because the former will adapt
  to the reduced horizontal width in list environments, avoiding shifting
  to the right the diplay math.

Use suffixed name for desktop entry.

Unix only:
Using suffixed lyx, we install the lyx-desktop for different versions with
different file name, but with the same 'Name'-entry.

Selecting the desired version with the desktop menu is difficult if there
is more than one lyx version installed.

We do not want to trim whitespace in the fallback, either.

Fix bug #9847.

Spaces are, amazingly, allowed at the end of bibliography keys. So we
introduce a new parameter allowing getVectorFromString not to trim
whitespace, and then use it.

Fix small display issue with biblatex's \textcite

Speed up exit time

Instantiating a single QSettings and using it for each ui element
can significantly shorten the time required to save the various
states at exit. The speed up can be better appreciated on *nix,
where the settings are saved on disk, rather than on Windows where
they are held in memory (in the registry).

Do not scan BibTeX files multiple times in a collectBibKeys() procedure.

Scanning is rather slow, so this improves performance in specific
situations (multiple inclusion of larger files in master/child or
chapterbib context)

Revert "Allow multiple calls to processUpdateFlags before redraw"

The concept is correct, but it seems a bit too disruptive.

This reverts commit c19c54dd5b85726df1b5187616d17d5430028c16.

Cmake doc creation: Python is choking on '\\'

Cmake doc creations: Correct indentation

Cmake doc creations: Setting environment PYTHONIOENCODING

This should force the python executable to use the desired encoding.
The command is moved to a wrapper script to be able to set the environment
at make-run-time.

Use of div for labels causes validation errors. See #8843.

Revert "ABOVE and CENTERED labels need special treatment."

This reverts commit 08bf9781ee86ea7fe2c9f3a0d670cad92807a017.

Fix bug #10556.

Update CSS for Standard layout.

Additional.lyx: replace quotes character by LyX quotes

ReplaceValues.py: partly revert commit f075f8ad

we need a string output not a byte

ReplaceValues.py: make the file compilable with Python 3

see the mailing list thread "compilation of LyX 2.3 fails with Python 3.6.2" for details

Remove "support::" in Buffer.cpp, since we are using namespace.

Rename Paragraph::setBuffer to Paragraph::setInsetBuffers, to avoid
confusion about what this routine does.

Fix the way that the Buffer gets set on undo.

Previously, we went through the entire Buffer and set it for every
single inset. Now we just do it for the insets we pasted.

Allow multiple calls to processUpdateFlags before redraw

The goal of this commit is to ensure that a processUpdateFlags call
that requires no redraw will not override a previous one that did
require a redraw.

To this end, the semantics of the flag argument is now different: its
value is now OR'ed with a private update_flags_ variable. This
variable is only reset after the buffer view has actually been
redrawn.

A new Update::ForceRedraw flag has been added. It requires a full
redraw but no metrics computation. It is not used in the main code
(yet), but avoids to compute metrics repeatedly in consecutive
processUpdateFlags calls.

Finally the dubious call to updateMacros in updateMetrics has been
removed for performance reasons.

Collapsable -> Collapsible (part 4)

The current spelling is not strictly wrong, but flagged as unusual or
historical by some authorities. It is also found fault with many
spell checkers. Thus we decided to move to the more standard "-ible"
form once and for all.

See #10678 for discussion

This last part updates the layout format and changes collapsable color.

This will all also all be backported to 2.3.x, for the sake of backwards
compatibility (cherry-picking).

Collapsable -> Collapsible (part 3)

The current spelling is not strictly wrong, but flagged as unusual or
historical by some authorities. It is also found fault with many
spell checkers. Thus we decided to move to the more standard "-ible"
form once and for all.

See #10678 for discussion

This part addresses the po files.

This will all also all be backported to 2.3.x, for the sake of backwards
compatibility (cherry-picking).

Collapsable -> Collapsible (part 2)

The current spelling is not strictly wrong, but flagged as unusual or
historical by some authorities. It is also found fault with many
spell checkers. Thus we decided to move to the more standard "-ible"
form once and for all.

See #10678 for discussion

This part covers the most tricky part: the internal naming.
Translations and layouts will follow.

This will all also all be backported to 2.3.x, for the sake of backwards
compatibility (cherry-picking).

Collapsable -> Collapsible (part 1)

The current spelling is not strictly wrong, but flagged as unusual or
historical by some authorities. It is also found fault with many
spell checkers. Thus we decided to move to the more standard "-ible"
form once and for all.

See #10678 for discussion

This part only covers the usage in comments and the like. More to follow.

This will all also all be backported to 2.3.x, for the sake of backwards
compatibility (cherry-picking).

Typo.

Revert "Fix bug #8782."

This caused slowness on undo and redo that was reported at
https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg202292.html

This reverts commit 02847641a8b97a57d1866120bb446427e0bda7a4.

Remove updateInfo() calls in favor of doing the relevant work
in updateBuffer().

This is in response to a reported crash. See
https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg202217.html
Commit af381a2f addressed the crash, and is worth doing anyway. But
this also makes sense.

Also adds a flag to tell us whether we need to recalculate the
relevant information. In some cases, there is no need to do so
more than once. (Note that, with the existing code, we *never*
recalculate, which would have given rise to bugs.)

Win installer: one DLL was removed in Qt 5.6.3

Win installer: update Qt

- Qt 5.6.3 uses a different path name scheme
- one DLL was removed in Qt 5.6.3

Update it.po

Fix issue with regular expressions

At least with gcc 6.4, if the first parameter passed to
regex_match() is afterward changed, the second one gets
corrupted. This is avoided by using a temporary string.

Update sk.po

updatelayouts.py: consider citeengines.

Update some more layouts

Adapt citengine to current layout format.

de.po

Some improvements to the graphics dialog

* Describe the viewport/bb situation more accurate
* Use generic term "coordinates", since bb and viewport are flavor-specific
* Add some tooltips
* Increase the width of the options widget.

Fix literal string in binary context

In 1a26f9b26cff827da5, all strings in the context of regexes and
comparisons have been converted to byte-like objects for python2/3
compatibility reasons. This has not been considered in the later
commit 5c545573507bb, which makes layout2layout break with
TypeError: can't concat str to bytes.

Fix PDF output of clipped graphics with recent graphics package

The previous redirection of the senseless option "bb" to "viewport" in
PDFLaTeX output has been removed in a recent graphics package update.

This breaks documents, since clipped graphics silently stop displaying.

This change restores the previous output by using "viewport" instaed of
"bb" for non-PS/DVI output, while leaving PS/DVI untouched (where "bb"
and "viewport" behave differently.

Fixes: #7910

Some improvements related to Debug::PAINTING

Avoid blinking the cursor when these debug messages are active.
This avoids a lot of debug information.

fix broken comment

Revert "autoconf"

I did not plan to push this

This reverts commit 0bd6cc30bc30d3ae4315709c23b8aeb4bdccaf74.

Do not forget to set cursor current font when selecting

Fix several unreleated cases where the `selecting' code path did not
reset cursor font correctly. This leads to a caret that does not have
the correct size, and can cause caret droppings with the new display
scheme.

autoconf

Fix issue with language switch in Arguments insets

In some insets such as Arguments, a local language switch has to be
used. However, if the language inside the inset was set to be equal
to the outer language, the code decided not to switch language. But
then got confused and tried to close a switch that was never opened.
This patch forces the switch even if the outer language is the same.

ja/EmbeddedObjects.lyx: correct file path

References to the central clipart directory must be prefixed with
".." for non-English manuals.

Add description comment for the lyxconvert utility

Reset cursor before beginning reload sequence.

Cmake tests: Another case of using temporary files

Ctest misinterprets '#' in filenames as comment start,

Seminar export fails with latest TeXLive.

Also update the comment on wrong output with "seminar".

Uninvert Japanese tests after fix to #8823

The export ja/lilypond_pdf fails because ps2pdf gives an error. It
is thus still inverted, under the category 'externalissues'. As
Jürgen discovered, ps2pdf succeeds if the -dNOSAFER flag is used.

Note that Kornel is seeing strange behavior with the sweave test,
and thus the label of that test might be changed soon (e.g. to
"unreliable"). For discussion, see:

https://www.mail-archive.com/search?l=mid&q=20171001032524.fr5xfngylththwv2%40steph

Cmake build: Needed variable for creation of debian package

The variable CPACK_DEBIAN_PACKAGE_RELEASE has to be in the form
of "^[A-Za-z0-9.+~]+$". We will use the abbreviated commit revision for now.
Without this change cmake 3.10 emits error.

RELEASE-NOTES: reference wiki with previous notes

If users are upgrading from an older version of LyX than the
previous major series, they should consult the release notes from
the releases in-between.

For more discussion on this topic, see the following mailing list
thread:

https://www.mail-archive.com/search?l=mid&q=20170902070549.47ciiszxjjqhyreq%40steph

Additional.lyx: update eLyXer URL

The previous link still worked but asked for bookmarks to be updated
to the newer URL.

Thanks to Ian Holliday for this correction.

ja/Math.lyx: correct the URL of the mhchem manual

It was pointing to the hyperref manual.

sweave.lyx: remove ref to previous homepage

The homepage does not exist anymore. Note that R installations now
come with Sweave already installed.

We now instead reference the user manual of Sweave.

poster-a0poster-simple.lyx: rm ref to a0poster-kh

a0poster-kh is unmaintained and non-standard. Note that URL in the
removed \item was broken.

For more information, see discussion on list:
https://www.mail-archive.com/search?l=mid&q=20171006055205.jdicpppko3tjrlef%40steph

docs: fix URL to hyperref manual

Our URL tests have been reporting the previous URL for a few months
as broken, and browser confirms it.

Add license and author infos

Cmake build: Don't use temporary doc-files as source for installation

docs: update URL of Astronomy & Astrophysics

The new URL was found by visiting the root URL and clicking on
"journals".

fr/Additional.lyx: remove duplicate URL

Add missing check for Qt version when using QPdfWriter class

Cmake build: Only use the new lyxconvert on MAC

Update noweb URL in docs

Norman Ramsey is now at Tufts, and URLs to his websites at previous
institutions are broken. The new link gives the same information as
the old links did (verified with web.archive.org).

Hebrew tutorial: update link to Hebrew set up

Dekel Tsur has a webpage with information on getting set up with
using Hebrew with LaTeX and LyX. We link to that webpage from our
Hebrew Tutorial. He appears to have changed universities, and the
new link gives the same information as the old link did (verified
with web.archive.org).

LaTeXConfig.lyx: fix URL of AEA author guide

Our URL tests have been reporting the previous URL for a few months
as broken, and browser confirms it.

Math.lyx: fix URL of amsguide.pdf

Our URL tests have been reporting the previous URL for a few months
as broken, and browser confirms it.

UserGuide.lyx: accept and distribute today's changes

Create new method GuiWorkArea::Private:::updateCaretGeometry

This replaces a showCaret/hideCaret pair and avoids an update.

Also remove an update() call in resizeBufferView: is is not necessary
since we are already in a pintEvent handler.

cmake tests: Try to use valid locales for testing

Restore compilation of ja/Additional.lyx

Remove a system-dependent path.

Add suffixed lyxconvert to cmake build.

This should amend f93ec4a, but is not tested on apple

Set default format to "default" for Japanese docs

Consistent with 27165841.

Add Qt-based fallback-converter for Mac to compensate missing ImageMagick convert utility

Improve the logic of caret repainting

For some reason the existing code only considered the bottom row that
contained the cursor. There is no need for that, and actually it
caused painting problems in nested insets.

Tweak the logic of repaint_caret_row_ a bit: there is no need for
repainting when there is currently no caret on screen.

Avoid empty space above the top of the document

If the height of the first row of the document has become smaller (for
example when it contains an image which size has shrunk), that the
anchor offset has to be adapted so that there is not a large blank
space at the top of the document.

Fix crash when closing master with children and grandchildren

In some cases, it is possible that the BufferPositionMap has
dangling pointers. We thus check whether the Buffer is loaded
before accessing it.

Fixes: #10766
(cherry picked from commit 4f50cbcfe405a840459be8682fd4c7521b235798)

Fix bad refresh when changing zoom level

Replace the tricky code in LFUN_SCREEN_FONT_UPDATE and replace it with
proper use of DispatchResult flags.

LFUN_BUFFER_ZOOM* does not need to call LFUN_SCREEN_FONT_UPDATE, since
it already does everything that is required.

Whitespace cosmetics

(cherry picked from commit 745b43784ce1b6981fd31f6c5f5fbd489eef51f4)

Cleanup some icons

- remove empty text
- convert texts to paths