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

Add placement param to figures #266

Merged
merged 2 commits into from
Jul 31, 2020
Merged

Add placement param to figures #266

merged 2 commits into from
Jul 31, 2020

Conversation

jvntf
Copy link
Contributor

@jvntf jvntf commented Jul 29, 2020

LaTeX floats objects like images because they should not be broken over pages. This means that by default it will decide where to put an image in the layout. I have added the h parameter to the figures in the latex markup which indicates place approximately here see docs. This should address #257, this is somewhat tough to test because of image upload problems.

I think this might be sufficient? TeX often requires tends to require additional commands inserted into the template to tune a layout to what the author wants, so it is hard to allow complete control from the front end without creating a 1:1 correspondence with LaTeX commands.

@jvntf jvntf requested a review from danielfdsilva July 29, 2020 15:18
Copy link
Collaborator

@danielfdsilva danielfdsilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The image is still not respecting the positioning.
To test:

@jvntf

@jvntf
Copy link
Contributor Author

jvntf commented Jul 31, 2020

I think that is the effect of the h param, LaTeX will still take some liberties. I changed it to H which is expected to be precise. @danielfdsilva

@danielfdsilva danielfdsilva self-requested a review July 31, 2020 15:53
Copy link
Collaborator

@danielfdsilva danielfdsilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as expected

@jvntf jvntf merged commit ef34890 into develop Jul 31, 2020
@jvntf jvntf deleted the fix/257-image-placement branch July 31, 2020 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants