Merge #72, #59, #45, Refactoring and Theme System

[VitoBarra]

Merged https://github.com/DigitalLogicSimCommunity/Digital-Logic-Sim-CE/pull/45. Made some alterations to to fix bugs, including making it work with zoom, making it work with small .bin files, and changing the IO to busses for ease of use. Merged https://github.com/DigitalLogicSimCommunity/Digital-Logic-Sim-CE/pull/59. Merged https://github.com/DigitalLogicSimCommunity/Digital-Logic-Sim-CE/pull/72. Fixed 2 minor bugs.

In addition, I have done

• Major refactoring of the project. (The important part of this PR :P)
• Added a Theme system for customizable wire and signal colors.
  • Change and Save\load Selected theme Color (Theme System completion)
  • Ability to Save\load Group size
  • Ability to Save\load Signal WireType
• Enhanced the signal property menu to allow for group size and bus value changes by adding functionality and new keybindings:
  • Q and E: to cycle between default themes.
  • R and F: Change the group size.
  • Save changes made in the GroupProperty menu using Enter and Space.
  • Discard changes made in the GroupProperty menu using Exit.
  • Use up arrow (+1) and down arrow (-1) to adjust integer input fields in the GroupProperty menu.

QOL of the main menu: now in the load project menu you can

• Delete a project
• Renominate a project
• Duplicate a project

Fix:

• issue #47
• issue #48 and related #49

notes

To implement these features, the format of save files needed to be reworked, for now the format is not very efficient, more data than needed are stored for each subchip, this maybe will be fixed in the next release

Credits haven't been updated

[crispeeweevile]

When will this PR be accepted? it's got a lot of cool features

[901238746]

This is so cool, why hasn't it been approved yet?

[901238746]

Man someone should approve this. Idk how to code but I dont think any of it is going to alt F4 the game

[901238746]

I'd love this pull request because Im making a 11x16(11-bit 16 addresses) For my ALU and later I might need to make it 16 by 16 so that it would work with a better version of my ALUs LU(Logic Unit) so it can do more operations and comparisions. I just need more space. I made this in the original and I would hate it if I had to make it again.

[VitoBarra]

This should be good to go, some parts of the main menu are not very nice but they work

[AOx0]

Hey, I've been trying the PR, seems good in the Unity Editor, so far. However, when producing a release build, the input system just stops working with pins, so wires don't work nor the activation/deactivation of the pins signal.

I suspect it's an issue with the new Unity Input system. I can reproduce the issue in the Unity Editor when changing the Event System components to the new suggested Input System UI Input Module.

My next thing on the list is to build the main branch's version and see if the issue is present there too, so I can discard the source code’s fault. Otherwise, if the main branch does work correctly, maybe it would be necessary to git bisect and revert whatever the change caused the button's fault.

I don’t know if you are noticing the same on your end? It may be an issue with Unity in Linux.

[VitoBarra]

It happens on my machine (Windows) as well. I will work on it, but after a first investigation, I found out that issues arise when the SebInput Module is used. This module depends on the new input system, so probably needs to be updated or not used, as in the case of the signal handler that works fine. I added a package (in the 3rdparty folder) that should be useful for debugging this problem.

[AOx0]

Thank you as always Vito!

Just one little nitpick, I don't know if this is intentional, but the changing of pin colors does not display the correct color to be placed. i.e. It displays red but when placed the pin is green.

I'm willing to merge as-is and fix later tho, so I'll be just waiting for your confirmation whether to commit the merge or not.

[VitoBarra]

I'm not sure that I understand the concern, but it is intentional that selecting the color for signals (input/output chip pins) and cables requires separate actions. With this said the PR should be ready c:

[AOx0]

Thank you Vito :D