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

In the direction of #190 - fixing table format for PDF render #194

Merged
merged 1 commit into from
Oct 9, 2021

Conversation

princefishthrower
Copy link
Contributor

@princefishthrower princefishthrower commented Oct 9, 2021

What type of PR is this? (check all applicable)

  • ♻️ Refactor
  • ✨ New Chapter
  • 🐛 Bug Fix/Typo
  • 👷 Optimization
  • 📝 Documentation Update
  • 🚩 Other

Description

As the title states, I've updated the mstat command flag table so it will render properly in the PDF - just as an example. Actually, all over this Ebook a form of markdown tables (which is indeed valid form) is not being parsed correctly by ibis when rendered to PDF.

Related Tickets & Documents

I've opened a ticket on ibis so that this other form of table format can be used:

themsaid/ibis#47

I believe creating a new parser on ibis to support the table format frequently used on this repo will be faster than going through and converting all the existing tables in this Ebook to the 'other' table format. 😄

An older issue on the ibis repo discussing the formatting currently possible with ibis for tables:

themsaid/ibis#27

Added to documentation?

  • 📜 readme
  • 🙅 no documentation needed

[optional] What gif best describes this PR or how it makes you feel?

thinking

@bobbyiliev
Copy link
Owner

Thank you for fixing this! This is a good catch! I will have to go through the chapters and correct that for all :D

@bobbyiliev bobbyiliev merged commit c5624b3 into bobbyiliev:main Oct 9, 2021
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