Tweak ruff configuration

Add I - isort rules (order of imports);
Ignore line too long warnings.

Add xcb lib if x11extras is detected

In this case we use directly some xcb api outside of the Qt framework
and hence we have to add the xcb library by ourselves.

Support qmake with Qt5 too

This extends the support of qmake to Qt5. Currently, the other
configuration methods are tried if qmake fails, but a message is
printed.

A nice goal would be to get rid of those legacy methods (pkg-config,
plain autoconf-style), if this proves possible.

Fix functions that used functions but did not defined it

The functions were selected from the parser files where document is
always defined.

The only case was this code was used was on document.warning.

Since our documents are well formed this code was never active and thus
the reason why we did not saw this before.

One possible solution to assure that these type of errors do not occur
is to make the convention that all the functions in lyx2lyx_tools have
as the first argument document.

Adapt to deprecation of \DeclareCollectionInstance

Change LaTeX output of smallLetterFrac. For more information, see:

  https://tex.stackexchange.com/questions/720579/do-i-need-to-replace-declarecollectioninstance-after-recent-package-change/

This fixes 44 failing tests from 074-76-letterlike-numberforms-arrows.lyx
that started failing after a tlmgr update around 2024-06-04.

more indentation fixes

GuiDocument: do not ask switch-back question on document closing

Fix indentation

Fix indentation

Remove undisclosed imports (from xxx import *)

This was the last example in this directory.

Add common pattern in linter exclusion list

# ambiguous-variable-name (E741)

Derived from the **pycodestyle** linter.

## What it does
Checks for the use of the characters 'l', 'O', or 'I' as variable names.

## Why is this bad?
In some fonts, these characters are indistinguishable from the
numerals one and zero. When tempted to use 'l', use 'L' instead.

## Reason to ignore in LyX?
With appropriated fonts that is not an issue. In our case we just use
the 'l', in particular in the context of iterators: 'i', 'j', 'k', 'l'.

Prevent division by zero

Fixes crash reported at
https://marc.info/?l=lyx-users&m=171842978728676

de/UserGuide: updates

Fix code

Run "ruff check --fix"

Remove unneeded imports;
Reword code to make clear the "not in" operator;
Put imports in its own line (readability);

Reformat lyx2lyx code using ruff

Add pyproject.toml file to manage some options for tools

This file is only relevant for tools that use it, in development, and so
it is not supposed to be installed. It does not harm but there is no
purpose.

Add comment from dev list.

* UG - note about gnuplot from Tobias Hilbricht

Fix depth bar optics

Update python requirements notice in INSTALL

update tex2lyx tests

Remove hebrew letter document class

The only difference from regular letter is the alignment of
"Send To Address" layout, but it no longer reflect the output
corrctly, probably because LyX revert the alignment of layouts
in RTL context now.

InsetMathMacoTemplate: recompute metrics on editXY too

This inset can change dimensions when edited (like macros can).
Therefore Update::SinglePar should be set with editXY() like it is
already set for edit().

Add "full" drawing strategy

With this patch, 3 draw strategies (set in prefs with
\draw_strategy partial|backingstore|full) are available:

- "partial": only draw the parts of text that have changed since last
  paint event (default for X11 and windows)

- "backingstore": the same, but drawing happens on an offspring
  pixmap; this breaks subpixel rendering (default for Wayland and
  macOS)

- "full": the screen is fully redrawn at each paint event and should
  therefore always be correct; this is presumably slower but
  introducing it will allow to test it.

This would deserve a proper UI eventually.

Update sk.po

Remove debug code that is not very useful as it is.

FindAdv: Omitting cursor advance at start of search if in mathed.

Fixes #13070 Advanced S&R does not find math symbols at start of document

