Add hyperlink support

[gierens]

We originally merged this feature in #1. Unfortunately it turned out to remove the Copy trait we require in https://github.com/eza-community/eza, therefore we had to revert it again. This issue is just so we don't forget about it, and maybe at some point find a good solution to implement this without breaking eza.

[tertsdiepraam]

I've just learned about this issue, so I might be totally off here.

If the issue was that Style can no longer be copy, then maybe Style should not contain the hyperlink. Instead it could be ANSIString (with an Option<Cow<str>>) or maybe a HyperlinkedANSIString or something like that. That way, you can keep everything Copy in existing codebases until you actually need hyperlinks.

Alternatively, you could wrap Style in something like HyperlinkStyle, which does not use Copy.

Happy to try to create a PR for something in this direction if you want.

[gierens]

Hey @tertsdiepraam

sorry for the late reply, have been a little too busy for my taste in the last weeks.

I've just learned about this issue, so I might be totally off here.

If the issue was that Style can no longer be copy

Yep, that was the problem ... I only realized after recompiling eza with this that #1 made it incompatible due to the removal of Copy.

then maybe Style should not contain the hyperlink. Instead it could be ANSIString (with an Option<Cow<str>>) or maybe a HyperlinkedANSIString or something like that. That way, you can keep everything Copy in existing codebases until you actually need hyperlinks.

Alternatively, you could wrap Style in something like HyperlinkStyle, which does not use Copy.

Thanks for taking a closer look into this ... So, I think your first suggestion sounds a lot cleaner.

Happy to try to create a PR for something in this direction if you want.

That'd be awesome! Thanks!