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

Correct typo in mimetype variable #11

Merged
merged 22 commits into from
Apr 3, 2024
Merged

Conversation

elipousson
Copy link
Collaborator

Also rename setPreD2RenderOptions helper to setD2Options and add setD2outputPath helper.

Also fix bug impacting diagrams when embed mode is linking and document is inside an RStudio Project

elipousson and others added 22 commits February 12, 2024 15:10
Also add DarkMauve and DarkFlagshipTerrastruct as explicit theme options
DarkMauve was already in the theme list
Also:

- Adds is_nonempty_string helper function
- Appends random number for diagram file name
- Use d2 file extension not txt for input
- Allow alternate syntax for attribute classes on code blocks
Also:

- Add support for reading d2 diagrams from external files using `file` parameter. Block text is ignored if file parameter is supplied.
- Add support for alternate code block syntax without curly braces.
- Insert rendered diagrams into the Pandoc mediabag when `embed_type="link"`
- Refactor to add helper functions `setPreD2RenderOptions` and `setD2RenderFormat`
Also

- add animate_interval parameter
- drop setD2RenderFormat helper function
- use quarto.project.output_directory to set default folder
- revise check for d2 code block class
- replace codeblock text if file is provided
- add support for tala layout
Also:

- add check to see if file exists
- Improve handling of code block formatting (when echo=true)
- remove duplicative processing for options.pad and options.animate_interval
Also add external d2 file to test file argument
Also reorder early return when file + cb text are nil
Also:

- Simplify flow for setting mimetype
- rename setPreD2RenderOptions to setD2Options
- Protect against filename values w/ file extension
- Use assert consistently for errors
- Simplify flow for returning data or outputPath
Also fixes issue w/ broken links inside Quarto projects + error outside of Quarto projects
Copy link
Member

@rcannood rcannood left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

In the meantime, you should have received an invitation to be added as a collaborator to this repo ☺️

@rcannood rcannood merged commit 1dd4dcc into data-intuitive:main Apr 3, 2024
@elipousson
Copy link
Collaborator Author

Awesome, thanks @rcannood. For your interest, I decided to start putting together a stand-alone R package with the goal of creating a custom knitr engine https://elipousson.github.io/d2r/index.html

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