Lars Kappert comments

Results 640 comments of


                                            Lars Kappert

Support Reveal's Lazy Loading

If anyone's up to look into it and submit a PR, I'm happy to look into that.

Print to PDF without fragments

I think you can custom css to hide things, e.g. `.animation { display: none }`

Update README.md for pdf print instructions

Thanks, @Zacaria! Are you on the latest version of reveal-md? The docs were updated to match the [Reveal.js v5 update](https://github.com/hakimel/reveal.js/releases/tag/5.0.0).

Update README.md for pdf print instructions

> Do you want me to try and fix it ? I would love that! Personally I'm not using the print/PDF functionality at all.

Patch for issue#409 is not sufficient and the protection can be bypassed by an attacker

Thanks @W0rty, please open a PR if you have a patch

Fix module resolution across multiple dependent workspaces with incompatible tsconfigs

Thanks for the PR! I _really_ appreciate you went deep into the codebase. I think you're on to something and this might fix some nasty bugs I wasn't aware of...

Fix module resolution across multiple dependent workspaces with incompatible tsconfigs

> we can now get rid of `internalWorkspaceFilePaths` altogether? Sorry, scratch that. After thinking it through a little bit more, maybe this is a more viable approach: - Never cache...

Fix module resolution across multiple dependent workspaces with incompatible tsconfigs

That's great to hear. Thanks for working on this! It was not an intuitive thing to do, and by default leads to surprising/incorrect behavior, but the performance/memory improvements by combining...

Fix module resolution across multiple dependent workspaces with incompatible tsconfigs

@dbudzins Just curious: is this still something you fancy working on?

Fix module resolution across multiple dependent workspaces with incompatible tsconfigs

I'll look into it and give it a shot today or tomorrow, and report back :)