Skip to content

Remove qt-webengine dependency #40565

@MialLewis

Description

@MialLewis

Describe the outcome that is desired.

Remove qt-webengine as a dependency, as this package is not available on condo-forge.

Use QTextBrowser instead

QTestBrowser does not support JS, or CSS for that matter (uses QSS, which could be a complication): https://forum.qt.io/topic/105196/javascript-in-qtextedit/12

We currently use JS to render LaTeX images: https://github.com/mantidproject/mantid/blame/83a7b971adf2397bdbed2316e3431bcf913741bd/dev-docs/source/Standards/DocumentationGuideForDevs.rst#L29C4-L29C4
using MathJax: https://www.sphinx-doc.org/en/master/usage/extensions/math.html#module-sphinx.ext.mathjax

This puts math as-is into the HTML files which then uses MathJax, a JS package, to load the LaTeX live. We can switch to rendering the LaTeX as images and injecting the images into the HTML (at the sacrifice of package size).

We could use typst as suggested by Shervin: https://github.com/typst/typst

Or may be simpler to stick with sphinx, as we already appear to support the use of sphinx.est.imgmath to remove the JS dependency: https://www.sphinx-doc.org/en/master/usage/extensions/math.html#module-sphinx.ext.imgmath

if this approach does not work, we will fall back to using the users system browser.

Metadata

Metadata

Assignees

Labels

ISIS: CoreIssue and pull requests at ISIS that relate to Core features and functionsMaintenanceUnassigned issues to be addressed in the next maintenance period.

Type

No type

Projects

Status

Todo

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions