openlibhums / openlibhums/janeway

Possible XSL regression for disp-formula

Open
#3,756 2 comments 0 reactions 1 assignee View on GitHub

@mauromsl is already working on this.

Since Oct 16, 2023.

bug dev-ready Size S XSLT
Dominant language
Python
Stars
238
Forks
97
Avg merge
9d 1h
Merged PRs (30d)
8

Description

Describe the bug

Andy patched the Material theme CSS in 2018 in #380 in such a way that indicates he expected each math formula to be inside an element with .disp-formula. At the time it seems our XSLT must have placed the equation inside this element:

https://github.com/BirkbeckCTP/janeway/blob/db71b7318ea56ff6df7537fef79c2c8ae6a6fe33/src/themes/material/assets/mat.css#L309-L311

However, the XSL now seems to place the math formulae outside .disp-formula:

image

This means Andy's patch stopped working and the bug reappeared in https://github.com/BirkbeckCTP/janeway/issues/3753.

Janeway version
d4999b8361e5b85ce44f891a691c2be73d497b1c

Expected behavior
The XSL perhaps should be placing the equations inside .disp-formula.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.