8000 Update LaTeX writer to align images by frasertweedale · Pull Request #2724 · jgm/pandoc · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update LaTeX writer to align images #2724

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

frasertweedale
Copy link
Contributor

Update the LaTeX writer to align images that have an "align"
attribute whose value is "left", "right" or "center". If the
attribute is absent or has an unrecognised values, the image is left
alone.

Also update the RST reader to read these horizonal alignment values
from the image directive (the first encountered value applies). See
http://docutils.sourceforge.net/docs/ref/rst/directives.html#image
for details.

Update the LaTeX writer to align images that have an "align"
attribute whose value is "left", "right" or "center".  If the
attribute is absent or has an unrecognised values, the image is left
alone.

Also update the RST reader to read these horizonal alignment values
from the image directive (the first encountered value applies). See
http://docutils.sourceforge.net/docs/ref/rst/directives.html#image
for details.
@bluebirch
Copy link
Contributor

+1

@mb21
Copy link
Collaborator
mb21 commented Oct 16, 2016

Although I cannot speak for @jgm, I guess for this to be merged, it would have to be implemented in the most commonly used writers (i.e. HTML probably emitting a style="text-align: left") and documented in the README.

@mb21
Copy link
Collaborator
mb21 commented Apr 9, 2018

Then again, 99b39ff meanwhile set a precedent to simply store alignment in the AST as simply a align-left or align-right class on the Image element...

@wallento
Copy link

Stumbled over this as a blocker in my work today. Anything I can do to help move it forward?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0