diff.rs icon indicating copy to clipboard operation
diff.rs copied to clipboard

Show something different for there is no conent.

Open HWienhold opened this issue 1 year ago • 2 comments

Hey, i realized there is not really anything displayed when selecting folders. So naturally i would go for displaying a gif with a carlton dance instead of the folder as a file

Alternatively might also be an idea to display diffs from contained files...

HWienhold avatar Oct 31 '24 19:10 HWienhold

You have a point there :D I think it would be good to show something.. although, I'm not fully sold on the dance 😆

I had some ideas too:

  • Could display a file list... but doesn't make so much sense, because there is already one on the left
  • Could display diffs from all contained files
  • Could make it so you can't select/click on folders
  • We could also go the GitHub/GitLab route and render the README.md (if there is one in the folder). Although, I don't know if there would be much value in doing so

If we show diffs from contained files, there are some advantages and disadvantages:

  • Downside: Might potentially be a lot of data, which could make the user interaction slow (there are ways around that, we can do some hacky stuff like lazy rendering only files that are visible, I wrote a hook for that in the yew-hooks crate called use_visible() that lets us do that.
  • Advantage: it means you can search through all files using the browser built-in Ctrl+F

To be honest I don't fully know what the best move would be.

xfbs avatar Oct 31 '24 21:10 xfbs

I'm not fully sold on the dance 😆

Bummer

* We could also go the GitHub/GitLab route and render the `README.md` (if there is one in the folder). Although, I don't know if there would be much value in doing so

I think diif.rs is more comparable to a PR view than to a repo view so i d say the Githuh route[^1] and imho the more useful for any user would prob be

[^1]: i think the all diffs -View might be actually the only view there is for diffs - at least for gitea with anchor for files and clickhandler on folder s just expand/reduce filetree - dont know github by heart, but presumably it s similar

* Could display diffs from all contained files

Especially if you d use it to check all the changes - it seems just easier to scroll to a few files than to open them all separately. But might be a good idea for the "repo browsing case"

* Could make it so you can't select/click on folders

well you should be able to expand the folder to see the files :-p.

I think the advantage you mentioned is quite huge - and the disadvantage - well in the end it could be a feature, nobody needs to use?

I wrote a hook for that in the yew-hooks crate called use_visible() that lets us do that.

Damn, thats helpful :-o

HWienhold avatar Oct 31 '24 23:10 HWienhold