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

Uh... I did some things #418

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

BionicFelps
Copy link

@BionicFelps BionicFelps commented Nov 7, 2016

This is an accept at your own risk courtesy request, I haphazardly updated everything to the latest libraries, and refactored where necessary... A couple things to note, the unit tests die horribly after completing successfully according to my IDE, a couple files seem to have been misplaced from the plugins directory (.gitignore), and I prevented a crash with a try/catch block in followersUiMediator, but I didn't actually do anything about it other than print out the fact that it happens.

Some files were removed entirely , mostly because I didn't know what they did and they were causing problems >.>

You might be better off to branch this, and fix up the things I likely trampled over, basically the entire point of this pull request is to make you aware of its existence, and not much else... The unit test problem may be because I deleted one of the tests, as it was testing a file that I also deleted >.> But I don't know much about unit testing...

Oh, and I didn't update the plugins, so they likely won't work...

This was an attempt to try to fix problems our level designer was having, his specific complaint was to do with the exception I added a try/catch block for, updating to latest libraries I assumed would generally improve stability, but honestly with all the refactoring that was required who knows at this point...

Call it an experimental build >.>

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 6.002% when pulling dbe4522 on BionicFelps:master into ba2a52e on UnderwaterApps:master.

@BionicFelps
Copy link
Author

BionicFelps commented Nov 11, 2016

Found a couple issues, the color chooser always goes to black, likely because I deleted the custom color chooser, and the export settings don't properly save, no idea why that is.

Fixed both of these and commited to my fork.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 6.002% when pulling d1e829c on BionicFelps:master into ba2a52e on UnderwaterApps:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 6.002% when pulling d1e829c on BionicFelps:master into ba2a52e on UnderwaterApps:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 6.001% when pulling fde708c on BionicFelps:master into ba2a52e on UnderwaterApps:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 6.0% when pulling 1b37510 on BionicFelps:master into ba2a52e on UnderwaterApps:master.

@azakhary
Copy link
Contributor

I'll try to have some time at monday to test this well. Does this also update version of VisUI? if so, does it break Color Picker?

@azakhary
Copy link
Contributor

It looks pretty cool and we also have another branch internally with some things fixed, as well as Spine stuff updated. Maybe it may be a good idea to merge this all together, test and push as new version. I'll try to make time for this.

@BionicFelps
Copy link
Author

BionicFelps commented Nov 14, 2016

I eliminated the customColorPicker class (yes, the new VisUI broke it, but it has its own color picker now), and related classes, used the stock VisUI color picker, and then fixed a minor bug from that.
As I said, I can't vouch for it being bug free, you might also want to look at UISkin.json as there were a lot of visUI changes in the way things worked, for it to be completely correct would require new assets being made for the UI, check boxes for instance work a little differently, basically I picked from existing assets things so that it wouldn't be noticeable that something was wrong, but instead of having a separate entire image for a checked checkbox, it now uses an image for a check, and an image for a box, and composites them when the box is ticked. Since the check tick is placed exactly centered on the check box, I just used the graphic of the checked checkbox. Visually nothing is wrong, but I know that it's drawing an empty checkbox and then a ticked checkbox ontop of it >.>

Basically what I did was change the versions in the build.gradle, attempt to compile, fix the errors it brought up, rinse, repeat until it worked, I also used the VisUI changelog to determine the correct thing to use in cases where classes were removed, like replacing NumberSelectors with Spinners. I did this over the course of a few hours with almost know deep knowledge of Overlap, OR VisUI codebases, as far as I can tell everything seems to work properly.

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.

None yet

3 participants