Upstream merge conflict #1
Draft
+487,304
−127,165
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Spelling correction
Add extra warning for GCC compiler
…ope once a namespace is used
warning: possible misuse of comma operator here [-Wcomma]
Removed some commented out css entries (were already gone / removed in the dark version)
One more
Remove commented out css entries
Remove unused css settings / definitions
…ettings Remove unused css settings
Prevent page break after the top line plus accompanying horizontal line in a doxygen parameter like type of table.
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4 to 5. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v4...v5) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
When having `PDF_HYPERLINKS = NO` specified (and having some tables defined) doxygen will give the error: ``` ! LaTeX Error: Environment NoHyper undefined. ``` when generating the pdf file. By defining a dummy `NoHyper` environment this problem can be overcome.
Undefined environment NoHyper
…ons/upload-artifact-5 build(deps): bump actions/upload-artifact from 4 to 5
…_head Prevent page break in doxygen parameter table after head.
Added possibility to use the `\ref` command in the `<summary>` tag
Add -Wcomma option for Clang
…s list of tables Added `entry=none` in case of no caption so tables without a caption won't be shown in the list of tables in case `\listof tables` is added to the latex processing
issue #11837 Tables in latex has empty caption that results in useless list of tables
…cies Remove jQuery dependencies from clipboard.js
…cies-navtree Replace jQuery dependencies with vanilla JavaScript in navtree.js
…ections_tooltips.js Co-authored-by: doxygen <1300762+doxygen@users.noreply.github.com>
Consistency corrections regarding usage of extensions. (Removed an incorrect `s` in config.xml)
# Conflicts: # src/docnode.cpp # src/docparser.cpp # src/docparser_p.h
…cies-again Replace jQuery with vanilla JavaScript ES6 in dynsections.js and dynsections_tooltips.js
Co-authored-by: David Hebbeker <david.hebbeker@gmail.com>
Consistency corrections regarding usage of extensions.
* Initial plan * Replace jQuery with vanilla JavaScript in darkmode_toggle.js Co-authored-by: doxygen <1300762+doxygen@users.noreply.github.com> * Replace deprecated keyCode with modern key property Co-authored-by: doxygen <1300762+doxygen@users.noreply.github.com> * Fix timing issue: mimic jQuery ready() behavior with readyState check Co-authored-by: doxygen <1300762+doxygen@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: doxygen <1300762+doxygen@users.noreply.github.com>
Bumps [docker/login-action](https://github.com/docker/login-action) from 3.6.0 to 3.7.0. - [Release notes](https://github.com/docker/login-action/releases) - [Commits](docker/login-action@5e57cd1...c94ce9f) --- updated-dependencies: - dependency-name: docker/login-action dependency-version: 3.7.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
…er/login-action-3.7.0 Bump docker/login-action from 3.6.0 to 3.7.0
Small precision regarding usage of the code span limits regarding `` ` ... ' ``
When having something like: ``` #if !defined(__linux__) /* @code */ #endif ``` we get a warning like: ``` warning: More #if's than #endif's found (might be in an included file). ``` as an attempt is made too match `@code` with `@endcode` even in a non doxygen comment
Documentation code span limits
Mismatch `#if` / `#endif` when "verbatim" command in non doxygen comment
Replace jQuery based smartmenus with vanilla JavaScript in menu.js ----- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Dimitri van Heesch <doxygen@gmail.com>
When having something like: ``` ## A fie def aa_fie1: # Inside fie with `tick pass ``` we get a warning like: ``` a.py:8: warning: Reached end of file while still searching closing '`' of a verbatim block starting at line 6 ``` this is not correct as line 8 is non-doxygen comment and should not be inspected.
Incorrect warning from non-doxygen comment
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Dimitri van Heesch <doxygen@gmail.com>
When having something like"
```
/// \file
/// \{
void foo_g();
void bar_g();
void baz_g();
/// \}
/// \brief text
/// \{
void foo_b();
void bar_b();
void baz_b();
/// \}
```
The spacing in the LaTeX brief description between the 2 (anonymous) groups is a bit large, this is corrected
Doxygen's LaTeX documentation has in principle 2 table of contents: - left navigation tree - main table of contents as first "Chapter" named "Contents. Especially the main table of contents is a bit large due to the sections and subsections in the appendices (in the navigation tree these are by default collapsed). I makes sense to remove the sections etc. from the mentioned table of contents for the appendices and have each appendix have its own local table of contents. See also: https://tex.stackexchange.com/questions/759482/local-table-of-content-versus-the-main-global-table-of-content
Large white space between grouping without names
More readable table of contents in doxygen's LaTeX documentation.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Upstream merge conflict with the fork by @shahmoradi