Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken tables in HTML master #3156

Closed
perost opened this issue Apr 27, 2022 · 7 comments · Fixed by #3208
Closed

Broken tables in HTML master #3156

perost opened this issue Apr 27, 2022 · 7 comments · Fixed by #3208
Labels
Generated HTML The generated HTML-code
Milestone

Comments

@perost
Copy link
Collaborator

perost commented Apr 27, 2022

It seems some tables are broken in the HTML-rendering of master, for example the inheritance table in 7.1.3 (note how expandable connector incorrectly spans several columns):
inheritance_table
Other tables with similar issues are e.g. the operator precedence table in 3.2 and many tables in 10 where entire columns are missing text.

I get the same issue in both Firefox and Chrome, and the missing text is actually missing in the HTML source code so it's not just a browser rendering issue. The PDF is fine though, as is the HTML for older versions of the specification.

@HansOlsson
Copy link
Collaborator

I believe it is best to start understanding the ones in chapter 10 - since the inheritance table has way too much special code; and my first guess is that it is a regression related to the double hline.

@HansOlsson HansOlsson added the Generated HTML The generated HTML-code label Apr 28, 2022
@henrikt-ma henrikt-ma self-assigned this Apr 28, 2022
@henrikt-ma
Copy link
Collaborator

OK. I put it on my to-do list to reproduce one of the problems in chapter 10 with a minimal example.

@sjoelund
Copy link
Member

It might also be related to the switch to texlive 2022

@henrikt-ma
Copy link
Collaborator

I guess it is, as I cannot reproduce any of these problems.

@henrikt-ma
Copy link
Collaborator

I guess we should still report this to LaTeXML, as it used to work with unguarded \lstinline?

@HansOlsson
Copy link
Collaborator

I guess we should still report this to LaTeXML, as it used to work with unguarded \lstinline?

Yup, already done; see link above.

@henrikt-ma
Copy link
Collaborator

Ah, I didn't pay attention to the automatically generated links. That's great!

HansOlsson added a commit to HansOlsson/ModelicaSpecification that referenced this issue Jun 21, 2022
@HansOlsson HansOlsson mentioned this issue Jun 22, 2022
HansOlsson added a commit that referenced this issue Jun 22, 2022
* Also need to handle first argument, or separator is lost.
Closes #3156
* All including first.

* Mistake

* FixLatex
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Generated HTML The generated HTML-code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants