Feature request: option to only show first n files

rtfpessoa / diff2html

Pretty diff to html javascript library (diff2html)

https://diff2html.xyz

MIT License

2.79k stars 272 forks source link

Open simhnna opened 8 months ago

simhnna commented 8 months ago

We have a couple of very large commits that take ages to load and sometimes even crash the browser tab.

Would you be open to a merge request that does one of the following:

add a config option to only show the first n files followed by a message saying that m file diffs are not shown
add a config option to only show the file list

xrgarcia commented 8 months ago

i would really need a loading spinner (if that doesn't already exist). How about some sort of paging?

rtfpessoa commented 2 months ago

👋 I have this lazy loader I started working but never finished maybe you can take a look and see it if works for you https://github.com/rtfpessoa/diff2html/pull/410 ?

xrgarcia commented 2 months ago

Thank you! I'll take a look