touying-greyc-ambrosia:0.1.0

[inspiros]

I am submitting

• a new package
• an update for a package

Description: touying-greyc-ambrosia is a Touying theme for creating presentation slides with various flavors. It is an unofficial template of the GREYC lab at the Université de Caen Normandie, France.

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

• ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

[OrangeX4]

Consider switching to 0.6.3, a version that fixes a lot of bugs. And make a simple little change:

For theme developers upgrading to v0.6.3:

1. Move config to the last position in utils.merge-dicts to allow user overrides:

   // Before
   self = utils.merge-dicts(self, config, config-page(...))
   
   // After
   self = utils.merge-dicts(self, config-page(...), config)

2. Replace paper with utils.page-args-from-aspect-ratio to support arbitrary aspect ratios:

   // Before
   config-page(paper: "presentation-" + aspect-ratio, ...)
   
   // After
   config-page(..utils.page-args-from-aspect-ratio(aspect-ratio), ...)

[inspiros]

Thanks for the suggestion. I would like to migrate to newer version whenever possible.

However, there seems to be an annoying bug exclusive to v0.6.3 that inserts an empty slide when one uses #show: appendix. I just opened an issue in the Touying repository touying-typ/touying#312. I have been trying to rule out the cause of the bug and fix it myself, but no luck yet.

[OrangeX4]

I did some research, and currently this issue is a bit tricky, so we can temporarily use 0.6.2 for now.

[elegaanz]

Thank you for this template. Do you want to wait for the Touying issue to be solved or should I already merge this version (once my review comments below are addressed)?

[elegaanz]

For images in your README, can you please:

• use permalinks (links with a git commit hash or tag name and not a branch name) so that they still reflect what this version of the package does even if you update them in the future in your repository.
• add a short alternative description, to give an idea of what the slides look like to people with poor internet connections or who rely on screen readers.

[elegaanz]

I think you are missing a : after the show in the last code snippet here.

[elegaanz]

The previous remark applies for links in general: permalinks should be preferred when linking to your own repository.

[elegaanz]

What about the various logos you ship with this template? Are you allowed to distribute them, and if so under which terms? Can you please check and make that information more explicit?

[inspiros]

I will ask my directeur de thèse, who happens to be the head of communication in the lab, and return to you :/