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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

radial menu vendor circuits #20378

Closed

Conversation

GDNgit
Copy link
Contributor

@GDNgit GDNgit commented Feb 6, 2023

What Does This PR Do

title

Why It's Good For The Game

it looks nice and you get a better idea on what you're gettin
also input() bad

Images of changes

Screenshot 2023-02-06 170834
Screenshot 2023-02-06 170900
Screenshot 2023-02-06 170942
Screenshot 2023-02-06 171000
Screenshot 2023-02-06 171020
Screenshot 2023-02-06 171040
Screenshot 2023-02-06 171101
Screenshot 2023-02-06 171120

Testing

yes

Changelog

馃啈
tweak: Vendor circuits now use radial menus
/:cl:

@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally label Feb 6, 2023
@Adrer
Copy link
Contributor

Adrer commented Feb 7, 2023

While I do like the look of this, I'm personally not particularly a fan of having to go through 7 radial menus to find the vendor I am looking for, especially if they have identical sprites.

@GDNgit
Copy link
Contributor Author

GDNgit commented Feb 7, 2023

While I do like the look of this, I'm personally not particularly a fan of having to go through 7 radial menus to find the vendor I am looking for, especially if they have identical sprites.

They're two different items, smartfridges are less distinct because they share the same sprite for most of the different smartfridges

@procdrone
Copy link
Contributor

Maybe a TGUI with a searchbar would be more fitting here?

@TrashDoxx
Copy link
Contributor

Maybe a TGUI with a searchbar would be more fitting here?

I agree.

@farie82 farie82 added the Tweak This PR tweaks something ingame label Feb 8, 2023
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting approval This PR is waiting for approval internally and removed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally labels Feb 8, 2023
@DoctorDrugs
Copy link
Contributor

a couple of the design team aren't fans of a radial menu with a huge amount of options like this that require sorting through. A searchable list of proper names (Medicine Smartfridge, Produce Smartfridge, etc) with a search bar would fit this much better than a radial menu we feel.

@GDNgit
Copy link
Contributor Author

GDNgit commented Feb 25, 2023

I got another idea...

@GDNgit GDNgit closed this Feb 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting approval This PR is waiting for approval internally Tweak This PR tweaks something ingame
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants