Skip to content

WIP: Courtroom transitions (HEAVY CLEANUP REQUIRED)#869

Closed
in1tiate wants to merge 39 commits intoAttorneyOnline:masterfrom
in1tiate:master
Closed

WIP: Courtroom transitions (HEAVY CLEANUP REQUIRED)#869
in1tiate wants to merge 39 commits intoAttorneyOnline:masterfrom
in1tiate:master

Conversation

@in1tiate
Copy link
Member

I wrote this for a personal project that will never see completion. It is fully functional but I had to remove a not-insignificant amount of existing functionality to make it work. The pair character has been repurposed. It also requires configuration to work properly. I will answer any questions posed to me about how exactly this works.

Here's what it looks like.
https://user-images.githubusercontent.com/32779090/192168204-abde05ee-9309-4e4f-84f1-bca2e48829f5.mp4

@in1tiate in1tiate marked this pull request as draft September 25, 2022 22:36
@Iamgoofball
Copy link
Collaborator

question:
does this function by having 1 really long courtroom /png background icon that we pan between spots on, or does it automatically generate the camera smear frames?

@in1tiate
Copy link
Member Author

question: does this function by having 1 really long courtroom /png background icon that we pan between spots on, or does it automatically generate the camera smear frames?

The former. Automatic generation of the smear frames proved too complex a solution.

@Iamgoofball
Copy link
Collaborator

Iamgoofball commented Nov 13, 2022

it's possible to autogenerate it but i think what we'd want to do is autogenerate the smear frames ahead of time with a tool instead of trying to do it real time with the client, that way we can load in the actual game smear frames where they exist and generate for the homebrewed courtrooms

@Salanto
Copy link
Contributor

Salanto commented Jan 17, 2023

Going to put this in the 2.11 backlog for now. Given that we already redo a lot, read ALL, of courtrooms core design, an addition like this might give a client that is currently purely reworked a little bit more spice.

It also is easer to add once you don't have to fight seven layers of AO jank.

@stonedDiscord
Copy link
Member

11 conflicting files

this is not gonna be fun

@in1tiate
Copy link
Member Author

Closing. If I'm going to add this upstream it will need to be cherry-picked, and this branch is active again.

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.

4 participants