Make the data conform to RandomAccessCollection

[MojtabaHs]

It now supports the entire range of hashable collections, including arrays, ranges, sets, and more

[gordan-glavas-codecons]

Thanks for contributing!

[gordan-glavas-codecons]

@MojtabaHs Is there a reason why you set the Data.Element to conform to Hashable? It's not necessary for the library to work, and historically people have complained about this constraint.

[MojtabaHs]

@MojtabaHs Is there a reason why you set the Data.Element to conform to Hashable? It's not necessary for the library to work, and historically people have complained about this constraint.

I think it is a leftover of my experiments with the enhancements I was going to apply to the library, like applying the pixel-perfect spacing instead of the current ambiguous padding which applies 2 spacings in between and on unintended at the sides of the layout.

I will double-check and take care of that in the next few PRs where I cleaned up and documented the code

[gordan-glavas-codecons]

@MojtabaHs Is there a reason why you set the Data.Element to conform to Hashable? It's not necessary for the library to work, and historically people have complained about this constraint.

I think it is a leftover of my experiments with the enhancements I was going to apply to the library, like applying the pixel-perfect spacing instead of the current ambiguous padding which applies 2 spacings in between and on unintended at the sides of the layout.

I will double-check and take care of that in the next few PRs where I cleaned up and documented the code

Thanks, I'll hold off on releasing a new version until that is sorted out.

[MojtabaHs]

Thanks, I'll hold off on releasing a new version until that is sorted out.

I have done it in the PR #15 . Also rebased the other PR based on this change.