feat: Math inline content#2878
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Summary
This PR adds math inline content to alongside the math block. The UX is similar but differs in determining when the popup is open. The math block can contain the text cursor and still have the popup closed, requiring an Enter keypress to toggle and Escape to dismiss. The inline math keeps the popup open while the text cursor is within in.
Rationale
It makes sense to ship both a math block and inline content together, as they are often used together and fulfil slightly different use-cases.
Changes
Not worth going in detail here, as files are likely to change after merging.
Impact
N/A
Testing
TODO after merging
Screenshots/Video
N/A
Checklist
Additional Notes
N/A