-
Notifications
You must be signed in to change notification settings - Fork 55
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
Possible merge? #49
Comments
Update:
|
Hi @talyssonoc, any interest in merging the changes by @MatejBransky into upstream? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi, I would like to find a common path where we can merge this repo (I used my starter for react lib and the source code is completely rewritten but the readme is forked from yours) with your repo. Do you think it's possible?
Differences:
<TeX>
which hasprops.block
for indicating block math (breaking change)peerDependencies
(so users don't need to wait for updating this lib, it should be their responsibility)<TeX />
component is memoized (newReact.memo(Component)
) by default so you can avoid unnecesary rerenders<TeX />
component accepts other props (likeclassName
,id
,style
,...) which are given to thespan
/div
props.settings
for KaTeX configurationreact-testing-library
What do you think?
The text was updated successfully, but these errors were encountered: