unicodesymbols Force substitution of NNBSP in utf8.

NNBSP (narrow no break space) is supported by LyX's unicodesymbols
but missing in LaTeX inputenc *.dfu definitions. (TeXLive 2017)

Try to compute row height like it should be done

Currently, our computation of row height is not completely standard:
* we ignore completely the QFontMetrics::leading() parameter

* we add arbitrarily 2 hardcoded pixels to the height.

This patch reverses these two choices, which leads to
* slightly larger spacing for MinionPro (which has a big leading).

* an additional spacing of 20% font height that depends on dpi and zoom.

Visual inspection with LibreOffice seems to imply that it disregards
the font leading but uses a interline which is 20% larger than the
font height.

Fix unicodesymbols.

Remove 4 spurious tags that slipped in in [4ebb415/lyxgit].

Update some special latex export tests.

Update unicodesymbols replacements

"textcomp" supports some more characters in its ts1enc.dfu Unicode definition file.

Development.lyx: check that PDF is good before inv

Before uninverting a test, we should check that the PDF looks
correct.

Development.lyx: remove an unmatched parenthesis

Allow to type a space in front of another one

This is a follow-up to 714b731e. This fixes the issue when the cursor
is in front of a space
  abc| def
and one inserts another space (to start a word). DEPM would eat one
space and produce
  abc |def
instead of
  abc | def

Additionally, fix the same_par boolean, wheich did not take the cell
index into account.

Related to ticket #11412.

ctests: update test tagging/categorization and export test for accents on Cyrillic letters.

ctests: ignore latex8 LyX 2.2.x and 2.3.x tests

The cls is obsolete, so all PDF exports are disabled. We keep the
1.6.x and 2.1.x tests because those only check roundtrip
convergence, and do not export to PDF.

ctests: ignore minted 2.2.x tests

As explained by Enrico, backward compatibility is not guaranteed for
these cases (and indeed currently fails):

  https://www.mail-archive.com/search?l=mid&q=20190113085555.GA7928%40GIOVE

ctests: add comment about formats that won't work

This way we will not try to test those formats for this file in the
future.

Fix copy-paste error

Amend 7b639d2db212

Not the (ASCII) encoding matters here, but the (TU) font encoding.

Script wrapper is needed with ASCII encoding

Remove PYTHONPATH from the environment on Windows

This make sure to use the the modules distributed with LyX,
avoiding reconfigure failures.

Replace environment variables in PATH prefix

The PATH prefix modified through the GUI was set without
replacing embedded environment variables. This might have
caused problems on Windows with external python installations.

Add support for U+2693 (ANCHOR)

Fixes: #10706

Fix alignment of rows when text width is variable

When several lines of text are in the same variable-width tabular
cell, it is not possible to align properly the rows until the cell
width is known.

Therefore a parameter is added to redoParagraph to skip this
computation, so that it can be done later in TextMetrics::metrics.
Other calls to redoParagraph in the code are not affected. It is not
clear at this point whether they may create artefacts.

computeRowMetrics has been renamed to setRowAlignment to better
reflect its use.

Fixes bug #11447.

Amend 23f7431c: Renamed files

latex/inputenc-utf8-plain.lyx -> latex/luainputenc-utf8.lyx

"Script chars" also need to be output as macro with full utf8 engines and TeX fonts

Fixes rest of #9681

ctests: update sorting and documentation of inverted and ignored tests.

Remove empty file discovered by Kornel.

Looks like an accidental commit.

> git log GuiGraphicsUi.h
commit a1cec91afaca91968b46e695533c10ad2a3f73d3
Author: André Pönitz <poenitz@gmx.net>
Date:   Fri Aug 31 05:53:55 2007 +0000

    move our stuff off the Q* namespace

    git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@19935 a592a061-630c-0410-9148-cb99ea01b6c8

That's all a rename, basically. The original file, QGraphicsUi.h, was commited at 12e5a52b92, and it was empty then, too.

Force more symbols with utf8-cjk

ctests: Update/clean up latex export tests after fix of #9681.

