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

[ENHANCEMENT] Frames/Grouped Fields in Events #3987

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

KoloInDaCrib
Copy link
Contributor

Does this PR close any issues? If so, link them below.

2876 shoutouts hundrec for pointing out the issue alongside others closed by linear

Briefly describe the issue(s) fixed.

had to change the grid to a vbox w/ hboxes sorry eric the frames didn't look good otherwise
anyways frames!!! they have a field called "children" where you put in SongEventFields and then those fields get rendered inside the frames

Include any relevant screenshots or videos.

Toolbox
image

Fucked up JSON w/ the fields
image

How they're added
image

Recursive Frames because it was funny (takes a bit to load them though!)
image

@github-actions github-actions bot added size: medium A medium pull request with 100 or fewer changes. pr: haxe PR modifies game code. and removed size: medium A medium pull request with 100 or fewer changes. labels Jan 11, 2025
@KoloInDaCrib
Copy link
Contributor Author

KoloInDaCrib commented Jan 11, 2025

wait shit i forgot about loading the events uhhh gimme a minute

edit: "fixed" it (its literally due to that one bug that i fixed in a previous pr fml), also why does github actions immidiately remove the size label this is for my 2nd pr in a row does the bot not like my prs,,,,,

@KoloInDaCrib KoloInDaCrib marked this pull request as draft January 11, 2025 14:14
@github-actions github-actions bot added size: medium A medium pull request with 100 or fewer changes. and removed size: medium A medium pull request with 100 or fewer changes. labels Jan 11, 2025
@KoloInDaCrib KoloInDaCrib marked this pull request as ready for review January 11, 2025 14:42
@EliteMasterEric EliteMasterEric added the status: pending triage Awaiting review. label Jan 17, 2025
@Hundrec Hundrec added type: enhancement Involves an enhancement or new feature. size: medium A medium pull request with 100 or fewer changes. labels Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: haxe PR modifies game code. size: medium A medium pull request with 100 or fewer changes. status: pending triage Awaiting review. type: enhancement Involves an enhancement or new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants