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

Implement support for user-configurable script export locations #2

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

Conversation

deviaze
Copy link

@deviaze deviaze commented Oct 14, 2024

Adds an easy way to change the auto-export location by clicking an instance in Explorer then clicking the relevant "Change Parent" button for either Client, Server, or Tooling.

  • Output locations are now displayed with actual Roblox paths (like Client Export Location: ReplicatedStorage["ZapClient"]) so users know exactly where their scripts are going.
  • Every time a user changes their Roblox Studio selection, starts typing in Zappy plugin textboxes, or clicks a "Change Parent" button, the rest of the Zappy UI updates. This makes it really responsive to change settings and figure out where scripts will get exported.
  • Made major changes to the plugin's UI and src/Zappy/init.server.luau to accommodate the new functionality.
  • The new Client Output/Server Output/Tooling Output sections can each be individually collapsed and are nested under a parent VerticalCollapsibleSection named Output Settings.
  • Client/Server/Tooling buttons are color-coded for sanity and don't take up the whole X-axis to visually distinguish them from the Generate button.
  • Uses CollectionService tags under the hood to keep track of state and save output location instances between Studio sessions.
  • Updated README.md to showcase the new UI and how to use it.

The new UI with the new menus expanded looks like:

image

@Ultrasonic1209 Ultrasonic1209 self-assigned this Oct 16, 2024
we may have to break backwards compatibility w.r.t configs as it's no longer reasonably feasible to do so
lsp didn't catch it probably because it choked while reading zaysynth (i can't blame it)
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.

2 participants