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

todonotes:0.1.0 #1315

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

todonotes:0.1.0 #1315

wants to merge 1 commit into from

Conversation

Tinggaard
Copy link
Contributor

I am submitting

  • a new package
  • an update for a package

Description: Adds todonotes like seen in the todonotes LaTeX package. This package also has the feature to display an outline of the todos.

I have read and followed the submission guidelines and, in particular, I

  • selected a name that isn't the most obvious or canonical name for what the package does
  • added a typst.toml file with all required keys
  • added a README.md with documentation for my package
  • have chosen a license and added a LICENSE file or linked one in my README.md
  • tested my package locally on my system and it worked
  • excluded PDFs or README images, if any, but not the LICENSE

@typst-package-check typst-package-check bot added the new A new package submission. label Nov 22, 2024
@ntjess
Copy link
Contributor

ntjess commented Nov 23, 2024

This looks interesting; would you consider collaborating to make a PR to drafting to solve ntjess/typst-drafting#11 and ntjess/typst-drafting#3 instead of creating another package?

My general perspective is fewer packages with more maintainers is better than splitting the same functionality across a wide area.

I would also consider migrating the drafting margin note functionality into this package if it is preferred by others in the community.

@Tinggaard
Copy link
Contributor Author

I did not see that there were open issues regarding this functionality on the drafting package. I definitely agree that fewer packages is preferable, I just made this because I know that some of my fellow students (and I) rely on the todonotes LaTeX package when writing reports, and I wished to port the functionality to Typst.

The code is not perfect, and I have not tested its functionality with an rtl layout (using inside / outside margins), but it solves the main issues I encountered. I will give the PR a try, and see if I can make it work in the drafting package

@ntjess
Copy link
Contributor

ntjess commented Nov 24, 2024

Sounds great! Please let me know where you want any help / pointers / etc.

I'm in the typst discord as well, if that works for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new A new package submission.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants