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

Feat/readme #99

Merged
merged 4 commits into from
Aug 22, 2024
Merged

Feat/readme #99

merged 4 commits into from
Aug 22, 2024

Conversation

ThijsZijdel
Copy link
Contributor

Readme feature editor is mostly done. Will inherit the extensions of the main MD editor in a later PR.

[Feature discussion] Readme generator #94

Screenshot 2024-08-20 at 23 13 11

@@ -0,0 +1,31 @@
import React from "react";
Copy link
Member

Choose a reason for hiding this comment

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

Great addition, I will rewrite other places with this hook after merge.

"react-live": "^4.1.5",
"react-pdf": "^7.7.3",
"react-router-dom": "6.22.3",
"react-shepherd": "^4.3.0",
"react-sizeme": "^3.0.2",
"recharts": "2.12.3",
"rehype-katex": "^7.0.0",
Copy link
Member

@Sparkenstein Sparkenstein Aug 22, 2024

Choose a reason for hiding this comment

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

@ThijsZijdel have we included example tempaltes of how to use these libraries in the the module? Ideally it'd be great if user knows somehow that they can write math markup.

also why two different plugins? isn't rehype-katex same as remark-math?


So you can use *markdown* and <em>HTML</em>.

$\\frac{-30}{8}$
Copy link
Member

Choose a reason for hiding this comment

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

ah here's math syntax I guess, looks goot to me

</Button>
) : null}
</Group>
<div style={{ width: "100%", height: "95%", display: "flex" }}>
Copy link
Member

Choose a reason for hiding this comment

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

PS: Mantine has Flex or Group, if you just want a div with display:flex.
you can also do <Box display="flex"> for simplicity if you want.

{Object.keys(templates).map((key) => {
const template = templates[key as keyof typeof templates];
return (
<div
Copy link
Member

Choose a reason for hiding this comment

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

need key

Copy link
Member

@Sparkenstein Sparkenstein left a comment

Choose a reason for hiding this comment

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

LGTM. merging

@Sparkenstein Sparkenstein merged commit 26c3e8f into fosslife:master Aug 22, 2024
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