Skip to content

Conversation

@ilyvion
Copy link
Contributor

@ilyvion ilyvion commented Aug 3, 2022

Fixes #100

I didn't really get any feedback on the issue whether or not this was a wanted change, but since PRs talk louder than issues, here's the fix if you want it. 😄

@ilyvion
Copy link
Contributor Author

ilyvion commented Aug 3, 2022

This change changes slotmap from
image
(as per #100) into
image
Just for the heck of it, I wanted to see what it would look like with both, which could look like this, e.g.:
image
Let me know if the latter is preferable, and I can commit changes to that effect instead.

@tinaun
Copy link
Contributor

tinaun commented Aug 3, 2022

thank you! I think it looks better with just the max version

@technetos technetos merged commit 64fafd0 into technetos:master Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

?crate command shows most recently released version rather than the latest version

3 participants