lyx2lyx: Minor formatting fix.

Force more symbols with utf8-cjk

force some symbols as macros with utf8-cjk

Probably more needed.

ctests: uninvert two 008-greek-and-coptic tests

The following two exports now succeed, likely thanks to recent
commits:

  008-greek-and-coptic_(utf8|utf8x)_pdf2

ctests: move IEEEtran-Journal_lyx16 to inverted

It used to give an endless loop, so we "ignored" it (did not run the
test). Now it gives a lyx2lyx warning, which is reported at #11455,
so it is appropriate to invert the test.

unicodesymbols: support for LOW ASTERISK, use textcomp for (non-combining) Breve and Caron

The textcomp Unicode support file "ts1enc.dfu" defines 0x204E Low Asterisk
as \textasteriskcentered. LyX should follow suit.
The ASTERISK OPERATOR (correctly) maps to the same macro,
the "deprecated" tag marks the upstream mapping as preferred choice.

ctests: Dedicated LaTeX export tests for problematic input encodings.

Remove omikron from math tests.

The letter Omikron is not available in TeX math mode because
it looks like the latin O and hence not used as mathematical symbol.

Use LatinModern fonts for Greek documentation Intro.

Since some years, the LatinModern 8-bit fonts work well with Greek,
using the high quality CB-Fonts as substitution (just as ComputerModern).

Fix Ticket #9904 ("strange dots" with input encoding iso8859-7).

Added "force=iso8859-7" for some characters:

The iso8859-7.def file for the Greek 8-bit input encoding
used \textbullet as placeholder for non-defined characters.
This is fixed in v1.7 2019/01/08.

Once the fixed version is in common use, all "force=iso8859-7"
tags can be removed.

Fix handling of Thai tis620-0 input encoding.

The Thai tis620-0 input encoding is supported via the inputenc "plug in"
(data) file tis620.def from https://ctan.org/pkg/babel-thai.

We can handle it like the other contributed input encodings, e.g.,
Greek (ISO 8859-7) and the several Cyrillic encodings from
http://www.ctan.org/pkg/latex-cyrillic.

Under TeXLive 2018, the input encoding defaults to utf8, if there is no call to
inputenc. The added test file fails without the patch but compiles fine, if the
file "tis620.def" is present in the TEXPATH.

CJKutf8 is only to be used with the dedicated utf8-cjk encoding

ctests: ingore a utf8-cjk test instead of invert

The 001-4-latin_utf8x_pdf2 test passes and the
001-4-latin_utf8-cjk_pdf2 test fails, which means that there are
characters in the .lyx file that are only available with
utf8-extended encoding, so the utf8 test is never expected to pass
in the future.

Thanks to Kornel.

ctests: improve a regex

The "004" did not actually match anything, so there is no change in
functionality.

This commit amends c2a99965.

ctests: utf8-plain now works with all export variants.

utf8-plain (Unicode (utf8 XeTeX)) is a power-user setting
for the input encoding with two use cases:

a) setup of system fonts or

b) setup of input encoding supportuser preamble

in the document class or user preamble.

The test file is an example for use case b.

ctests: uninvert CJK exports that now succeed

These tests were likely fixed with d193cd05. There are still some
failing CJK tests, but many were fixed.

Improve readability

No functional change.

use range-based for loops

When using polyglossia (bidi), paragraph directions don't need to be swapped

Bidi does that.

Fixes: #11399

Enable error-free compilation of ko splash.lyx with xetex and lautex

This amends somehow 9238004c. Since the system font NanumMyeongjo
and the "mj" spec as CJK latex font are related, this change
seems appropriate.

Remove outdated comment

Remove now unused routine.

Use combo box data rather than looking up info from InsetRef.

ctests: uninvert ko splash.lyx dvi, pdflatex tests

These exports now pass thanks to the recent CJK improvement at
d193cd05 and specifying "mj" as CJK font (0cfaf406).

ko splash.lyx: specify "mj" as CJK font

