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

Chemical simulator TGUI #6568

Merged
merged 26 commits into from
Jul 11, 2024
Merged

Conversation

kiVts
Copy link
Contributor

@kiVts kiVts commented Jun 26, 2024

About the pull request

The one of last NanoUIs are nuked, chemical simulator was moved to tgui, along with other changes needed to accomplish that.
Consider that the 1.5 part of research rework

Conflicting and combining properties are now in GLOB lists, one bugfix in save_new_properties, it literally never worked, you cannot add conflicting properties in create, relate in conflict, and etc etc

Its working and is 100% complete, but I need to do some final polish(grammer, looks, etc). but if you dont mind ugliness then its working. Done and looks pretty cool
resolves #6644

Explain why it's good for the game

Conflicting properties feature, less nanouis, more friendly to new features.

Testing Photographs and Procedure

Screenshots & Videos

image
image

Changelog

🆑 Kivts
ui: The Company spent some money to upgrade the chemical simulator to work on the TGUI, Including a few prepaid features.
/:cl:

@github-actions github-actions bot added the UI deletes nanoui/html label Jun 26, 2024
@kiVts
Copy link
Contributor Author

kiVts commented Jun 30, 2024

image

@kiVts kiVts marked this pull request as ready for review June 30, 2024 03:38
@kiVts
Copy link
Contributor Author

kiVts commented Jun 30, 2024

image

cm13-github added a commit that referenced this pull request Jul 3, 2024
@cm13-issue-bot cm13-issue-bot mentioned this pull request Jul 4, 2024
cm13-github added a commit that referenced this pull request Jul 4, 2024
@ghost ghost mentioned this pull request Jul 5, 2024
3 tasks
@cm13-github
Copy link
Contributor

Conflicts have been resolved. A maintainer will review the pull request shortly.

@cm13-github cm13-github removed the Merge Conflict PR can't be merged because it touched too much code label Jul 7, 2024
cm13-github added a commit that referenced this pull request Jul 8, 2024
cm13-github added a commit that referenced this pull request Jul 10, 2024
@harryob harryob marked this pull request as draft July 10, 2024 18:30
cm13-github added a commit that referenced this pull request Jul 10, 2024
cm13-github added a commit that referenced this pull request Jul 10, 2024
kiVts and others added 2 commits July 10, 2024 18:41
@kiVts kiVts marked this pull request as ready for review July 10, 2024 22:47
cm13-github added a commit that referenced this pull request Jul 11, 2024
cm13-github added a commit that referenced this pull request Jul 11, 2024
@harryob harryob added this pull request to the merge queue Jul 11, 2024
Merged via the queue into cmss13-devs:master with commit 87c4a81 Jul 11, 2024
28 checks passed
cm13-github added a commit that referenced this pull request Jul 11, 2024
@Toastina
Copy link

just wanna note. you cant relate research chem with other research chem. and basic chemicals+ medical chem. I'm testing it right now and its pain

@kiVts
Copy link
Contributor Author

kiVts commented Jul 15, 2024

just wanna note. you cant relate research chem with other research chem. and basic chemicals+ medical chem. I'm testing it right now and its pain

read the issue you created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI deletes nanoui/html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

synthesis simulator UI runtime
4 participants