Fix change tracking colors with RTL languages (#12923)

Fix greyedout with RTL languages (#8647)

Reformat imports for easier reading

This is in line with the current hints for better readability:

* one import per line;
* order the imports alphabetically to be easier to search;

Take the chance to remove unused imports.

Update generate contributions to Python 3+

Update Python scripts to Python 3+

Remove support for Python 2

Use formatted strings where appropriated

Update lyx2lyx folder to Python 3+

Remove support for Python 2
Take advantage of new features (euphemism) not possible before due to
python 2 compatibility

Update configure.py to Python3+

Remove Python 2 support

This simplifies the code and allows to use other more modern features,
like formatted strings, that were not possible while keeping Python 2
compatibility.

Update build time Python requirement to 2.8 (cmake edition)

Update Python build time requirement to 3.8

Python runtime detection requirement upgraded to 3.8

Remove -tt option that is a no-op in Python 3.

Remove search for Python 2

Brackets.

I read somewhere that comments after an if-statement without brackets
is dangerous.

Fix bug #13028.

Bindings for 'join lines.

ammend 64aad586f49b904

Remove an empty auxiliary file that snuck in (sorry...)

Fix textstyle-update handling in mathed

The code was expecting that all properties were specified
in the argument and not only some. Simply do not process
those properties not specified in the argument.

missing background in xetex and fix RTL stuff in preview

* previwes with xetex did not produce any background (\pagecolor does not work for some reason), adding the direct command for that, \special{background <color>} inside the preview box solved it.

* similar to #12923 we should push the color node in horizontal mode

* there is a bug in preview with LuaTeX with main RTL language, this commit fix this bug, but it should be removed when it is fixed upstream (the code related to \pr@set@pagerightoffset)

Return false on invalid input in Font::fromString()

Fixes crash with invalid font-update lfun

rename function

Fix Python warning of invalid escape in lyx_pot.py

Newer Python versions (e.g., Python 3.12.3) gave the following
warning:

  po/lyx_pot.py:607: SyntaxWarning: invalid escape sequence '\w'

An alternative fix would be to double the backslash, but that might
be less readable than this fix.

Implement InsetLayout::requiresPackage()

Text::dispatch: ignore single par update when there is a full update

Reorder the tests so that, when the flag Update::SinglePar has been
specified, it does not take precedence over Update::Force flag.

This fixes the crash with inset-split.

Fixup 7acfbe0fccc7: forgot to change one location

Do not draw appendix lines below en of document

This bug is pretty old, but it was not visible because the grey area
below the document would overwrite it.

It would be better to close the frame at the end of the document, but
this is for later.

Fixup de5f63eeb: the code did not do what it was supposed to

Strip quotes from PackageOptions. Amends a77c84a0b4d5

Update tex info in readme

Some comments into bind file

Use ranges

avoid adding redundant preamble to double column hebrew documents (#12919)

he.po

de.po

Revert "Update layouts"

This reverts commit 7c041af6425001d7779f5e44d72918d8591ea37d.

We will hold this back until we fix the issue discussed here:

  https://marc.info/?l=lyx-devel&m=171733833610242&w=2

Show full inset context-menu when clicking on text part of collapsible (#10370)

Re-fix this since #12907 has been resolved

Update sk.po

Adv. F&R: strange dialog text

Fixes #12997
Added 'Scope' to better indicate the searched area.

Fix bug #13066

Patch from Lorenzo

Fix table crash reported on Windows.

Update layouts

Whitespace

Support half und full line parskip class options (#12946)

Currently only employed by the KOMA classes.

Layout format change.

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

Fixes bug #12940.

Patch from Daniel.

Extend tab context menu features

Add

- Close Other Tabs
- Close Tabs to Left/Right
- Move Tab to Start/End
- Show Enclosing Folder

to the tabs context menus.

Fix for bug #11963

Fix bug #12934

Fix bug #11796.

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

Patch from Daniel.

Fix bug #8856.

Show values of cross-references in tooltips.

Remove setBuffer call in Cursor::push.

This is potentially slow. It also should be unnecessary after
some of JMarc's recent commits. See bug #13050.

Fix bug #12981

Fix bug #12981

update tex2lyx tests

Add support for glue length in parskip (#12867)

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.

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

Restrict the removing to math insets

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)

Wininstaller Qt6.7 changed file name

Cmake build: Initialize variable 'CMAKE_INSTALL_PREFIX' before use

Spotted by Eugene Chornyi <yu_jin@lyx.org>

(cherry picked from commit ea74574d8f02481732b591306410e3c49faae87d)

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

XHTML: in InsetGraphics, use the bounding box that LyX computes for each image instead of using the scale.

ctests: keep a DocBook test inverted for test file

DocBook is not currently expected to work with Beamer.

ctests: mark a fixed export test as fixed

The ed62816e test file was fixed at 6ef18c35.

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

Address Qt 6.7 deprecation warning

Initialize is_long_tabular before updateIndexes() (which employs it)

Limit the nopassthurchars case in beamer to URL

and also generally extend to classes that provide hyperref

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.

XHTML: fix typo in comment.

InsetGraphics: use the `scale` CSS property instead of `width` with percentages.

The effect is closer to what LyX offers. Previously, the percentage was of the HTML container (often, the whole page), meaning that images were oversized. Now, the scale is respected in the same way as LyX, but its bounding box is off. Overall, the result is better, but not what users expect.

Return a reference, not a copy.

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

ctests: automatically invert more lyxbugs files

Only files in the directory with a trac number prefixed were
inverted.

Now files with a Git short hash (length 7 or 8) are additionally
inverted.

Test file with knitr chunk in beamer

This file fails to compile starting with ed62816e.

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

Make comment clearer.

Cmake build: Set LYX_DATA_SUBDIR in cache unconditionaly

The value is wrong if CMakeCache.txt already exists and the version is
changed inside configure.ac in AC_INIT()

Comment typo

Fix up a4d9315b: handle metrics of not visible paragraphs

The code is not ready for situations where some paragraphs that are
not visible have metrics available. This has been made visible by
a4d9315b, which removes a couple of full metrics updates: the screen
was not correctly scrolled after a scrollToCursor() call.

In PararagraphMetrics, some methods are added to be able to handle the
fact that paragraphs have or do not have a position.

In TextMetrics, a new method returns the first visible paragraph.

Finally, in BufferView::updateMetrics, the paragraphs' positions are
reset (in the case where everything is not cleared) and some care is
taken to skip the ones that are not relevant.

The assumption outside of this method is that all the paragraphs that
are in the TextMetrics are visible (we are talking about top-level
TextMetrics here). This could be changed (in order to avoid
recomputing paragraph metrics), but the cost is high in terms of
complexity and it is not clear that the gain in terms of performance
would be important.