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

Fix table display in markdown viewer #193

Merged
merged 2 commits into from Aug 27, 2021
Merged

Fix table display in markdown viewer #193

merged 2 commits into from Aug 27, 2021

Conversation

drewbutlerbb4
Copy link
Contributor

Signed-off-by: Andrew-Butler Andrew.Butler@ibm.com

Tables were displayed incorrectly in the markdown viewer due to difficulties displaying Github-flavored markdown. This PR uses the remark-gfm plugin to parse the table and some specialized css to deal with table formatting issues.

Signed-off-by: Andrew-Butler <Andrew.Butler@ibm.com>
Signed-off-by: Andrew-Butler <Andrew.Butler@ibm.com>
@ckadner ckadner self-requested a review August 26, 2021 21:26
@ckadner
Copy link
Member

ckadner commented Aug 26, 2021

Hi @drewbutlerbb4 -- could you throw in a screenshot of the before and after to show the different table renderings?

@drewbutlerbb4
Copy link
Contributor Author

Before:
Screen Shot 2021-08-27 at 10 59 14 AM
After:
Screen Shot 2021-08-27 at 10 47 12 AM

@drewbutlerbb4
Copy link
Contributor Author

@ckadner Pictured above

@ckadner
Copy link
Member

ckadner commented Aug 27, 2021

@ckadner Pictured above

Thanks Andrew. That is a big improvement!

Copy link
Member

@ckadner ckadner left a comment

Choose a reason for hiding this comment

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

/lgtm

@mlx-bot
Copy link
Collaborator

mlx-bot commented Aug 27, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ckadner, drewbutlerbb4

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [ckadner,drewbutlerbb4]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mlx-bot mlx-bot merged commit 37ccef6 into machine-learning-exchange:main Aug 27, 2021
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.

None yet

3 participants