Skip to content

Updates#23

Closed
liambeguin wants to merge 9 commits into
gregzuro:masterfrom
liambeguin:updates
Closed

Updates#23
liambeguin wants to merge 9 commits into
gregzuro:masterfrom
liambeguin:updates

Conversation

@liambeguin
Copy link
Copy Markdown

minor changes:

  • reorder diagramTypes and kroki-test.md to match kroki.io list
  • add DBML
  • add Symbolator
  • add Tikz
  • center rendered figures
  • limit rendered figure width

Signed-off-by: Liam Beguin <liambeguin@gmail.com>
This allows us to fold sections in Obsidian, making it easier to ignore
some formats.

Signed-off-by: Liam Beguin <liambeguin@gmail.com>
Signed-off-by: Liam Beguin <liambeguin@gmail.com>
Signed-off-by: Liam Beguin <liambeguin@gmail.com>
Signed-off-by: Liam Beguin <liambeguin@gmail.com>
Signed-off-by: Liam Beguin <liambeguin@gmail.com>
Signed-off-by: Liam Beguin <liambeguin@gmail.com>
Signed-off-by: Liam Beguin <liambeguin@gmail.com>
Signed-off-by: Liam Beguin <liambeguin@gmail.com>
@gregzuro
Copy link
Copy Markdown
Owner

Thanks @liambeguin — most of this PR landed in 1.3.0 (#29, https://github.com/gregzuro/obsidian-kroki/releases/tag/1.3.0):

  • DBML, Symbolator, TikZ — added to DEFAULT_SETTINGS.diagramTypes and kroki-test.md (the TikZ example is wrapped in \documentclass{standalone} + \begin{document} so Kroki accepts it).
  • Centering — taken, but moved from dest.style = "display:flex;justify-content:center" into styles.css (.kroki-diagram { display: flex; justify-content: center; }) to play nicely with themes and avoid the lint smell.
  • Width/height 100% — also in styles.css as max-width: 100% rather than fixed 100% (lets small diagrams stay small).
  • ⚠️ D2 was already merged separately (commit e788f6d).
  • ⏭️ The diagramTypes array reorder and kroki-test.md heading reorg were skipped — they would have churned existing users' data.json (settings are merged by key, not position, but the diff is unnecessary noise). The reorder is harmless if you want to redo it as a tiny follow-up PR.

Closing — your work is in.

@gregzuro gregzuro closed this May 14, 2026
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