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

Color-coding operations #80

Open
wants to merge 33 commits into
base: main
Choose a base branch
from
Open

Conversation

distay0xGit
Copy link

This PR changes ArrayV to allow operations to be color-coded, with a legend included, if given operations it is able to color-code.

@Gaming32
Copy link
Owner

Seems good, but I do have a few optimizations in mind... (such as not creating an array from a keyset on every frame)

@Gaming32 Gaming32 added enhancement New feature or request new-feature labels Feb 24, 2022
@Gaming32
Copy link
Owner

@distay0xGit Can you fix the checkstyle and build issues?

@Gaming32
Copy link
Owner

This is not a sorts PR. Can you please remove Explicium Sort?

@Gaming32
Copy link
Owner

Git, what are you doing?

We'll see if we can work out a better solution.
Gaming32 and others added 9 commits March 30, 2022 19:59
Git, what the heck?
oops! bad coding practice
AtomicInteger-related type changes I desperately want to avoid a conflict for and removing an interface from ArrayVlist
_why doesn't it want me to change the packages imported for ArrayVList_
the conflict checker is running off black magic and all the spells in it are specifically against pushing Groovy support into this PR
I just hope I can revert this weird hack after this PR is brought up to speed with all the changes in main
@rylid-umil
Copy link

Could the custom color coding be a toggleable option? like to switch between color-coded operations and normal highlighting?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request new-feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants