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

Some bugfixes (upscaling, preanimation, webp on char buttons, viewport layout) #202

Merged
merged 6 commits into from
Nov 23, 2023

Conversation

SymphonyVR
Copy link
Contributor

@SymphonyVR SymphonyVR commented Nov 22, 2023

This fixes:

Changes:

  • Now every image, except the desk (which needs work done), is centered on the X and Y axis in the viewport. This makes it more similar to how AO 2.8+ manages the viewport by default.
  • Adds support for webp buttons: This was the only place that didn't have webp support.
  • Better upscaling/downscaling detection. It applies pixelated/fast to every gif and apng, while the png and webp have the smooth filter.

Current state:
Stable

By default it used to make all images pixelated.
Fix for: Empty messages don't play the preanimation AttorneyOnline#191
Fix: background scaling AttorneyOnline#92
Changes: Now every image on the viewport is centered in the X  and Y.
@SymphonyVR SymphonyVR marked this pull request as draft November 22, 2023 16:01
Copy link
Member

@stonedDiscord stonedDiscord left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove the extra top: and we can merge it

top: 0;
left: 0;
top: 50%;
left: 50%;
transform: translate(-50%, -50%);
}

#client_court {
position: absolute;
height: 100%;
top: 0;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove the old top

}

#client_court {
position: absolute;
height: 100%;
top: 0;
left: 0;
top: 50%;
left: 50%;
transform: translate(-50%, -50%);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a known trick to center stuff in css but it doesn't work in all situations, need to test this

@@ -311,17 +325,20 @@

.client_bench {
position: absolute;
height: auto;
width: 100%;
height: 100%;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some benches are only a picture of the bench and this way they would stretch the entire height of the viewport and block it

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bancodefensa

@stonedDiscord
Copy link
Member

i'm willing to go for it with the benches, people should keep their content up to date

@SymphonyVR SymphonyVR marked this pull request as ready for review November 22, 2023 16:34
@SymphonyVR
Copy link
Contributor Author

Should be ready now, I think.

@stonedDiscord stonedDiscord merged commit 0bde198 into AttorneyOnline:master Nov 23, 2023
1 check passed
@OmniTroid
Copy link
Contributor

Did this fix #92 completely or partially?

@SymphonyVR
Copy link
Contributor Author

Did this fix #92 completely or partially?

Is partially fixed @OmniTroid. See below for more details

Bugs as of now:

Now the image of the character is centered:

image

But the background and desk isn't centered at all on 16:9:

image
Tho that only happens on judge.
In all the other positions the background stretches, minus the desk.

Compared to AO2 (16:9 judge):

image

4:3 (works fine)

image

16:9

image

16:9 (on AO2):

image

4:3 (witness)

image

16:9 (witness)

image

16:9 (AO2 - witness)

image

4:3 (Prosecution)

image

16:9 (Prosecution)

image

16:9 (AO2 - Prosecution)

image

Expected behaviour (coming from AO2 client):

  • Background, Character and desk should be centered and not stretched.

For it to not have blank space on the sides, content creators would need to fill it (make the background and desk with 16:9 aspect ratio in mind).

If anyone asks the default aspect ratio is 4:3 as of now, making #180 solved (idk when it got resolved).

@OmniTroid
Copy link
Contributor

Thanks for the detailed breakdown. Would be great to have this in issue #92

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.

3 participants