WIP: 2.0 #324
Merged
WIP: 2.0 #324
Conversation
This file contains 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
This was referenced Jun 10, 2018
Closed
bfirsh
force-pushed
the
2.0
branch
18 times, most recently
from
9e91a7d
to
1357efd
Compare
bfirsh
force-pushed
the
2.0
branch
3 times, most recently
from
9930649
to
4110335
Compare
bfirsh
force-pushed
the
2.0
branch
4 times, most recently
from
81bcc4f
to
63587ed
Compare
Merged
bfirsh
force-pushed
the
2.0
branch
2 times, most recently
from
c7b3424
to
94dcded
Compare
In Parcel 1.9 they fixed a bug that was actually a feature for us. Previously, absolute paths were absolute filesystem paths. They are now absolute relative to the project. So, we have to put the output in the project somewhere.
This removes support for custom bullets, which I don't really want to allow anyway.
Previously, `yarn convert` would use the built assets inside the image instead of the locally built version.
Also, convert to ES6 modules. This allows us to run the client JavaScript on DOM fragments instead of just entire pages.
This is so they work for Storybook, but also simplifies the snapshotting code a bit.
This makes it easier to reference from Storybook.
bfirsh
force-pushed
the
2.0
branch
2 times, most recently
from
8c78af3
to
3264964
Compare
bfirsh
force-pushed
the
2.0
branch
2 times, most recently
from
a3b90e3
to
ca21260
Compare
Mainly based on a design by @markhurrell, using TeX's Computer Modern font.
|
Design is functional, but still a work in progress. Needs a lot of spit and polish, and some stuff is flat out broken. Arxiv Vanity is now pinned to an old version of Engrafo, so merging so we can continue work in master |
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.
An implementation of #255.
Background
Currently, Engrafo works a bit like this:
That's really complicated. The goal of this pull request is to not rewrite the HTML (or rewrite as little as possible) and instead write some CSS to style the LaTeXML HTML directly.
Stuff that needs to be done
normalize.css?Add normalize.css to ensure everything is consistent cross-browser #407deal with things that are too wide on mobile (tables, images, math)Ensure wide things never break mobile layout (tables, images, math, etc) #408Hover boxes for ltx_author_notesMake hover boxes work for .ltx_author_notes #406Things to fix in latexml
This is a list for myself to keep track of things we should fix upstream.
Abstract, contributing, references headings should be semanticallyMake abstract, references, etc <h2> brucemiller/LaTeXML#1042<h2>not<h6>Subfigures and so on use tables. This is suboptimal. There must be a better way (floats, flex, etc).HTML subfigures via flexbox rather than tables? brucemiller/LaTeXML#979Ref links should be bigger - e.g. in a ref that reads "figure 5", the link is just "5", but should be the whole "figure 5". It doesn't look very good and is really fiddly to click.Reference links should be bigger brucemiller/LaTeXML#1043.ltx_missing_citationwould be useful on citation, not the ref number withinThere is no way to put javascript at bottom of body, which is the most efficient way to load javascript in old browsers.It should be possible to insert JavaScript at the end of the <body> brucemiller/LaTeXML#1044Future todo
See also
Closes #255
Closes #10
Closes #86
Closes #246
Closes #321
Closes #32