Now (thanks to d193cd05), the Korean splash.lyx compiles
out-of-the-box with pdflatex.

Thanks to Jürgen.

Add comment.

Remove debugging code.

Fix problem caused by re-ordering of menu at 5f6332bf4.

This broke the activation and de-activation of plural, capitalize,
etc. I guess that tells us how much those get used....

Don't output CJK environment with non-TeX fonts

This amends d193cd05a8e3b.

Amend 348b3e5e49

thinko!

Properly implement CJKutf8

If we use that, the document actually needs to be in utf8 encoding, and
the CJK environment needs to account for it.

Candidate for stable.

Add license info for inkscape start script

ctests: ignore Bibliography.lyx PDF export tests

Bibliography.lyx is not expected to be compiled on its own. See:

  https://www.mail-archive.com/search?l=mid&q=20190104195414.opgeuggeqakxl2fo%40barna

ctests: invert ar UserGuide.lyx lyx22x and lyx23x

These tests are expected to fail since the default export is
expected to fail.

Correct converter configuration for inkscape on Mac

On Mac the inkscape binary is started by a wrapper script. This script changes the
working directory internally and fails to process files with relative path names.
The previous attempt to solve it was to pass the file names with absolute names
by prepending them with the $$p variable (representing the directory name of the files).
This broke the on screen conversion (used for SVGZ to PNG e.g. in the users guide)
because here the $$p variable is undefined.

Now the wrapper script of LyX which is used to locate the Inkscape.app bundle converts
the relative path names into absolute names and the $$p variable is removed from the
converter definitions for inkscape again.

ctests: invert ko splash lyx22x and lyx23x tests

The Korean splash.lyx is expected to fail with pdflatex. The lyx22x
and lyx23x tests were not failing before because they were exporting
to XeTeX with system fonts, which succeeds. After c9e62dec (which
corrects the export format to the default), the lyx22x and lyx23x
tests should be inverted.

Bibliography.lyx: remove preamble and master doc

The preamble is not used since this document should only be compiled
from documents that include it. Similarly, setting the master
document is not useful.

Update it.po

Avoid crash with gzipped file

In computing the length of the extension, the code does not account
for the prefix "unzipped_", which is added when the zipped filename
does not have one of the extensions "gz", "z", "Z", or "svgz", and
thus the used index is out of bounds. See also this thread:
https://www.mail-archive.com/lyx-devel@lists.lyx.org/msg207360.html

Cmake lyx2lyx tests: Correct use of value for \use_non_tex_fonts

Amend eec3d1eb,7568571a
We are trying to check if the resulting file after lyx2lyx
is compilable, we have to ensure that copying the original file
to the test directory does not mangle use_non_tex_fonts-entry.

ctests: do not invert a test for 2.3.x format

The test 'export/lyx2lyx/lyx_2_3_test_lyx' is expected to fail for
formats before 2.3.x, but is expected to pass for 2.3.x.

ctests: extend a regex to cover 2.3.x export

The tests of "attic/aastex6_sample" are ignored for previous LyX
format exports, and should also be ignored for the newly extended
2.3.x tests.

Bibliography.lyx: remove citation, add explanation

Bibliography.lyx is not expected to be compiled on its own. This
commit reverts 3ac2dd8e and instead adds a LyX note explaining that
an error is expected if the .lyx file is compiled.

Fixup 5c1d65af: proper spacing between text and label in rtl

Fix bug #11425.

Fix drawing of labels in insets

In many cases, the origin xo_ of insets was forgotten.

Fixes bug #11425.

Revert "ctests: invert JSS-article lyx2lyx tests"

This reverts commit 5ad0395dca5876672d959f3ac65f66ac4e18f8c5.

rewrite revert_jss reversion routine

Fixes: #11448

Avoid crash when using inset-select-all in tabular

When changing cell idx, it is important to set the pit to a reasonable
value.

Remove unused mathod (CursorSlice::at_cell_end is the same)

Always update bibfiles cache with bibliography environment before opening citation dialog

