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

Changes to the final HTML document #196

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Ndot
Copy link

@Ndot Ndot commented Sep 25, 2024

Without being to opinionated on taste it adds the following:

  • padding on code blocks (for better readability)
  • center and restrict "id=content" element to a comfortable read size
  • images are set to "width: 100%" to fit properly
  • "margin: 1em 0" on table elements to prevent the scroll bar from
    overlapping the table last row
  • small border radius on elements with background, like images and
    code blocks (this one is a bit opinionated, but I argue those sharp
    corner could be dangerous)

Here's an image to see the differences:

org-msg-diff small

Also update all the function calls to work with the new arguments
layout.
Without being to opinionated on taste it adds the following:

- padding on code blocks (for better readability)
- center and restrict "id=content" element to a comfortable read size
- images are set to "width: 100%" to fit properly
- "margin: 1em 0" on table elements to prevent the scroll bar from
  overlapping the table last row
- small border radius on elements with background, like images and
  code blocks (this one is a bit opinionated, but I argue those sharp
  corner could be dangerous)
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.

1 participant