Skip to content

[Feat] Add utilities for pandoc divs and spans -- Part 2#6032

Merged
nathanlesage merged 11 commits into
Zettlr:developfrom
benniekiss:pandoc-ui
Feb 7, 2026
Merged

[Feat] Add utilities for pandoc divs and spans -- Part 2#6032
nathanlesage merged 11 commits into
Zettlr:developfrom
benniekiss:pandoc-ui

Conversation

@benniekiss

Copy link
Copy Markdown
Collaborator

Description

This PR adds utilities for pandoc fenced divs and bracketed spans, including formatting bar improvements and a popover utilty.

See #5848 for more information

Changes

Additional information

Tested on:

@benniekiss benniekiss changed the title [Feat] Add utilities for pandoc divs and spans [Feat] Add utilities for pandoc divs and spans -- Part 2 Nov 30, 2025
@benniekiss benniekiss force-pushed the pandoc-ui branch 3 times, most recently from 393fd4c to 6ce2f08 Compare December 13, 2025 15:38
@nathanlesage

Copy link
Copy Markdown
Member

Quick heads up: Going to postpone this to after 4.0!

@benniekiss benniekiss force-pushed the pandoc-ui branch 2 times, most recently from 8ceaf3d to e2117f7 Compare January 3, 2026 21:49

@nathanlesage nathanlesage left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good and adds a ton of value, there are just a few minor things to change (plus one potentially larger refactor).

Comment thread source/app/service-providers/config/get-config-template.ts Outdated
Comment thread source/app/service-providers/config/get-config-template.ts Outdated
Comment thread source/common/modules/markdown-editor/commands/markdown.ts Outdated
Comment thread source/common/modules/markdown-editor/commands/markdown.ts
Comment thread source/common/modules/markdown-editor/commands/markdown.ts
Comment thread source/common/modules/markdown-editor/commands/markdown.ts Outdated
Comment thread source/common/modules/markdown-editor/commands/markdown.ts Outdated
Comment thread source/common/modules/markdown-editor/util/configuration.ts Outdated
@benniekiss

Copy link
Copy Markdown
Collaborator Author

Its been a minute since I really looked through this, but I agree with pretty much all your comments!

I'll dedicate some time to this over the next couple days to respond in detail

* update name scheme from `fence`/`bracket` to `div`/`span`
* restore highlight default to `==`
* re-organize code

@nathanlesage nathanlesage left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We're getting there. I really like how this is developing! :)

Comment thread source/common/pandoc-util/parse-pandoc-attributes.ts
Comment thread source/common/pandoc-util/parse-pandoc-attributes.ts
Comment thread source/common/pandoc-util/parse-pandoc-attributes.ts Outdated
Comment thread source/common/modules/markdown-editor/commands/markdown.ts Outdated
* strip braces if the string has them, otherwise
* process the attributes as usual
* removed some braces from tests
* simplified test structure since the function no longer errors

@nathanlesage nathanlesage left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So, now, finally. Looks good!

@nathanlesage nathanlesage merged commit dcd9faa into Zettlr:develop Feb 7, 2026
1 check passed
@benniekiss benniekiss deleted the pandoc-ui branch February 7, 2026 14:42
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