We need to do this unconditionally, since there is no (trivial and non-
awkward) way to update it when the bibliography paragraphs are modified.

Fixes: #4899

lyx2lyx tests: Add conversions to lyx23

ctests: invert JSS-article lyx2lyx tests

The regression has been reported at #11448.

lyx2lyx tests: Use the temp-dir also for converted lyx files

Important if including data previously copied there.
Fixes test "export/doc/Additional_lyx22"

Bibliography.lyx: add a reference so it compiles

Fix compilation of chess-article.lyx with TL 2018

Consistent with 32b77f1a.

Also pay attention to OutputParams::free_spacing.

Fix bug #11432.

Take account of Layout::free_spacing.

Fix vertical offset of fonts like MinionPro

For some reason, adding a LTR/RTL Override unicode character before a
MinionPro string increases the ascent of the line.

This creates vertical offset issues when painting if the ascent used
as reference is the one of the font. Use the QTextLine ascent instead
for better results.

This is a followup to 1bed76e2a.

Fixes (reopened) ticket #11284.

Update tex2lyx tests

Fixed remaining glitches in tcolorbox reversion routines

The indexes were off at several places, which made the routines leak.

lyx2lyx tests: Copy also original lyx files to temp dir

Necessary only if the lyx-file includes some data from
a relative path, nonetheless it happens in some of our tests.

maa-monthly.layout: declare dep maa-monthly.sty

ctests: restore thesis.lyx lyx2lyx tests

These tests failed because of (multiple instances of) the following
warning:

  Warning: a buffer should not have two parents!

This warning is expected, because, for example, chapter-1 includes
Bibliography.lyx and chapter-2 includes Bibliography.lyx. thesis.lyx
includes both chapter-1 and chapter-2, so both are loaded (but
hidden), and Bibliography.lyx is loaded (but hidden), and LyX then
sees that more than one buffer includes Bibliography.lyx

The consequence of ignoring this warning is that we will not catch
any future regression that triggers this warning when it should not.

Fix compilation of chessgame.lyx with only TL 2018

chess-workshop-symbols.sty is no longer (on an updated TL 2018)
included in the TL package "skak". This commit replaces the loading
of chess-workshop-symbols in the custom preamble with the loading of
the package chessfss.

chessfss is in TL 2016 and TL 2018, and the example file now
compiles with both TL installations.

Before this commit, compilation of chessgame.lyx was possible if
chess-workshop-symbols.sty were installed manually (i.e., outside of
TL), but it is preferred to keep dependencies to only TL if
possible.

Add override statements to please clang

Eventually, all overridden virtual methods should be marked properly.
Currently, clang only warns about those in classes that already use
override in at least one place (which was the case dor GuiApplication).

Remove premature (and bad) optimization

When blinking the caret, it looks like a cool idea to only update the
small rectangle containing the caret. Actually it is an awful idea, since
the paint event will paint some rows, and these rows will not be
properly painted outside of the small rectangle. Unfortunately, the
painter will skip those "painted" rows on next paint events.

This leads to painting errors that depend on the ordering of "real"
and "caret" paint events. This is the reason why they only appeared
with split screens.

Quote of the day: ``The real problem is that programmers have spent far
too much time worrying about efficiency in the wrong places and at the
wrong times; premature optimization is the root of all evil (or at
least most of it) in programming.''
-- Donald Knuth, The Art of Computer Programming.

Fixes bug #11354.

Amend 12aaeb25, add Bibliography.lyx to the set of not tested subdocuments

fix tcolorbox reversion routines

Fix conversion of math braces to html

Fix Include dialog UI

#11443

Amend [2dcb7300/lyxgit]

Avoid a crash by checking that a cell is not empty before
accessing its content.

Fis some old box reversion routines.

Mark labels in non-outputting insets as inactive.

Fixes: #11437

Add inactive labels to the toc

It makes sense to navigate to them.

They are marked as inactive with the cross.

Fix gotoLabel with inactive labels

This now frist looks for the active label. Only if none is found, it
jumps to the (first) inactive label.