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

Images cannot float in EPUB rendering #405

Closed
Thanatermesis opened this issue Aug 2, 2021 · 5 comments
Closed

Images cannot float in EPUB rendering #405

Thanatermesis opened this issue Aug 2, 2021 · 5 comments
Milestone

Comments

@Thanatermesis
Copy link

Thanatermesis commented Aug 2, 2021

When trying to align an image with text is not possible, I have tried all the possible options without success
Example code:

== Test section
.Title for image
image::einstein-relatividad.jpg[,340,role="related thumb left"]
Lorem Ipsum is simply dummy text of the printing a.....

So the HTML rendering (asciidoctor) renders very good, like:

HTML:

image


But for PDF or for EPUB3, it always renders it in a strange and unfriendly way (I tried different options for the images, nothing found to work nicely yet):


Epub:

image


PDF:

image

@mojavelinux
Copy link
Member

Can you please repost your entire report here?

@Thanatermesis
Copy link
Author

done

@mojavelinux
Copy link
Member

My advice here would be to go ahead and add the same styles that are present in the default stylesheet. The reason I didn't add them originally is because many ePUB readers won't honor them. But that's really up to the writer to decide whether to use layouts that are only supported in a handful of ePUB readers. Caveat emptor.

@Thanatermesis
Copy link
Author

So what would happen in the case an epub reader (let's say "kindle") cannot read this style? it just switches to the default mode ? (first image, then text) or it directly don't show it? so if fallbacks to a different styling i don't see the problem

@mojavelinux
Copy link
Member

It would just ignore the request to float the image and display it as it currently appears.

@slonopotamus slonopotamus added this to the v1.6.0 milestone Dec 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants