Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Layout + tinting rework #927

Merged
merged 1 commit into from
Nov 11, 2020
Merged

Conversation

GladOSkar
Copy link
Contributor

@GladOSkar GladOSkar commented Nov 11, 2020

In response to #925

I restructured and cleaned up a lot which turned out not to be necessary for the final solution but might be nice anyways. I don't think i introduced any regressions but that should be tested.

Alternating tint is now anchored to the right by counting the visible blocks beforehand and starting the alternating sequence accordingly on the left.

I barely have any rust experience, please lmk if something could be done better :)

src/util.rs Outdated Show resolved Hide resolved
@GladOSkar
Copy link
Contributor Author

There ya go

@ammgws
Copy link
Collaborator

ammgws commented Nov 11, 2020

Thanks for adding the comments as well!

@ammgws ammgws merged commit 0132196 into greshake:master Nov 11, 2020
@GladOSkar GladOSkar deleted the layout-tint-rework branch November 11, 2020 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants